ARM-software
Long path of ArmSystemReady build CI is causing log output of UEFI-SCT to be truncated
Long path in ArmSystemReady build CI is causing the truncation UEFI-SCT log prints.
For e.g, please see the logs below. The existing line is truncated and the bolded part is actually the next line.
/home/runner/work/arm-systemready/arm-systemready/SR/scripts/edk2-test/uefi-sct/SctPkg/TestCase/UEFI/EFI/Protocol/DevicePathToText/BlackBoxTest/DevicePathToTextBBTestCoverage.c:304: Convert result: MemoryMapped(0xE,0x123456789ABCDEF,0xFEDCBA9876543210) - Expected: MemoryMapped(14,0x123456789ABCDEF,0xFEDEFI_DEVICE_PATH_FROM_TEXT_PROTOCOL - ConvertDeviceNodeToText must correctly recover the converting ConvertTextToDeviceNode has acted on the device node string -- PASS
The same log, from a standalone UEFI-SCT build is as below: /hdd/sct_workspace/SctPkg/TestCase/UEFI/EFI/Protocol/DevicePathToText/BlackBoxTest/DevicePathToTextBBTestCoverage.c:306: Convert result: memorymapped(0XE,0X123456789ABCDEF,0XFEDCBA9876543210) - Expected: MemoryMapped(14,0x123456789ABCDEF,0xFEDCBA9876543210)
Suggestion: shorter path may be used in CI to build the ACS image: "/home/runner/work/arm-systemready/arm-systemready/SR" is long
Hi @edhay,
For the daily builds which is generated using Github CI, the default path /home/runner/work/arm-systemready cannot be changed. (https://github.com/actions/checkout) .
Any working path given will be relative to the default GITHUB_WORKSPACE path.
Rest of the path SR/scripts/edk2-test/uefi-sct/SctPkg/TestCase/UEFI/EFI/Protocol/DevicePathToText/BlackBoxTest/DevicePathToTextBBTestCoverage.c is path where SCT tests gets cloned.
For the pre-built images, we will use local build with shorter path of working directory and not github generated images.
Thanks, Chetan
Hi @edhay,
The prebuilt images are build with minimal build path /build and will not have long prefix of system build directory.
But as mentioned above, daily build taken from github will have the default build path of github CI runner and which cannot be removed.
Thanks, Chetan