Open-CMSIS-Pack / devtools

Open-CMSIS-Pack development tools - C++
Apache License 2.0
76 stars 59 forks source link

[csolution] convert with option --quiet suppresses warnings #1767

Open ReinhardKeil opened 2 months ago

ReinhardKeil commented 2 months ago

The Problem To Be Solved

Using csolution convert ... --quiet suppresses warnings. As consequence the cbuild setup command does also not show any warnings.

>csolution convert USB_Device.csolution.yml --context-set --quiet 
error csolution: file 'C:/Test/USB_D/HID/RTE/USB/USBD_Config_0.h' update required; merge content from update file, rename update file to base file and remove previous base file

>csolution convert USB_Device.csolution.yml --context-set
C:/Test/USB_D/USB_Device.cbuild-set.yml - info csolution: file is already up-to-date
C:/Test/USB_D/USB_Device.cbuild-pack.yml - info csolution: file is already up-to-date
error csolution: file 'C:/Test/USB_D/HID/RTE/USB/USBD_Config_0.h' update required; merge content from update file, rename update file to base file and remove previous base file
warning csolution: file 'C:/Test/USB_D/HID/RTE/USB/USBD_Config_HID_0.h' update recommended; merge content from update file, rename update file to base file and remove previous base file
C:/Test/USB_D/HID/HID.Debug+STM32U585AIIx.cbuild.yml - info csolution: file generated successfully
C:/Test/USB_D/USB_Device.cbuild-idx.yml - info csolution: file is already up-to-date

Suggest A Solution

I believe warnings should be also shown when using the option --quiet

jkrech commented 2 months ago

csolution has the option:

 -d, --debug                   Enable debug messages
 -q, --quiet                     Run silently, printing only error messages
 -v, --verbose                 Enable verbose messages

Why does cbuild setup invoke csolution convert --quiet ? What happens if neither -d nor -q nor -v used instead of changing the meaning of quiet.

ReinhardKeil commented 2 months ago

cbuild with default options will no longer call csolution with --quite. Review if csolution info message should be under --verbose.