tools: Use RIOT maintained cosy #21070
Merged
Conversation
This file contains 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
chrysn
added
Type: enhancement
|
This PR may warrant a statement that "the RIOT project has taken in development of the cosy tool under its umbrella", but we have no flag "Releas notes: noteworthy". |
|
@chrysn with all PRs in at https://github.com/RIOT-OS/cosy, would you mind bumping the pkg commit to current master? |
chrysn
added
the
CI: ready for build
|
Updated but can't test it right now (too deep in other parts); could you run a final test? |
mguetschow
approved these changes
Jan 31, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
make cosyis an easy and well-established way to see how cosily the full built code fits in the microcontroller.Hauke has apparently ceased maintaining the upstream respository, and we have forked it into the RIOT-OS group on GitHub to keep maintaining it there. (It also fits: The tool may be adjusted for other systems, but its code has a few inherent assumptions about RIOT baked in that makes that nontrivial).
This PR
Testing procedure
make cosy BOARD=microbit-v2(or any board you like)Issues/PRs references
The patches dropped were merged in RIOT-OS/cosy#4 and RIOT-OS/cosy#7; other patches have been merged in since.
This PR is not ready for review yet because I'd like to also get RIOT-OS/cosy#6 in, and maybe also RIOT-OS/cosy#2; in fact, the commit hash currently in here points to a dangling commit pushed to test merging PR6 in it.