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.

Sign up for GitHub

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

Jump to bottom

Tarpaulin removed -v shorthand for --verbose in 0.27 #1388

Closed
msrd0 opened this issue Sep 19, 2023 · 3 comments · Fixed by #1389
Closed

Tarpaulin removed -v shorthand for --verbose in 0.27 #1388

msrd0 opened this issue Sep 19, 2023 · 3 comments · Fixed by #1389
Assignees
@xd009642

Comments

@msrd0
Copy link
Contributor

msrd0 commented Sep 19, 2023

Describe the bug
I used to use -v flag for tarpaulin. It stopped working on the latest version.

To Reproduce
Install version 0.27 of tarpaulin and try running tarpaulin with the -v flag:

  • sudo pacman -Sy cargo-tarpaulin=0.27.0-2
  • cargo tarpaulin -v

Expected behaviour
Tarpaulin runs with verbose logging.

Actual behaviour

error: unexpected argument '-v' found

  tip: to pass '-v' as a value, use '-- -v'

Usage: cargo tarpaulin [OPTIONS] [-- <ARGS>...]

For more information, try '--help'.
@xd009642
Copy link
Owner

@MiniaczQ do you mind having a look at this if you have time as it's related to the clap 2 -> 4 migration. If not I can try and find some time after work some day this week

@msrd0 msrd0 mentioned this issue Sep 19, 2023
Merged
@msrd0
Copy link
Contributor Author

msrd0 commented Sep 19, 2023

This turned out to be pretty easy to fix

@MiniaczQ
Copy link
Contributor

Sorry for that 😅

ansasaki added a commit to ansasaki/rust-keylime that referenced this issue Sep 27, 2023
@ansasaki
tests: Fix tarpaulin issues with dropped -v option
37b7af7 
This is a workaround for xd009642/tarpaulin#1388

The issue was fixed upstream but not released yet, which makes our
pipelines to fail.

This also adds the --engine llvm to try to solve the eventual timeout
that happens on our tests.

Signed-off-by: Anderson Toshiyuki Sasaki <[email protected]>
@ansasaki ansasaki mentioned this issue Sep 27, 2023
Merged
ansasaki added a commit to ansasaki/keylime-tests that referenced this issue Sep 27, 2023
@ansasaki
upstream: Fix tarpaulin issues with dropped -v option
0bd61bb 
This is a workaround for xd009642/tarpaulin#1388

The issue was fixed upstream but not released yet, which makes our
pipelines to fail.

This also adds the --engine llvm to try to solve the eventual timeout
that happens on our tests.

Signed-off-by: Anderson Toshiyuki Sasaki <[email protected]>
@ansasaki ansasaki mentioned this issue Sep 27, 2023
Merged
ansasaki added a commit to keylime/rust-keylime that referenced this issue Sep 28, 2023
@ansasaki
tests: Fix tarpaulin issues with dropped -v option
8da18de 
This is a workaround for xd009642/tarpaulin#1388

The issue was fixed upstream but not released yet, which makes our
pipelines to fail.

This also adds the --engine llvm to try to solve the eventual timeout
that happens on our tests.

Signed-off-by: Anderson Toshiyuki Sasaki <[email protected]>
kkaarreell pushed a commit to RedHat-SP-Security/keylime-tests that referenced this issue Oct 2, 2023
@ansasaki @kkaarreell
upstream: Fix tarpaulin issues with dropped -v option
3a6c9b2 
This is a workaround for xd009642/tarpaulin#1388

The issue was fixed upstream but not released yet, which makes our
pipelines to fail.

This also adds the --engine llvm to try to solve the eventual timeout
that happens on our tests.

Signed-off-by: Anderson Toshiyuki Sasaki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xd009642 xd009642

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add back the short -v flag msrd0/tarpaulin
3 participants
@xd009642 @msrd0 @MiniaczQ