build-system: fix RIOTBOOT_SKIP_COMPILE#21938
Merged
maribu merged 2 commits intoRIOT-OS:masterfrom Dec 8, 2025
Merged
Conversation
maribu
added
Type: bug
benpicco
approved these changes
Dec 8, 2025
The RIOTBOOT_SKIP_COMPILE flag is already present in our code base, but it did not really result in compilation of `riotboot` as a dependency of another app to be skipped. This fixes the issue and allows users to explicitly build riotboot for a given set of boards upfront, and then build any number of apps depending on `riotboot` to be build with `RIOTBOOT_SKIP_COMPILE=1` and cutting the compilation time in half. This is particularly valuable as various parameters such as `RIOT_CI_BULLD` or `CCACHE` do not get passed down to the implicit `riotboot` build. In a CI that makes use of `CCACHE`, this results in the CI time quickly being dominated by implicit builds of `riotboot` when many apps use SUIT.
This allows two apps using RIOTBOOT being build for the same board in parallel to not step on each others toes.
maribu
force-pushed
the
riotboot/allow-skip-implicit-build
branch
from
ab6c401 to
0bbd23b
Compare
Member
Author
|
Thx ❤️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contribution description
The RIOTBOOT_SKIP_COMPILE flag is already present in our code base, but it did not really result in compilation of
riotbootas a dependency of another app to be skipped.This fixes the issue and allows users to explicitly build riotboot for a given set of boards upfront, and then build any number of apps depending on
riotbootto be build withRIOTBOOT_SKIP_COMPILE=1and cutting the compilation time in half.This is particularly valuable as various parameters such as
RIOT_CI_BULLDorCCACHEdo not get passed down to the implicitriotbootbuild. In a CI that makes use ofCCACHE, this results in the CI time quickly being dominated by implicit builds ofriotbootwhen many apps use SUIT.Testing procedure
Running
in
masterresults inriotbootto be build (which can easily be noticed by that build ignoringRIOT_CI_BUILD=1and being more verbose). With this PR the implicit build ofriotbootindeed does not occur (which easily can be seen by the faster compilation time and the less verbose output).Issues/PRs references
None