Lokesh, I had applied the GenBin tool rebuild patch before this. I will check again on my side but please recheck on your side.
Thanks, Supreeth -----Original Message----- From: Lokesh Belathur Veerappa <[email protected]> Sent: Tuesday, November 27, 2018 2:04 AM To: Supreeth Venkatesh <[email protected]>; [email protected]; [email protected] Subject: RE: [edk2-test][PATCH] SctPkg/Tools: Fix incorrect line ending detection by GenBin tool Hi Supreeth, The 'GenBin' tool has to be rebuilt with this change. The GenBin tool build support is added in the patch "[edk2-test][PATCH] SctPkg/build: Add support for GenBin tool build". I will upload a single patch which includes both the changes. Please verify and merge it. Thanks, Lokesh -----Original Message----- From: Supreeth Venkatesh <[email protected]> Sent: Tuesday, November 27, 2018 1:56 AM To: Lokesh Belathur Veerappa <[email protected]>; [email protected]; [email protected] Cc: Lokesh Belathur Veerappa <[email protected]> Subject: RE: [edk2-test][PATCH] SctPkg/Tools: Fix incorrect line ending detection by GenBin tool With the latest edk2, I am getting this compilation error (even when this patch is applied). I will be looking at it tomorrow. Please check this out if you get a chance. GenBin /data/users/supreeth/supven01/work/sct_workspace/edk2/SctPkg/TestCase/UEFI/EFI/Protocol/Decompress/BlackBoxTest/Dependency/UncompressedFile2/UncompressedFile2.ini /data/users/supreeth/supven01/work/sct_workspace/edk2/Build/UefiSct/DEBUG_GCC49/AARCH64/Decompress_UncompressedFile2.ucmp Error: Invalid format (Line 30) Error: Cannot generate the binary file GNUmakefile:232: recipe for target '/data/users/supreeth/supven01/work/sct_workspace/edk2/Build/UefiSct/DEBUG_GCC49/AARCH64/Decompress_UncompressedFile2.ucmp' failed make: *** [/data/users/supreeth/supven01/work/sct_workspace/edk2/Build/UefiSct/DEBUG_GCC49/AARCH64/Decompress_UncompressedFile2.ucmp] Error 255 build.py... : error 7000: Failed to execute command make tbuild [/data/users/supreeth/supven01/work/sct_workspace/edk2/Build/UefiSct/DEBUG_GCC49/AARCH64/SctPkg/TestCase/UEFI/EFI/Protocol/Decompress/BlackBoxTest/Dependency/UncompressedFile2/UncompressedFile2] Thanks, Supreeth -----Original Message----- From: Lokesh B V <[email protected]> Sent: Tuesday, November 20, 2018 12:50 AM To: [email protected]; Supreeth Venkatesh <[email protected]>; [email protected] Cc: Lokesh Belathur Veerappa <[email protected]> Subject: [edk2-test][PATCH] SctPkg/Tools: Fix incorrect line ending detection by GenBin tool Some windows editors uses "\r\n" for line feed. While processing uefi testcase info file, the GenBin tool logic to skip line feed doesn't consider the presence of carraige return(\r) in line feed. So this results in incorrect format error. Signed-off-by: Lokesh B V <[email protected]> --- uefi-sct/SctPkg/Tools/Source/GenBin/GenBin.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/uefi-sct/SctPkg/Tools/Source/GenBin/GenBin.c b/uefi-sct/SctPkg/Tools/Source/GenBin/GenBin.c index 61bb35b..ce271a1 100644 --- a/uefi-sct/SctPkg/Tools/Source/GenBin/GenBin.c +++ b/uefi-sct/SctPkg/Tools/Source/GenBin/GenBin.c @@ -176,6 +176,7 @@ Trim ( for (Index1 = 0; Index1 < Length; Index1++) { if ((String[Index1] != ' ' ) && (String[Index1] != '\t') && + (String[Index1] != '\r') && (String[Index1] != '\n')) { break; } @@ -193,6 +194,7 @@ Trim ( for (Index1 = 0; Index1 < Length; Index1++) { if ((String[Length - 1 - Index1] != ' ' ) && (String[Length - 1 - Index1] != '\t') && + (String[Length - 1 - Index1] != '\r') && (String[Length - 1 - Index1] != '\n')) { break; } -- 2.7.4 IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
