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

[cpackget] Debug log level not working on some commands #131

Closed
lud0v1c opened this issue Nov 4, 2022 · 3 comments
Closed

[cpackget] Debug log level not working on some commands #131

lud0v1c opened this issue Nov 4, 2022 · 3 comments
Assignees
Labels
bug Something isn't working fixed

Comments

@lud0v1c
Copy link
Contributor

lud0v1c commented Nov 4, 2022

Describe the bug
The -v flag is not showing debug verbose messages in some commands, like the recently added checksum- and signature- commands.

To Reproduce
cpackget signature-create -v Pack.Vendor.1.2.3.pack -k key.pem -c cert.pem or any valid input for the command to run.

Expected behavior
Debug messages should be printed.

Environment (please complete the following information):

  • OS: Ubuntu 22.04.1 LTS
  • Version v0.8.3
@lud0v1c lud0v1c self-assigned this Nov 4, 2022
@lud0v1c lud0v1c added the bug Something isn't working label Nov 4, 2022
@jkrech jkrech changed the title Debug log level not working on some commands [cpackget] Debug log level not working on some commands Jan 24, 2023
@jkrech jkrech assigned thorstendb-ARM and unassigned lud0v1c Mar 7, 2023
@thorstendb-ARM
Copy link
Collaborator

thorstendb-ARM commented Jun 13, 2023

Init was missing in COBRA structs:
PersistentPreRunE: configureInstaller

thorstendb-ARM added a commit that referenced this issue Jun 13, 2023
fixed:
- Init was missing in COBRA structs: PersistentPreRunE: configureInstaller
- function crashes on empty string: sanitizeVersionForSignature()
thorstendb-ARM added a commit that referenced this issue Jun 14, 2023
fixed:
- initializing verbose option
- fixed crash in sanitizeVersionForSignature() accessing [0] of empty string
thorstendb-ARM added a commit that referenced this issue Jun 14, 2023
fixed:
- initializing verbose option
- fixed crash in sanitizeVersionForSignature() accessing [0] of empty string
@jkrech
Copy link
Member

jkrech commented Jun 15, 2023

I am not sure how to validate this:

cpackget checksum-create ARM.CMSIS.5.9.0.pack -v
does not print anything but maybe this is expected.

@thorstendb-ARM
Copy link
Collaborator

Please wait until release 0.9.5 or later.

@jkrech jkrech closed this as completed Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed
Projects
None yet
Development

No branches or pull requests

3 participants