I didn't see Laszlo provided any comments for this patch. Did you mean the below comments from him for OvmfPkg's change?
(1) Please add the following to the commit message: "The TFTP command was converted from a NULL class library instance to a dynamic shell command in commit 0961002352e9. This patch complements commit f9bc2f876326, which only removed the old library, but didn't add the new dynamic command." Thanks/Ray > -----Original Message----- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Wednesday, November 29, 2017 5:45 PM > To: Ni, Ruiyu <ruiyu...@intel.com> > Cc: edk2-devel@lists.01.org; Laszlo Ersek <ler...@redhat.com> > Subject: Re: [edk2] [PATCH v3 3/6] ArmVirtPkg: Fix build failure due to Tftp > library removal > > On 29 November 2017 at 09:36, Ni, Ruiyu <ruiyu...@intel.com> wrote: > > Ard, > > Would you mind to give a r-b for this patch? > > > > Hi Ray, > > Could you please incorporate the feedback you received from Laszlo first? > Thanks. > > > > > >> -----Original Message----- > >> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf > >> Of Ruiyu Ni > >> Sent: Wednesday, November 29, 2017 9:00 AM > >> To: edk2-devel@lists.01.org > >> Cc: Laszlo Ersek <ler...@redhat.com>; Ard Biesheuvel > >> <ard.biesheu...@linaro.org> > >> Subject: [edk2] [PATCH v3 3/6] ArmVirtPkg: Fix build failure due to > >> Tftp library removal > >> > >> Contributed-under: TianoCore Contribution Agreement 1.1 > >> Signed-off-by: Ruiyu Ni <ruiyu...@intel.com> > >> Cc: Laszlo Ersek <ler...@redhat.com> > >> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> > >> Cc: Julien Grall <julien.gr...@linaro.org> > >> --- > >> ArmVirtPkg/ArmVirt.dsc.inc | 11 +++++++---- > >> ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 3 ++- > >> ArmVirtPkg/ArmVirtXen.fdf | 3 ++- > >> 3 files changed, 11 insertions(+), 6 deletions(-) > >> > >> diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc > >> index 5d7edff104..9874637a3d 100644 > >> --- a/ArmVirtPkg/ArmVirt.dsc.inc > >> +++ b/ArmVirtPkg/ArmVirt.dsc.inc > >> @@ -1,7 +1,7 @@ > >> # > >> # Copyright (c) 2011-2015, ARM Limited. All rights reserved. > >> # Copyright (c) 2014, Linaro Limited. All rights reserved. > >> -# Copyright (c) 2015, Intel Corporation. All rights reserved. > >> +# Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved. > >> # > >> # This program and the accompanying materials # are licensed and > >> made available under the terms and conditions of the BSD License @@ > >> -55,6 +55,8 @@ [LibraryClasses.common] > >> HiiLib|MdeModulePkg/Library/UefiHiiLib/UefiHiiLib.inf > >> > >> UefiHiiServicesLib|MdeModulePkg/Library/UefiHiiServicesLib/UefiHiiSer > >> UefiHiiServicesLib|vice > >> sLib.inf > >> SortLib|MdeModulePkg/Library/UefiSortLib/UefiSortLib.inf > >> + ShellLib|ShellPkg/Library/UefiShellLib/UefiShellLib.inf > >> + > >> + FileHandleLib|MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.in > >> + f > >> > >> UefiRuntimeLib|MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf > >> > >> > OrderedCollectionLib|MdePkg/Library/BaseOrderedCollectionRedBlackTree > >> OrderedCollectionLib|L > >> ib/BaseOrderedCollectionRedBlackTreeLib.inf > >> @@ -217,8 +219,6 @@ [LibraryClasses.common.UEFI_APPLICATION] > >> > >> > PerformanceLib|MdeModulePkg/Library/DxePerformanceLib/DxePerforma > >> nceLib.inf > >> > >> > MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocationLib/UefiMemo > >> ryAllocationLib.inf > >> HiiLib|MdeModulePkg/Library/UefiHiiLib/UefiHiiLib.inf > >> - ShellLib|ShellPkg/Library/UefiShellLib/UefiShellLib.inf > >> - > >> FileHandleLib|MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.inf > >> > >> [LibraryClasses.common.UEFI_DRIVER] > >> > >> > UefiDecompressLib|IntelFrameworkModulePkg/Library/BaseUefiTianoCusto > >> mDecompressLib/BaseUefiTianoCustomDecompressLib.inf > >> @@ -383,6 +383,10 @@ [Components.common] > >> # > >> # UEFI application (Shell Embedded Boot Loader) > >> # > >> + > >> > ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand. > >> inf { > >> + <PcdsFixedAtBuild> > >> + gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE > >> + } > >> ShellPkg/Application/Shell/Shell.inf { > >> <LibraryClasses> > >> > >> > ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellComma > >> ndLib.inf > >> @@ -393,7 +397,6 @@ [Components.common] > >> > >> > NULL|ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Com > >> mandsLib.inf > >> > >> NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1C > >> NULL|om > >> mandsLib.inf > >> > >> > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1 > >> CommandsLib.inf > >> - > >> > NULL|ShellPkg/Library/UefiShellTftpCommandLib/UefiShellTftpCommandLib > >> .inf > >> > >> HandleParsingLib|ShellPkg/Library/UefiHandleParsingLib/UefiHandlePars > >> HandleParsingLib|ingL > >> ib.inf > >> PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf > >> > >> > BcfgCommandLib|ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfg > >> CommandLib.inf > >> diff --git a/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc > >> b/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc > >> index 744006d13c..89f95b2d99 100644 > >> --- a/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc > >> +++ b/ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc > >> @@ -1,7 +1,7 @@ > >> # > >> # Copyright (c) 2011-2015, ARM Limited. All rights reserved. > >> # Copyright (c) 2014-2016, Linaro Limited. All rights reserved. > >> -# Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved. > >> +# Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved. > >> # > >> # This program and the accompanying materials # are licensed and > >> made available under the terms and conditions of the BSD License @@ > >> -103,6 > >> +103,7 @@ [FV.FvMain] > >> # UEFI application (Shell Embedded Boot Loader) > >> # > >> INF ShellPkg/Application/Shell/Shell.inf > >> + INF > >> > ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand. > >> inf > >> > >> # > >> # Bds > >> diff --git a/ArmVirtPkg/ArmVirtXen.fdf b/ArmVirtPkg/ArmVirtXen.fdf > >> index > >> 70e76df228..6586ce6be5 100644 > >> --- a/ArmVirtPkg/ArmVirtXen.fdf > >> +++ b/ArmVirtPkg/ArmVirtXen.fdf > >> @@ -1,7 +1,7 @@ > >> # > >> # Copyright (c) 2011-2015, ARM Limited. All rights reserved. > >> # Copyright (c) 2014, Linaro Limited. All rights reserved. > >> -# Copyright (c) 2015 - 2016, Intel Corporation. All rights > >> reserved.<BR> > >> +# Copyright (c) 2015 - 2017, Intel Corporation. All rights > >> +reserved.<BR> > >> # > >> # This program and the accompanying materials # are licensed and > >> made available under the terms and conditions of the BSD License @@ > >> -169,6 > >> +169,7 @@ [FV.FvMain] > >> # UEFI application (Shell Embedded Boot Loader) > >> # > >> INF ShellPkg/Application/Shell/Shell.inf > >> + INF > >> > ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand. > >> inf > >> > >> # > >> # Bds > >> -- > >> 2.15.0.gvfs.1.preview.4 > >> > >> _______________________________________________ > >> edk2-devel mailing list > >> edk2-devel@lists.01.org > >> https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel