Re: [PATCH 6/8] IntelFrameworkPkg: Remove the redundant INFs

["Ni, Ruiyu" <ruiyu.ni@Intel.com>]

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.gao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Shenglei Zhang <shenglei.zhang@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.inf
> -  IntelFrameworkPkg/Library/FrameworkUefiLib/FrameworkUefiLib.inf
> -  IntelFrameworkPkg/Library/DxeSmmDriverEntryPoint/DxeSmmDriverEntryPoint.inf
> -  IntelFrameworkPkg/Library/PeiSmbusLibSmbusPpi/PeiSmbusLibSmbusPpi.inf
> -  IntelFrameworkPkg/Library/PeiHobLibFramework/PeiHobLibFramework.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