-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[darwin] Add back the colored output with additional informations for… #24094
[darwin] Add back the colored output with additional informations for… #24094
Conversation
0452faa
to
50ff98d
Compare
PR #24094: Size comparison from 79f8599 to 50ff98d Increases (6 builds for bl702, esp32, psoc6, telink)
Decreases (6 builds for bl602, nrfconnect, psoc6, telink)
Full report (53 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
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.
Thank you!
50ff98d
to
b3f0bf3
Compare
I was considering doing something similar just via an ifdef in the main stdio implementation so Linux also benefits from it. I’ve got a branch somewhere but didn’t put up a PR due to some build issues I was getting. This is fine anyway though, we can consider making it common with Linux later. |
PR #24094: Size comparison from b49c3ed to b3f0bf3 Increases (4 builds for cc13x2_26x2, esp32, psoc6, telink)
Decreases (4 builds for cyw30739, nrfconnect, psoc6, telink)
Full report (53 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Fast-tracking platform-specific change with platform owner review. |
… chip-tool
Problem
#23528 has enhance the
os_log
integration but it removes both the coloured output and the additional informations that are logged withchip-tool
. I have found both useful many times, whether it is to spot the error (in red) quickly, to look at the timing between 2 events, or to check on which thread something is runned.This PR adds it back (mostly copy paste the code that previously exists but moved it to a separate file)