Sorry about the failure and thank you all for the quick fix. Sent from a small-screen device
在 2017年9月22日,下午10:53,Laszlo Ersek <ler...@redhat.com<mailto:ler...@redhat.com>> 写道: On 09/22/17 16:00, Carsey, Jaben wrote: Reviewed-by: Jaben Carsey <jaben.car...@intel.com<mailto:jaben.car...@intel.com>> Thank you both; pushed as commit ba30d5f05121. Laszlo -----Original Message----- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Friday, September 22, 2017 6:44 AM To: Laszlo Ersek <ler...@redhat.com<mailto:ler...@redhat.com>> Cc: edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>; Carsey, Jaben <jaben.car...@intel.com<mailto:jaben.car...@intel.com>>; Ni, Ruiyu <ruiyu...@intel.com<mailto:ruiyu...@intel.com>>; Shi, Steven <steven....@intel.com<mailto:steven....@intel.com>> Subject: Re: [PATCH] ShellPkg/DmpStore: fix GCC/CLANG build failure Importance: High On 22 Sep 2017, at 06:34, Laszlo Ersek <ler...@redhat.com<mailto:ler...@redhat.com>> wrote: ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c:525:66: error: passing argument 2 of 'gEfiShellProtocol->GetGuidName' from incompatible pointer type [-Werror=incompatible-pointer-types] Status = gEfiShellProtocol->GetGuidName(&FoundVarGuid, &GuidName); ^ ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c:525:66: note: expected 'const CHAR16 ** {aka const short unsigned int **}' but argument is of type 'CHAR16 ** {aka short unsigned int **}' Pointer-to-(pointer-to-CHAR16) and pointer-to-(pointer-to-const-CHAR16) are incompatible types; GCC and CLANG are right to complain. Cc: Ard Biesheuvel <ard.biesheu...@linaro.org<mailto:ard.biesheu...@linaro.org>> Cc: Jaben Carsey <jaben.car...@intel.com<mailto:jaben.car...@intel.com>> Cc: Ruiyu Ni <ruiyu...@intel.com<mailto:ruiyu...@intel.com>> Cc: Steven Shi <steven....@intel.com<mailto:steven....@intel.com>> Reported-by: Jenkins Build Host <jenk...@kraxel.org<mailto:jenk...@kraxel.org>> Reported-by: Steven Shi <steven....@intel.com<mailto:steven....@intel.com>> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=716 Fixes: 09e8678380aaaf0a5ef59179ff59e0a045d1b0bf Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Laszlo Ersek <ler...@redhat.com<mailto:ler...@redhat.com>> --- Notes: I'm going to push this as soon as I get an R-b from anyone. Reviewed-by: Ard Biesheuvel <ard.biesheu...@linaro.org<mailto:ard.biesheu...@linaro.org>> Thanks Laszlo ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c index 062ab5dc3a9a..5791da9acc8c 100644 --- a/ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c +++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c @@ -424,7 +424,7 @@ CascadeProcessVariables ( CHAR16 *AttrString; CHAR16 *HexString; EFI_STATUS SetStatus; - CHAR16 *GuidName; + CONST CHAR16 *GuidName; if (ShellGetExecutionBreakFlag()) { return (SHELL_ABORTED); -- 2.14.1.3.gb7cf6e02401b _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org<mailto: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