Skip to content

tests: boot: Update imgtool path in west_sign_wrapper #90740

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 30, 2025

Conversation

gchwier
Copy link
Collaborator

@gchwier gchwier commented May 28, 2025

Fixes tests/boot/with_mcumgr in case, when imgtool is not installed. Added imgtool path to west sign wrapper.

Log from internal CI before fix:

INFO: CMD: west sign -t imgtool --no-hex --build-dir twister-out/nrf52840dk_nrf52840/zephyr/tests/boot/with_mcumgr/boot.with_mcumgr.test_upgrade/with_mcumgr --sbin twister-out/nrf52840dk_nrf52840/zephyr/tests/boot/with_mcumgr/boot.with_mcumgr.test_upgrade/test_0_0_2_0.signed.bin -- --key bootloader/mcuboot/root-rsa-2048.pem --version 0.0.2+0
WARNING: west sign using imgtool is deprecated and will be removed in a future release
FATAL ERROR: imgtool not found; either install it (e.g. "pip3 install imgtool") or provide --tool-path
FAILED

to test it one can run:
$ZEPHYR_BASE/scripts/twister -T tests/boot/with_mcumgr --device-testing --hardware-map hardware_map.yml --west-flash="--erase" --enable-slow -vv -c -ll debug

Fixes tests/boot/with_mcumgr in case, when imgtool is
not installed. Added imgtool path to west sign wrapper.

Signed-off-by: Grzegorz Chwierut <[email protected]>
Copy link

@de-nordic de-nordic requested a review from carlescufi May 29, 2025 08:30
@de-nordic de-nordic assigned de-nordic and unassigned d3zd3z May 29, 2025
@kartben kartben merged commit c6a41a4 into zephyrproject-rtos:main May 30, 2025
22 checks passed
@gchwier gchwier deleted the grch-imgtool-path branch May 31, 2025 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants