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

Shell output going to stderr rather than stdout #845

Closed
oneonestar opened this issue Jun 16, 2023 · 1 comment · Fixed by #853 or #854
Closed

Shell output going to stderr rather than stdout #845

oneonestar opened this issue Jun 16, 2023 · 1 comment · Fixed by #853 or #854
Milestone

Comments

@oneonestar
Copy link

I'm trying to update from 3.21.0 to 3.23.0.
When both output and error are not tty (dumb terminal), console becomes null. OutputStream will point to FileDescriptor.err.
The default behavior in the previous version was FileDescriptor.out.
This causes the output go to stderr rather than stdout.
I think this is an unintended incompatible change.

image

console == TerminalProvider.Stream.Output ? FileDescriptor.out : FileDescriptor.err),

new FileOutputStream(FileDescriptor.out),

https://github.com/jline/jline3/pull/788/files#diff-8f75c2bcaf44443b94e6a2a293331a90047529aabbc2d0e2bae5811ce22ad692L448

Ref: apache/accumulo#3446
trinodb/trino#17916

@Colerar
Copy link

Colerar commented Jun 27, 2023

Yes, this is an annoying incompatible change after debugging for few hours...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants