RIOT icon indicating copy to clipboard operation
RIOT copied to clipboard

tools: Use RIOT maintained cosy

Open chrysn opened this issue 1 year ago • 1 comments

Contribution description

make cosy is 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

  • Places our fork as the new upstream,
  • drops all patches (because they were now merged), and
  • updates to the latest cosy.

Testing procedure

  • Pick any example (eg. hello-world)
  • make cosy BOARD=microbit-v2 (or any board you like)
  • Look at the web page indicated by the output.

Issues/PRs references

The patches dropped were merged in https://github.com/RIOT-OS/cosy/pull/4 and https://github.com/RIOT-OS/cosy/pull/7; other patches have been merged in since.

This PR is not ready for review yet because I'd like to also get https://github.com/RIOT-OS/cosy/pull/6 in, and maybe also https://github.com/RIOT-OS/cosy/pull/2; in fact, the commit hash currently in here points to a dangling commit pushed to test merging PR6 in it.

chrysn avatar Dec 10 '24 13:12 chrysn

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 avatar Dec 10 '24 13:12 chrysn

@chrysn with all PRs in at https://github.com/RIOT-OS/cosy, would you mind bumping the pkg commit to current master?

mguetschow avatar Jan 31 '25 12:01 mguetschow

Murdock results

:heavy_check_mark: PASSED

4201f4f09facae4c3b3f766c711cce75112a346e tools: Use RIOT maintained cosy

Success Failures Total Runtime
1 0 1 01m:11s

Artifacts

riot-ci avatar Jan 31 '25 13:01 riot-ci

Updated but can't test it right now (too deep in other parts); could you run a final test?

chrysn avatar Jan 31 '25 13:01 chrysn