-
Notifications
You must be signed in to change notification settings - Fork 8.7k
scripts: Add HTML build report script #104431
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -33,6 +33,7 @@ message(" ram_plot - Build and create RAM usage in a sunburst chart as a vi | |
| message(" rom_plot - Build and create ROM usage in a sunburst chart as a visual representation") | ||
| message(" traceconfig - Generate a file in the build directory that details how each symbol got its final value") | ||
| message(" footprint - Create JSON RAM/ROM usage report in the build directory") | ||
| message(" buildreport - Generate an HTML report with footprint, traceconfig, initlevels, devicetree, and other details") | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Other commands, such as ram_report, rom_report, initlevels, etc produces textual output directly. Commands not doing so or relies on dedicated program indicates this is their target name, such as pahole and puncover. I would like For example
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Makes sense. There are number of locations whose name should be consistent. Currently the PR uses:
Based on suggestions so far I will update the west command to What are your thoughts on the script location and name? Would this be preferred:
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Another naming idea could be "dashboard":
|
||
| message(" initlevels - Display the initialization sequence") | ||
| message(" boards - Display supported boards") | ||
| message(" shields - Display supported shields") | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps move it a level down, such as: