nasa / cFE

The Core Flight System (cFS) Core Flight Executive (cFE)
Apache License 2.0
409 stars 202 forks source link

Fix #2561, send CMake message to stdout instead of stderr #2525

Closed Codym48 closed 4 months ago

Codym48 commented 7 months ago

Checklist (Please check before submitting)

Describe the contribution Fix #2561.

While executing the drac0-rc5 training exercises, I noticed on the first make prep call that the following message is printed to stderr

NOTE: Selected /home/cmartin/code/cFS/sample_defs/cpu1_cfe_es_startup.scr as source for cfe_es_startup.scr on cpu1

This seems like an informational message that would be more appropriate for stdout than stderr, so I added the STATUS keyword to the cmake message call.

Testing performed Before change

make prep > stdout.log # observe NOTE printed to screen despite passing stdout to file

After change

rm -rf build
make prep > stdout.log # observe nothing printed to screen 🎉

Expected behavior changes

System(s) tested on

Additional context This is a bit of a trial balloon to feel out the cFS contribution process. I won't be offended if you close this Pull Request with no action. Thank you for your hard work on this product and its outstanding documentation!

Contributor Info - All information REQUIRED for consideration of pull request Full name and company/organization/center of all contributors ("Personal" if individual work)

Codym48 commented 4 months ago

FYI @jphickey as the last developer that touched this line