From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (using TLSv1 with cipher CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 82C1A1A1E28 for ; Fri, 21 Oct 2016 12:58:55 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga102.fm.intel.com with ESMTP; 21 Oct 2016 12:58:56 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,526,1473145200"; d="scan'208";a="1048170137" Received: from jessica1-mobl.amr.corp.intel.com (HELO localhost) ([10.252.136.91]) by orsmga001.jf.intel.com with ESMTP; 21 Oct 2016 12:58:46 -0700 MIME-Version: 1.0 To: Ard Biesheuvel , "edk2-devel-01" , "Kinney, Michael D" , "Leif Lindholm" , "afish@apple.com" Message-ID: <147707992484.13791.10042868456965197315@jljusten-ivb> From: Jordan Justen In-Reply-To: Cc: "Laszlo Ersek" , "Gao, Liming" References: User-Agent: alot/0.3.7 Date: Fri, 21 Oct 2016 12:58:44 -0700 Subject: Re: [Bug 164] Add the build option "/D DISABLE_NEW_DEPRECATED_INTERFACES" in package DSC files X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 21 Oct 2016 19:58:55 -0000 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On 2016-10-21 12:37:21, Ard Biesheuvel wrote: > I don't remember seeing any discussion regarding > DISABLE_NEW_DEPRECATED_INTERFACES on the list, so I am a bit surprised > seeing these bugs being filed and assigned. > I agree. Also, the terminology seems confusing. 'new deprecated' seems like a contradiction. I guess it means 'newly deprecated', but that seems like a term that is quickly going to become obsolete. Soon there will be old deprecated items that are disabled with this switch. DISABLE_DEPRECATED_INTERFACES sounds better. But, shouldn't we have platforms opt-in to using the deprecated interfaces rather than adding DISABLE_NEW_DEPRECATED_INTERFACES to the build command line for every EDK II platform? Not using deprecated items should be the default for EDK II platforms. If a platform has to opt-in to the deprecated content in their .dsc, then it is obvious that they are relying on deprecated functionality. So, I guess I'd propose adding ENABLE_DEPRECATED_INTERFACES instead. -Jordan > Before making any such changes, I would like a strong commitment from > other package owners that deprecating an interface brings along with > it the responsibility to update all existing callers, otherwise > setting this define will only result in more breakage, and ARM has > seen its share of inadvertent breakage in the past when changes to > core code were made without taking other architectures into account. > > On 21 October 2016 at 02:21, wr= ote: > > https://bugzilla.tianocore.org/show_bug.cgi?id=3D164 > > > > yonghong.zhu@intel.com changed: > > > > What |Removed |Added > > -----------------------------------------------------------------------= ----- > > Priority|Lowest |Normal > > Status|UNCONFIRMED |CONFIRMED > > Assignee|michael.d.kinney@intel.com |ard.biesheuvel@linaro.= org > > Ever confirmed|0 |1 > > Release(s) the| |EDK II Trunk > > issues must be| | > > fixed| | > > > > --- Comment #1 from yonghong.zhu@intel.com --- > > Assign to Package owner. > > > > -- > > You are receiving this mail because: > > You are the assignee for the bug. > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel