Re: [edk2-devel] [patch v2 0/3] Merge MdeModulePkg\BaseTianoCustomDecompressLib into MdePkg\BaseUefiDecompressLib

Re: [edk2-devel] [patch v2 0/3] Merge MdeModulePkg\BaseTianoCustomDecompressLib into MdePkg\BaseUefiDecompressLib

[Dandan Bi]

Correct the info for this patch series:
Patch 2  update consumer in edk2 repo to use this new one in MdePkg.
Patch 3 remove the one in MdeModulePkg finally.

Thanks,
Dandan
> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Dandan Bi
> Sent: Wednesday, May 15, 2019 10:20 AM
> To: devel@edk2.groups.io
> Cc: Gao, Liming <liming.gao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Ni, Ray
> <ray.ni@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: [edk2-devel] [patch v2 0/3] Merge
> MdeModulePkg\BaseTianoCustomDecompressLib into
> MdePkg\BaseUefiDecompressLib
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1722
> 
> V2:
> 1. Address the comments for patch 1 and take R-b tags from Liming.
> 2. Since QuarkPlatformPkg and Vlv2TbltDevicePkg has been moved into
> edk2-platforms repo, drop the update for these platforms in this v2 series,
> and will send seperate patches for them.
> 3. No change for other patches.
> 
> We plan to merge the BaseUefiTianoCustomDecompressLib in
> MdeModulePkg into the BaseUefDecompressLib in MdePkg.
> In order to reduce the duplicated codes and maintain easily.
> Patch 1 adds a new fdf file in BaseUefDecompressLib
> (BaseUefiTianoCustomDecompressLib.inf) to keep the same functionality
> and usage model with the one in MdeModulePkg.
> Patch 2-4  update consumer to use this new one in MdePkg.
> Patch 5 remove the one in MdeModulePkg finally.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com> Dandan Bi (3):
>   MdePkg: Merge TianoCustomDecompress algorithm into
>   Nt32Pkg: Update UefiDecompressLib instance path
>   MdeModulePkg/Library: Remove BaseUefiTianoCustomDecompressLib
> 
>  MdeModulePkg/Include/Guid/TianoDecompress.h   |   22 -
>  .../BaseUefiTianoCustomDecompressLib.c        | 1063 -----------------
>  ...aseUefiTianoCustomDecompressLibInternals.h |  214 ----
>  .../BaseUefiTianoDecompressLib.uni            |   16 -
>  MdeModulePkg/MdeModulePkg.dec                 |    4 -
>  MdeModulePkg/MdeModulePkg.dsc                 |    1 -
>  .../BaseUefiDecompressLib.c                   |   72 +-
>  .../BaseUefiDecompressLib.uni                 |    6 +-
>  .../BaseUefiDecompressLibInternals.h          |   44 +-
>  .../BaseUefiTianoCustomDecompressLib.c        |  213 ++++
>  .../BaseUefiTianoCustomDecompressLib.inf      |   11 +-
>  MdePkg/MdePkg.dec                             |    5 +
>  MdePkg/MdePkg.dsc                             |    1 +
>  Nt32Pkg/Nt32Pkg.dsc                           |    2 +-
>  14 files changed, 330 insertions(+), 1344 deletions(-)  delete mode 100644
> MdeModulePkg/Include/Guid/TianoDecompress.h
>  delete mode 100644
> MdeModulePkg/Library/BaseUefiTianoCustomDecompressLib/BaseUefiTian
> oCustomDecompressLib.c
>  delete mode 100644
> MdeModulePkg/Library/BaseUefiTianoCustomDecompressLib/BaseUefiTian
> oCustomDecompressLibInternals.h
>  delete mode 100644
> MdeModulePkg/Library/BaseUefiTianoCustomDecompressLib/BaseUefiTian
> oDecompressLib.uni
>  create mode 100644
> MdePkg/Library/BaseUefiDecompressLib/BaseUefiTianoCustomDecompress
> Lib.c
>  rename {MdeModulePkg/Library/BaseUefiTianoCustomDecompressLib =>
> MdePkg/Library/BaseUefiDecompressLib}/BaseUefiTianoCustomDecompres
> sLib.inf (76%)
> 
> --
> 2.18.0.windows.1
> 
> 
>