Skip to content

Commit

Permalink
xtensa-build-zephyr.py: require zephyr.strip
Browse files Browse the repository at this point in the history
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
zephyrproject-rtos/zephyr#51954

Signed-off-by: Marc Herbert <[email protected]>
  • Loading branch information
marc-hb authored and kv2019i committed Jan 23, 2023
1 parent 35dda2a commit cc00c57
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scripts/xtensa-build-zephyr.py
Original file line number Diff line number Diff line change
Expand Up @@ -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),
Expand Down

0 comments on commit cc00c57

Please sign in to comment.