Skip to content
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

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

Open
ReinhardKeil opened this issue Sep 13, 2024 · 2 comments
Open

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

ReinhardKeil opened this issue Sep 13, 2024 · 2 comments
Labels
discussion done indicates an issue's discussion is completed enhancement New feature or request

Comments

@ReinhardKeil
Copy link
Collaborator

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

@ReinhardKeil ReinhardKeil added enhancement New feature or request discussion Indicates an issue being in discussion labels Sep 13, 2024
@jkrech
Copy link
Member

jkrech commented Sep 17, 2024

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
Copy link
Collaborator Author

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

@ReinhardKeil ReinhardKeil added discussion done indicates an issue's discussion is completed and removed discussion Indicates an issue being in discussion labels Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion done indicates an issue's discussion is completed enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants