Skip to content
New issue

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

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

Already on GitHub? # to your account

examples: ensure the proper build order #124

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

jbech-linaro
Copy link
Contributor

The Makefiles for the examples lacked dependencies between rules, resulting in some cases resulted in build error. To maintain a coherent build process, this commit introduces dependencies to ensure that binaries are built before being stripped and signed.

@b49020
Copy link
Contributor

b49020 commented Mar 19, 2024

Reviewed-by: Sumit Garg <sumit.garg@linaro.org>

@DemesneGH Can you trigger the CI here?

@jbech-linaro jbech-linaro force-pushed the fix-build-dependencies branch 2 times, most recently from 90f27b4 to 3b98cab Compare March 19, 2024 08:02
@jbech-linaro
Copy link
Contributor Author

R-B tag added and fixed a repetition in the commit message.

@DemesneGH
Copy link
Contributor

Acked-by: Yuan Zhuang <yuanz@apache.org>
thanks!

The Makefiles for the examples lacked dependencies between rules,
resulting in some cases a build error. To maintain a coherent build
process, this commit introduces dependencies to ensure that binaries are
built before being stripped and signed.

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Tested-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Sumit Garg <sumit.garg@linaro.org>
Acked-by: Yuan Zhuang <yuanz@apache.org>
@jbech-linaro jbech-linaro force-pushed the fix-build-dependencies branch from 3b98cab to 711e11e Compare March 19, 2024 11:53
@jbech-linaro
Copy link
Contributor Author

Thanks @DemesneGH ! Looked like the CI passed, I've added your Acked-by tag and force pushed.

Squashed, rebased and tag(s) applied (should be) ready for merge! Thanks!

@DemesneGH DemesneGH merged commit ec3eefd into apache:no-std Mar 19, 2024
5 checks passed
@jbech-linaro
Copy link
Contributor Author

Thanks @b49020 and @DemesneGH for the quick turn around time on this one! Much appreciated!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants