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

[minor] Replace usage of which #108

Closed
cbiedl opened this issue Nov 30, 2021 · 0 comments · Fixed by #125
Closed

[minor] Replace usage of which #108

cbiedl opened this issue Nov 30, 2021 · 0 comments · Fixed by #125

Comments

@cbiedl
Copy link

cbiedl commented Nov 30, 2021

Hello,

tests/jose-jwe-enc uses the which program - that however is to be removed in Debian and currently triggers usage warnings. No harm is done (yet) for the testsuite, but please consider replacing it with command -v.
sarroutbi added a commit to sarroutbi/jose that referenced this issue Mar 2, 2023
@sarroutbi
Use "command -v" instead of "which" (deprecated)
19c8924 
Resolves: latchset#108

Signed-off-by: Sergio Arroutbi <sarroutb@redhat.com>
sarroutbi added a commit to sarroutbi/jose that referenced this issue Mar 2, 2023
@sarroutbi
Use "command -v" instead of "which" (deprecated)
3145d4c 
Resolves: latchset#108

Signed-off-by: Sergio Arroutbi <sarroutb@redhat.com>
@sarroutbi sarroutbi mentioned this issue Mar 2, 2023
Merged
sarroutbi added a commit that referenced this issue Mar 8, 2023
@sarroutbi
Use "command -v" instead of "which" (deprecated) (#125)
e1d66f1 
Resolves: #108

Signed-off-by: Sergio Arroutbi <sarroutb@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use "command -v" instead of "which" (deprecated) sarroutbi/jose
1 participant
@cbiedl