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

Error messages should be output to stderr #877

Closed
lewisdiamond opened this issue Jun 22, 2018 · 0 comments
Closed

Error messages should be output to stderr #877

lewisdiamond opened this issue Jun 22, 2018 · 0 comments
Assignees
@dominikschulz
Labels
feature Enhancements and new features
Milestone
1.8.2 Bugfix release

Comments

@lewisdiamond
Copy link

Summary

gopass list -f blah

(red)Entry 'blah' not found

gopass list blah 2>/dev/null

(red)Entry 'blah' not found

Expected behavior

gopass list blah 2>/dev/null

(EMPTY)

Environment

Linux

  • gopass Version: 1.8.0
@flokli flokli mentioned this issue Jun 22, 2018
Merged
9 tasks
flokli added a commit to flokli/nixpkgs that referenced this issue Jun 22, 2018
@flokli
gopass: override GOPASS_CONFIG to fix completion generation
404c3c1 
gopass tries to write a version number to it's configuaration, even when
just generating the shell completion scripts. This fails, as
/homeless-shelter is read-only inside the sandbox.
As error messages are printed to stdout instead of stderr
(see gopasspw/gopass#877), the error message
lands inside the completion script, thus breaking it.

Workaround that by setting GOPASS_CONFIG to `/dev/null`
xeji pushed a commit to NixOS/nixpkgs that referenced this issue Jun 24, 2018
@flokli @xeji
gopass: override GOPASS_CONFIG to fix completion generation (#42400)
1588d6a 
gopass tries to write a version number to it's configuaration, even when
just generating the shell completion scripts. This fails, as
/homeless-shelter is read-only inside the sandbox.
As error messages are printed to stdout instead of stderr
(see gopasspw/gopass#877), the error message
lands inside the completion script, thus breaking it.

Workaround that by setting GOPASS_CONFIG to `/dev/null`
@dominikschulz dominikschulz self-assigned this Jun 28, 2018
@dominikschulz dominikschulz added the feature Enhancements and new features label Jun 28, 2018
@dominikschulz dominikschulz added this to the 1.8.2 Bugfix release milestone Jun 28, 2018
dominikschulz added a commit to dominikschulz/gopass that referenced this issue Jun 28, 2018
@dominikschulz
Redirect errors to stderr
ed30d34 
Fixes gopasspw#877
@dominikschulz dominikschulz mentioned this issue Jun 28, 2018
Merged
dominikschulz added a commit to dominikschulz/gopass that referenced this issue Jun 28, 2018
@dominikschulz
Redirect errors to stderr
ccc2360 
Fixes gopasspw#877

Signed-off-by: Dominik Schulz <dominik.schulz@gauner.org>
dominikschulz added a commit to dominikschulz/gopass that referenced this issue Jun 28, 2018
@dominikschulz
Redirect errors to stderr
4c0ef0c 
Fixes gopasspw#877

Signed-off-by: Dominik Schulz <dominik.schulz@gauner.org>
dominikschulz added a commit that referenced this issue Jun 28, 2018
@dominikschulz
Redirect errors to stderr (#880)
330d23d 
Fixes #877

Signed-off-by: Dominik Schulz <dominik.schulz@gauner.org>
kpitt pushed a commit to kpitt/gopass that referenced this issue Jul 21, 2022
@dominikschulz
Redirect errors to stderr (gopasspw#880)
a0c7e6a 
Fixes gopasspw#877

Signed-off-by: Dominik Schulz <dominik.schulz@gauner.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dominikschulz dominikschulz

Labels
feature Enhancements and new features
Projects
None yet
Milestone
1.8.2 Bugfix release
Development

No branches or pull requests
2 participants
@lewisdiamond @dominikschulz