Shenglei, I would prefer we work towards the goal of removing the use of the Intel Framework Packages by all platforms so the entire packages can be removed from edk2/master.
This would be better than trying to remove a few items at a time. Thanks, Mike > -----Original Message----- > From: Zhang, Shenglei > Sent: Tuesday, November 13, 2018 7:32 PM > To: Ni, Ruiyu <ruiyu...@intel.com>; edk2- > de...@lists.01.org > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; > Gao, Liming <liming....@intel.com> > Subject: RE: [edk2] [PATCH 6/8] IntelFrameworkPkg: > Remove the redundant INFs > > Ray > Thanks for your constructive comments. I'll improve it > in next version. > > Thanks, > Shenglei > > -----Original Message----- > > From: Ni, Ruiyu > > Sent: Wednesday, November 14, 2018 11:12 AM > > To: Zhang, Shenglei <shenglei.zh...@intel.com>; edk2- > de...@lists.01.org > > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; > Gao, Liming > > <liming....@intel.com> > > Subject: Re: [edk2] [PATCH 6/8] IntelFrameworkPkg: > Remove the redundant > > INFs > > > > On 11/13/2018 4:35 PM, Shenglei Zhang wrote: > > > All INFs of unused Library instances in > IntelFrameworkPkg > > > are removed as they are not actually used. > > > https://bugzilla.tianocore.org/show_bug.cgi?id=1190 > > > > > > Cc: Liming Gao <liming....@intel.com> > > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > > Contributed-under: TianoCore Contribution Agreement > 1.1 > > > Signed-off-by: Shenglei Zhang > <shenglei.zh...@intel.com> > > > --- > > > IntelFrameworkPkg/IntelFrameworkPkg.dsc | 7 ----- > -- > > > 1 file changed, 7 deletions(-) > > > > > > diff --git > a/IntelFrameworkPkg/IntelFrameworkPkg.dsc > > b/IntelFrameworkPkg/IntelFrameworkPkg.dsc > > > index bd5df8c5d9..802a80e3eb 100644 > > > --- a/IntelFrameworkPkg/IntelFrameworkPkg.dsc > > > +++ b/IntelFrameworkPkg/IntelFrameworkPkg.dsc > > > @@ -63,13 +63,6 @@ > > > # generated for it, but the binary will not > be put into any firmware > > volume. > > > # > > > > > > ####################################################### > ### > > ######################################### > > > -[Components] > > > - > IntelFrameworkPkg/Library/DxeIoLibCpuIo/DxeIoLibCpuIo.i > nf > > > - > IntelFrameworkPkg/Library/FrameworkUefiLib/FrameworkUef > iLib.inf > > > - > > > IntelFrameworkPkg/Library/DxeSmmDriverEntryPoint/DxeSmm > DriverEntryP > > oint.inf > > > - > > > IntelFrameworkPkg/Library/PeiSmbusLibSmbusPpi/PeiSmbusL > ibSmbusPpi.in > > f > > > - > > > IntelFrameworkPkg/Library/PeiHobLibFramework/PeiHobLibF > ramework.inf > > > - > > > [BuildOptions] > > > *_*_*_CC_FLAGS = -D > DISABLE_NEW_DEPRECATED_INTERFACES > > > > > > > > Shenglei, > > You cannot remove the INF in the separate patch like > this. > > You should either put this patch in the first patch > in your serial, > > or combine the DSC change with your library removal. > > The goal is the the build won't fail when committing > the patches one by one. > > > > -- > > Thanks, > > Ray _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
