Reviewed-by: Jaben Carsey <jaben.car...@intel.com> >-----Original Message----- >From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] >Sent: Tuesday, August 25, 2015 6:04 AM >To: edk2-devel@lists.01.org; Carsey, Jaben; Qiu, Shumin >Cc: Ard Biesheuvel >Subject: [PATCH] ShellPkg: use UEFI_APPLICATION module type for Shell >components >Importance: High > >Some of the libraries under ShellPkg/Library/ are only intended to be >loaded into the Shell by means of a NULL LibraryClass resolution, and >serve no other purpose. Since the Shell itself is a UEFI_APPLICATION, it >makes sense to set the module type of those libraries to UEFI_APPLICATION >as well. > >This allows us to use different compiler flags for the Shell application >itself but also for the majority of its constituent parts that are built >separately via these libraries. > >Contributed-under: TianoCore Contribution Agreement 1.0 >Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> >--- > .../Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.inf | 2 +- > .../Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.inf | 2 +- > .../Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsLib.inf | 2 +- > .../Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib.inf | 2 +- > .../Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsLib.inf | 2 +- > .../UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf | 2 +- > ShellPkg/Library/UefiShellTftpCommandLib/UefiShellTftpCommandLib.inf | 2 +- > 7 files changed, 7 insertions(+), 7 deletions(-) > >diff --git >a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.inf >b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.inf >index d8451c397652..55371eb37194 100644 >--- >a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.inf >+++ >b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.inf >@@ -16,7 +16,7 @@ [Defines] > INF_VERSION = 0x00010006 > BASE_NAME = UefiShellDebug1CommandsLib > FILE_GUID = 90330D51-A99B-4cc8-A2EB-AE22542A3F45 >- MODULE_TYPE = UEFI_DRIVER >+ MODULE_TYPE = UEFI_APPLICATION > VERSION_STRING = 1.0 > LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER > CONSTRUCTOR = UefiShellDebug1CommandsLibConstructor >diff --git >a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.inf >b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.inf >index a813b3b614f9..f2a2e4c581b0 100644 >--- >a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.inf >+++ >b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.inf >@@ -16,7 +16,7 @@ [Defines] > INF_VERSION = 0x00010006 > BASE_NAME = UefiShellDriver1CommandsLib > FILE_GUID = 313D3674-3ED4-48fd-BF97-7DB35D4190D1 >- MODULE_TYPE = UEFI_DRIVER >+ MODULE_TYPE = UEFI_APPLICATION > VERSION_STRING = 1.0 > LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER > CONSTRUCTOR = UefiShellDriver1CommandsLibConstructor >diff --git >a/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsLib.inf >b/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsLib.inf >index d21ad41ca9f3..4d3713b46092 100644 >--- >a/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsLib.inf >+++ >b/ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsLib.inf >@@ -16,7 +16,7 @@ [Defines] > INF_VERSION = 0x00010006 > BASE_NAME = UefiShellLevel1CommandsLib > FILE_GUID = 50cb6037-1102-47af-b2dd-9944b6eb1abe >- MODULE_TYPE = UEFI_DRIVER >+ MODULE_TYPE = UEFI_APPLICATION > VERSION_STRING = 1.0 > LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER > CONSTRUCTOR = ShellLevel1CommandsLibConstructor >diff --git >a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib.inf >b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib.inf >index 324d210d31ca..76540fb3a364 100644 >--- >a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib.inf >+++ >b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib.inf >@@ -17,7 +17,7 @@ [Defines] > INF_VERSION = 0x00010006 > BASE_NAME = UefiShellLevel2CommandsLib > FILE_GUID = CBF3931C-A2DF-40e5-B77E-CCA9555E9755 >- MODULE_TYPE = UEFI_DRIVER >+ MODULE_TYPE = UEFI_APPLICATION > VERSION_STRING = 1.0 > LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER > CONSTRUCTOR = ShellLevel2CommandsLibConstructor >diff --git >a/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsLib.inf >b/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsLib.inf >index 7cc49605ee86..a13cd9babf55 100644 >--- >a/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsLib.inf >+++ >b/ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsLib.inf >@@ -19,7 +19,7 @@ [Defines] > INF_VERSION = 0x00010006 > BASE_NAME = UefiShellLevel3CommandsLib > FILE_GUID = 71374B42-85D7-4753-AD17-AA84C3A0EB93 >- MODULE_TYPE = UEFI_DRIVER >+ MODULE_TYPE = UEFI_APPLICATION > VERSION_STRING = 1.0 > LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER > CONSTRUCTOR = ShellLevel3CommandsLibConstructor >diff --git >a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf >b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf >index 6dfbbe52b279..92d47d114f6b 100644 >--- >a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf >+++ >b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf >@@ -17,7 +17,7 @@ [Defines] > INF_VERSION = 0x00010006 > BASE_NAME = UefiShellNetwork1CommandsLib > FILE_GUID = 9A929F7E-3861-45ce-87AB-7371219AE255 >- MODULE_TYPE = UEFI_DRIVER >+ MODULE_TYPE = UEFI_APPLICATION > VERSION_STRING = 1.0 > LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER > CONSTRUCTOR = ShellNetwork1CommandsLibConstructor >diff --git >a/ShellPkg/Library/UefiShellTftpCommandLib/UefiShellTftpCommandLib.inf >b/ShellPkg/Library/UefiShellTftpCommandLib/UefiShellTftpCommandLib.inf >index 43b367d388e9..96db25892078 100644 >--- a/ShellPkg/Library/UefiShellTftpCommandLib/UefiShellTftpCommandLib.inf >+++ b/ShellPkg/Library/UefiShellTftpCommandLib/UefiShellTftpCommandLib.inf >@@ -18,7 +18,7 @@ [Defines] > INF_VERSION = 0x00010006 > BASE_NAME = UefiShellTftpCommandLib > FILE_GUID = D2B61A25-9835-4E5D-906A-15615E1FF668 >- MODULE_TYPE = UEFI_DRIVER >+ MODULE_TYPE = UEFI_APPLICATION > VERSION_STRING = 1.0 > LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER > CONSTRUCTOR = ShellTftpCommandLibConstructor >-- >1.9.1
_______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel