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.

Sign up for GitHub

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

Jump to bottom

dist/tools/cosy: allow to specify port #18194

Merged
merged 1 commit into from
Jul 4, 2022
Merged

dist/tools/cosy: allow to specify port #18194

merged 1 commit into from
Jul 4, 2022

Conversation

benpicco
Copy link
Contributor

@benpicco benpicco commented Jun 11, 2022 •
edited
Loading

Contribution description

This allows to run multiple instances of cosy in parallel to compare binary sizes.

Testing procedure

make -C examples/hello-world cosy COSY_PORT=8888

Result validation: both size outputs below should match
Computed sums of parsed symbols:
   text    data     bss     dec     hex
  13384   11458  432958  457800   6fc48
Output of the 'size' command:
   text	   data	    bss	    dec	    hex	filename
  26878	    548	  47796	  75222	  125d6	/home/benpicco/dev/RIOT/examples/hello-world/bin/native/hello-world.elf

Started frontend server, connect you browser to http://localhost:8888

Issues/PRs references

upstream PR haukepetersen/cosy#14

@github-actions github-actions bot added Area: build system Area: Build system Area: tools Area: Supplementary tools labels Jun 11, 2022
@benpicco benpicco requested a review from haukepetersen June 11, 2022 12:49
@benpicco benpicco added CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jun 11, 2022
@maribu maribu enabled auto-merge July 4, 2022 12:44
@maribu maribu merged commit 04fc22e into RIOT-OS:master Jul 4, 2022
@benpicco benpicco deleted the dist/tools/cosy-port branch July 4, 2022 17:17
@chrysn chrysn added this to the Release 2022.07 milestone Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maribu maribu maribu approved these changes

@haukepetersen haukepetersen Awaiting requested review from haukepetersen

Assignees
No one assigned
Labels
Area: build system Area: Build system Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs
Projects
None yet
Milestone
Release 2022.07
Development

Successfully merging this pull request may close these issues.
3 participants
@benpicco @maribu @chrysn