-
Notifications
You must be signed in to change notification settings - Fork 2.1k
sys/result_output: Initial result output abstraction #15039
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
Conversation
Initial implementation uses the fmt library to print data and results.
9359b97
to
c7e3aa7
Compare
Updated to full json output as per conversation with @kaspar030 since it will work better with murdock metrics |
Updated
|
Updated
|
I will run murdock to test metrics |
This does seem to increase build size... I thought I could make some gains by using the |
why not just |
I wanted to somehow namespace data. |
yeah, all stdio code is still used (just by "Hello, this is RIOT!")... |
gross, oh well, that is optimization (probably also used in the interactive sync). |
I guess #13710 will help with that! |
@fjmolinas & @kaspar030 |
@kaspar030 ping, There is an issues with this: If we say the convenience of this is work that (and you should be able to replace these with just a printf) then we can move forward and even make a |
After talking with @leandrolanzieri and @jia200x it seems that we think the best way is to have a way to open a result container chronologically print the data from the results and close it (ideally with an exit status similar to linux). Structure can be enforced with the CI for example, every command would have an exit status. Adding a generic parser to The one thing that wasn't agreed on is to allow unformatted data from I will survey some serializer APIs to see what would be needed and provide something a bit more complete, and maybe a test that can show the capabilities a bit better. |
closed with #15950 |
Contribution description
This provides an abstraction and implementation of simple data print and result functions to be used to standardize RIOT test firmware.
Testing procedure
run
make flash term -C tests/periph_cpuid
and check the output.Issues/PRs references
Fixes #15037