Re: [edk2] [PATCH] Nt32Pkg: Remove PcdShellFile in DSC

2018-11-07 Thread Wu, Hao A 
> -Original Message-
> From: Zhang, Shenglei
> Sent: Thursday, November 08, 2018 10:28 AM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu; Wu, Hao A
> Subject: [PATCH] Nt32Pkg: Remove PcdShellFile in DSC
> 
> https://bugzilla.tianocore.org/show_bug.cgi?id=1298
> This change is based on the change below:
> https://lists.01.org/pipermail/edk2-devel/2018-November/031875.html

Hi,

The patch is good to me.
Reviewed-by: Hao Wu 

However, IMO it will be more clear to append this patch and the one for
EmulatorPkg after the patch series you mentioned above:

[PATCH v3 0/5] Remove EdkShellBinPkg in FDF, DSC and DEC

and make a v4 for the series.

Best Regards,
Hao Wu

> 
> Cc: Ruiyu Ni 
> Cc: Hao Wu 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Shenglei Zhang 
> ---
>  Nt32Pkg/Nt32Pkg.dsc | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc
> index bbaa415ee0..4dbde0cc45 100644
> --- a/Nt32Pkg/Nt32Pkg.dsc
> +++ b/Nt32Pkg/Nt32Pkg.dsc
> @@ -280,7 +280,6 @@
>gEfiNetworkPkgTokenSpaceGuid.PcdAllowHttpConnections|TRUE
>  !endif
> 
> -  gEfiIntelFrameworkModulePkgTokenSpaceGuid.PcdShellFile|{ 0x83, 0xA5,
> 0x04, 0x7C, 0x3E, 0x9E, 0x1C, 0x4F, 0xAD, 0x65, 0xE0, 0x52, 0x68, 0xD0, 0xB4,
> 0xD1 }
> 
>  !if $(SECURE_BOOT_ENABLE) == TRUE
># override the default values from SecurityPkg to ensure images from all
> sources are verified in secure boot
> --
> 2.18.0.windows.1

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

[edk2] [PATCH] Nt32Pkg: Remove PcdShellFile in DSC

2018-11-07 Thread Shenglei Zhang 
https://bugzilla.tianocore.org/show_bug.cgi?id=1298
This change is based on the change below:
https://lists.01.org/pipermail/edk2-devel/2018-November/031875.html

Cc: Ruiyu Ni 
Cc: Hao Wu 
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang 
---
 Nt32Pkg/Nt32Pkg.dsc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc
index bbaa415ee0..4dbde0cc45 100644
--- a/Nt32Pkg/Nt32Pkg.dsc
+++ b/Nt32Pkg/Nt32Pkg.dsc
@@ -280,7 +280,6 @@
   gEfiNetworkPkgTokenSpaceGuid.PcdAllowHttpConnections|TRUE
 !endif
 
-  gEfiIntelFrameworkModulePkgTokenSpaceGuid.PcdShellFile|{ 0x83, 0xA5, 0x04, 
0x7C, 0x3E, 0x9E, 0x1C, 0x4F, 0xAD, 0x65, 0xE0, 0x52, 0x68, 0xD0, 0xB4, 0xD1 }
 
 !if $(SECURE_BOOT_ENABLE) == TRUE
   # override the default values from SecurityPkg to ensure images from all 
sources are verified in secure boot
-- 
2.18.0.windows.1

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel