From cc00c57057b56840fa1c2ccdc5ca4a97bb5217e5 Mon Sep 17 00:00:00 2001 From: Marc Herbert Date: Thu, 19 Jan 2023 16:42:34 -0800 Subject: [PATCH] xtensa-build-zephyr.py: require zephyr.strip Make CONFIG_BUILD_OUTPUT_STRIPPED mandatory so we can always compare builds. It makes practically zero build space and time difference and has huge reproductibility value, see discussion in https://github.com/zephyrproject-rtos/zephyr/pull/51954 Signed-off-by: Marc Herbert --- scripts/xtensa-build-zephyr.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/xtensa-build-zephyr.py b/scripts/xtensa-build-zephyr.py index 529c1e85045e..b831d95b31ea 100755 --- a/scripts/xtensa-build-zephyr.py +++ b/scripts/xtensa-build-zephyr.py @@ -767,7 +767,7 @@ class InstFile: # CONFIG_BUILD_OUTPUT_STRIPPED # Renaming ELF files highlights the workaround below that strips the .comment section - InstFile(BIN_NAME + ".strip", renameTo=f"stripped-{BIN_NAME}.elf", optional=True), + InstFile(BIN_NAME + ".strip", renameTo=f"stripped-{BIN_NAME}.elf"), # Not every platform has intermediate rimage modules InstFile("main-stripped.mod", renameTo="stripped-main.elf", optional=True),