Skip to content
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

Added missing MCUBOOT_VERSION_TWEAK in Zephyr port #2171

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

m5k8
Copy link
Contributor

@m5k8 m5k8 commented Jan 11, 2025

There are several components in Zephyr app VERSION file and assignment of VERSION_TWEAK/APP_TWEAK to MCUBOOT_VERSION_TWEAK was missing, causing MCUBOOT_VERSION_TWEAK to be 0 all the time.

@m5k8
Copy link
Contributor Author

m5k8 commented Jan 13, 2025

It looks like I'm expected to add "Signed-off-by" to commit message, is that correct?

@nordicjm
Copy link
Collaborator

It looks like I'm expected to add "Signed-off-by" to commit message, is that correct?

Yes, i.e. git commit --amend -s

@m5k8 m5k8 force-pushed the add-zephyr-app-tweak branch from 66457e7 to e24d4a8 Compare January 13, 2025 15:52
@m5k8
Copy link
Contributor Author

m5k8 commented Jan 13, 2025

OK, corrected.

@m5k8
Copy link
Contributor Author

m5k8 commented Jan 14, 2025

Still failing? I can't see why, I added sign-off clause, and I can see it's there in the patch.
So, what's wrong with it? Please help.

@nordicjm
Copy link
Collaborator

You have: Signed-off-by: Michal Konieczny <[email protected]> and it is expecting: Signed-off-by: Michael Konieczny <[email protected]> so there is a mismatch

@m5k8 m5k8 force-pushed the add-zephyr-app-tweak branch from e24d4a8 to 9d522d5 Compare January 14, 2025 09:43
@m5k8
Copy link
Contributor Author

m5k8 commented Jan 14, 2025

Good eye! Thank you. Corrected, again.

@nordicjm nordicjm merged commit 256a02c into mcu-tools:main Jan 15, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants