On 07/08/15 10:16, Ard Biesheuvel wrote: > On 8 July 2015 at 08:24, jiaxinwu <jiaxin...@intel.com> wrote: >> Version2 update with a proper commit message. >> > > This commit message still does not explain the purpose of this change.
I agree absolutely. The commit message should be as follows, in *every single* patch that touches a package in this manner: <PackageName>: remove Ip4ConfigDxe module from the build Ip4ConfigDxe used to provide the Ip4Config protocol, on which many of the network drivers (used in this package) had traditionally depended. Since git commit 1f6729ff (SVN r17853), subject MdeModulePkg: Update Ip4Dxe driver to support Ip4Config2 protocol the Ip4Dxe driver itself provides a new version of the same protocol: Ip4Config2. Dependent network drivers, libraries and applications have been migrated to (or extended to) the new protocol version. For example: - git 7c25b7ea (SVN r17869): ping & ifconfig - git 00a6ad1b (SVN r17870): UefiHandleParsingLib - git 6c5c70d6 (SVN r17873): DxeNetLib - git 39561686 (SVN r17874): IpSecDxe - git c581e503 (SVN r17875): EfiSocketLib At this point, no driver, library or application in the edk2 tree remains with a dependency (or an exclusive dependency) on the old Ip4Config protocol. Therefore we can remove the providing driver, Ip4ConfigDxe, from this build. Jiaxin, please submit a v3 of the series, with this commit message in each relevant patch. (Please don't forget to customize <PackageName> in the subject line as appropriate.) Thanks Laszlo > >> Cc: Laszlo Ersek <ler...@redhat.com> >> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: jiaxinwu <jiaxin...@intel.com> >> --- >> ArmVirtPkg/ArmVirt.dsc.inc | 1 - >> ArmVirtPkg/ArmVirtQemu.fdf | 1 - >> 2 files changed, 2 deletions(-) >> >> diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc >> index 7ec0de4..6f47d8e 100644 >> --- a/ArmVirtPkg/ArmVirt.dsc.inc >> +++ b/ArmVirtPkg/ArmVirt.dsc.inc >> @@ -364,11 +364,10 @@ >> # Networking stack >> # >> MdeModulePkg/Universal/Network/DpcDxe/DpcDxe.inf >> MdeModulePkg/Universal/Network/ArpDxe/ArpDxe.inf >> MdeModulePkg/Universal/Network/Dhcp4Dxe/Dhcp4Dxe.inf >> - MdeModulePkg/Universal/Network/Ip4ConfigDxe/Ip4ConfigDxe.inf >> MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Dxe.inf >> MdeModulePkg/Universal/Network/MnpDxe/MnpDxe.inf >> MdeModulePkg/Universal/Network/VlanConfigDxe/VlanConfigDxe.inf >> MdeModulePkg/Universal/Network/Mtftp4Dxe/Mtftp4Dxe.inf >> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Dxe.inf >> diff --git a/ArmVirtPkg/ArmVirtQemu.fdf b/ArmVirtPkg/ArmVirtQemu.fdf >> index e822fdf..4448018 100644 >> --- a/ArmVirtPkg/ArmVirtQemu.fdf >> +++ b/ArmVirtPkg/ArmVirtQemu.fdf >> @@ -181,11 +181,10 @@ READ_LOCK_STATUS = TRUE >> # Networking stack >> # >> INF MdeModulePkg/Universal/Network/DpcDxe/DpcDxe.inf >> INF MdeModulePkg/Universal/Network/ArpDxe/ArpDxe.inf >> INF MdeModulePkg/Universal/Network/Dhcp4Dxe/Dhcp4Dxe.inf >> - INF MdeModulePkg/Universal/Network/Ip4ConfigDxe/Ip4ConfigDxe.inf >> INF MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Dxe.inf >> INF MdeModulePkg/Universal/Network/MnpDxe/MnpDxe.inf >> INF MdeModulePkg/Universal/Network/VlanConfigDxe/VlanConfigDxe.inf >> INF MdeModulePkg/Universal/Network/Mtftp4Dxe/Mtftp4Dxe.inf >> INF MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Dxe.inf >> -- >> 1.9.5.msysgit.1 >> ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel