-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix imgtool import for CYTFM_064B0S2_4343W signing #14362
Conversation
@LDong-Arm, thank you for your changes. |
@LDong-Arm, thank you for your changes. |
The scripts have been frozen you may want to add @Patater and @rwalton-arm as reviewers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can make an exception for this change, as it won't affect the online IDE or compiler.
@hugueskamba @Patater Thanks for the review, we're waiting for #14354 to get in to rebase this PR |
Fixes: ARMmbed#14153 The target CYTFM_064B0S2_4343W is signed using `cysecuretools` which depends on `imgtool` managed by `pip`. However, Arm Musca targets requires a modified/wrapped version of the image tool copied from trusted-firmware-m + MCUboot. To avoid conflicts in the version of `imgtool` used, we should only add the copied version of `imgtool` to python's system path for Musca targets instead of doing it globally for all targets.
1e1c649
to
5ef82f5
Compare
Rebased and retargeted to the master branch. |
CI restarted |
Jenkins CI Test : ❌ FAILEDBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
@LDong-Arm looks like we may have some genuine failures |
Jenkins CI Test : ❌ FAILEDBuild Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
@adbridge It's still a CI issue. From the log:
This path to |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 3 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
This is ready for merge, as we are making an exception for the Travis frozen tools. |
Summary of changes
Fixes: #14153
The target CYTFM_064B0S2_4343W is signed using
cysecuretools
which depends onimgtool
managed bypip
. However, Arm Musca targets requires a modified/wrapped version of the image tool copied from trusted-firmware-m + MCUboot.To avoid conflicts in the version of
imgtool
used, we should only add the copied version ofimgtool
to python's system path for Musca targets instead of doing it globally for all targets.Impact of changes
Building and signing of CYTFM_064B0S2_4343W complete successfully.
Migration actions required
None.
Documentation
None.
Pull request type
Test results
Reviewers
@ARMmbed/team-cypress @romanjoe @hugueskamba @0xc0170