Jordan: For now, we will keep the duplicated libraries in IntelFrameworkModulePkg.
Yes. I can submit TortoiseGit email issues. Thanks Liming -----Original Message----- From: Justen, Jordan L Sent: Wednesday, April 01, 2015 3:46 PM To: Gao, Liming; edk2-devel@lists.sourceforge.net Subject: RE: [edk2] [Patch 0/2] MdeModulePkg: Add two library instances On 2015-03-31 18:48:56, Gao, Liming wrote: > Jordan: > As I mention in this patch, the ones in IntelFrameworkModulePkg are > still kept for compatibility. So, I don't remove it now. Is this necessary if we change all EDK II platforms? Isn't that kind of compatibility more of an issue for UDK releases? Or, will we just leave the duplicated libraries indefinitely? > I attach the patch to keep them in the same mail thread. Seemly, > TortoiseGit doesn't support to it. Add it to the list of tortoisegit email issues? :) If you find the email Message-ID header then you can use --in-reply-to with git send-email. -Jordan > -----Original Message----- > From: Justen, Jordan L > Sent: Wednesday, April 01, 2015 2:51 AM > To: Gao, Liming; edk2-devel@lists.sourceforge.net > Subject: RE: [edk2] [Patch 0/2] MdeModulePkg: Add two library > instances > > On 2015-03-31 03:25:10, Gao, Liming wrote: > > Jordan: > > Good comments. Here, I attach the patches to clean up Duet, Nt32 > > and OvmfPkg. Please help review them. > > What about the part about removing the IntelFrameworkModulePkg versions? > > Why not send the other patches with git send-email? > > -Jordan > > > -----Original Message----- > > From: Justen, Jordan L > > Sent: Tuesday, March 31, 2015 12:08 AM > > To: edk2-devel@lists.sourceforge.net; Gao, Liming; > > edk2-devel@lists.sourceforge.net > > Subject: Re: [edk2] [Patch 0/2] MdeModulePkg: Add two library > > instances > > > > On 2015-03-30 00:04:57, Liming Gao wrote: > > > Copy below two library instances from IntelFrameworkModulePkg to > > > MdeModulePkg. Then, Platform dsc can refer to them from MdeModulePkg, and > > > remove the dependency of IntelFrameworkPkg and IntelFrameworkModulePkg. > > > The ones in IntelFrameworkModulePkg are still kept for compatibility. > > > > Since the UDK releases have separate branches, maybe we can clean up EDK II > > platforms and remove the IntelFrameworkModulePkg versions? > > > > Series Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com> > > > > > 1. PeiDxeDebugLibReportStatusCode > > > 2. LzmaCustomDecompressLib > > > > > > Liming Gao (2): > > > MdeModulePkg: Add LzmaDecompress Guid > > > MdeModulePkg: Add two library instances > > > > > > Include/Guid/LzmaDecompress.h | 35 + > > > .../F86GuidedSectionExtraction.c | 218 +++++ > > > .../GuidedSectionExtraction.c | 201 ++++ > > > .../LzmaCustomDecompressLib/LZMA-SDK-README.txt | 4 + > > > .../LzmaArchCustomDecompressLib.inf | 66 ++ > > > .../LzmaArchDecompressLib.uni | Bin 0 -> 2480 bytes > > > .../LzmaCustomDecompressLib.inf | 62 ++ > > > Library/LzmaCustomDecompressLib/LzmaDecompress.c | 220 +++++ > > > .../LzmaCustomDecompressLib/LzmaDecompressLib.uni | Bin 0 -> 2274 bytes > > > .../LzmaDecompressLibInternal.h | 96 ++ > > > Library/LzmaCustomDecompressLib/Sdk/C/7zVersion.h | 7 + > > > Library/LzmaCustomDecompressLib/Sdk/C/Bra.h | 60 ++ > > > Library/LzmaCustomDecompressLib/Sdk/C/Bra86.c | 85 ++ > > > Library/LzmaCustomDecompressLib/Sdk/C/CpuArch.h | 69 ++ > > > Library/LzmaCustomDecompressLib/Sdk/C/LzFind.c | 770 +++++++++++++++ > > > Library/LzmaCustomDecompressLib/Sdk/C/LzFind.h | 107 ++ > > > Library/LzmaCustomDecompressLib/Sdk/C/LzHash.h | 54 ++ > > > Library/LzmaCustomDecompressLib/Sdk/C/LzmaDec.c | 1026 > > > ++++++++++++++++++++ > > > Library/LzmaCustomDecompressLib/Sdk/C/LzmaDec.h | 223 +++++ > > > Library/LzmaCustomDecompressLib/Sdk/C/Types.h | 231 +++++ > > > Library/LzmaCustomDecompressLib/Sdk/history.txt | 236 +++++ > > > Library/LzmaCustomDecompressLib/Sdk/lzma.txt | 594 ++++++++++++ > > > Library/LzmaCustomDecompressLib/UefiLzma.h | 47 + > > > Library/PeiDxeDebugLibReportStatusCode/DebugLib.c | 461 +++++++++ > > > .../PeiDxeDebugLibReportStatusCode.inf | 54 ++ > > > .../PeiDxeDebugLibReportStatusCode.uni | Bin 0 -> 1896 bytes > > > MdeModulePkg.dec | 5 + > > > MdeModulePkg.dsc | 3 + > > > 28 files changed, 4934 insertions(+) create mode 100644 > > > Include/Guid/LzmaDecompress.h create mode 100644 > > > Library/LzmaCustomDecompressLib/F86GuidedSectionExtraction.c > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/GuidedSectionExtraction.c > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/LZMA-SDK-README.txt > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/LzmaArchCustomDecompressLib.inf > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/LzmaArchDecompressLib.uni > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/LzmaCustomDecompressLib.inf > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/LzmaDecompress.c > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/LzmaDecompressLib.uni > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/LzmaDecompressLibInternal.h > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/Sdk/C/7zVersion.h > > > create mode 100644 Library/LzmaCustomDecompressLib/Sdk/C/Bra.h > > > create mode 100644 Library/LzmaCustomDecompressLib/Sdk/C/Bra86.c > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/Sdk/C/CpuArch.h > > > create mode 100644 Library/LzmaCustomDecompressLib/Sdk/C/LzFind.c > > > create mode 100644 Library/LzmaCustomDecompressLib/Sdk/C/LzFind.h > > > create mode 100644 Library/LzmaCustomDecompressLib/Sdk/C/LzHash.h > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/Sdk/C/LzmaDec.c > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/Sdk/C/LzmaDec.h > > > create mode 100644 Library/LzmaCustomDecompressLib/Sdk/C/Types.h > > > create mode 100644 > > > Library/LzmaCustomDecompressLib/Sdk/history.txt > > > create mode 100644 Library/LzmaCustomDecompressLib/Sdk/lzma.txt > > > create mode 100644 Library/LzmaCustomDecompressLib/UefiLzma.h > > > create mode 100644 > > > Library/PeiDxeDebugLibReportStatusCode/DebugLib.c > > > create mode 100644 > > > Library/PeiDxeDebugLibReportStatusCode/PeiDxeDebugLibReportStatusCode. > > > inf create mode 100644 > > > Library/PeiDxeDebugLibReportStatusCode/PeiDxeDebugLibReportStatusCode. > > > uni > > > > > > -- > > > 1.9.5.msysgit.0 > > > > > > > > > > > > ------------------------------------------------------------------ > > > -- > > > -- > > > -------- Dive into the World of Parallel Programming The Go > > > Parallel Website, sponsored by Intel and developed in partnership > > > with Slashdot Media, is your hub for all things parallel software > > > development, from weekly thought leadership blogs to news, videos, > > > case studies, tutorials and more. Take a look and join the conversation > > > now. > > > http://goparallel.sourceforge.net/ > > > _______________________________________________ > > > edk2-devel mailing list > > > edk2-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
