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

[cbuild2cmake] Trigger project executes after project build step #195

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

EaselinkBachmann
Copy link
Contributor

@EaselinkBachmann EaselinkBachmann commented Sep 26, 2024

This PR adds a new CONTEXT-executes target to CMake that is automatically run after build that triggers the executes statements that depend on the build outputs of that project.

Fixes #185.

This is the local fix that I described in cbuild#244. I'm not sure if this is exactly how you intend on resolving this, but I'm providing my patch in the hope that it will be useful.

@jkrech jkrech requested a review from brondani October 1, 2024 05:49
@EaselinkBachmann EaselinkBachmann force-pushed the feature/trigger-executes-on-build branch 2 times, most recently from 8424e92 to ded205c Compare October 8, 2024 08:14
This adds a new CONTEXT-executes target to CMake that is automatically
run after build that triggers the executes statements of that project.
Fixes Open-CMSIS-Pack#185.
@EaselinkBachmann EaselinkBachmann force-pushed the feature/trigger-executes-on-build branch from ded205c to e8eed57 Compare October 8, 2024 12:00
@EaselinkBachmann
Copy link
Contributor Author

EaselinkBachmann commented Oct 8, 2024

Fixed to also correctly work with solution level executes. Each execute statement is now added to the ${CONTEXT}-executes target of each context in the execute statement's depends-on: key.

Currently doesn't deal with multiple dependent executes statements though (i.e., an executes-A depends on executes-B depends on context-X currently leads to executes-A not being automatically run). Handling these kinds of transitive reverse-dependencies is difficult.

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.

[cbuild2cmake] Use ExternalProject_Add_Step for post-build executes
1 participant