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

place errors on stderr, instead of stdout #638

Closed
AladW opened this issue Apr 26, 2016 · 5 comments
Closed

place errors on stderr, instead of stdout #638

AladW opened this issue Apr 26, 2016 · 5 comments
Milestone

Comments

@AladW
Copy link

AladW commented Apr 26, 2016

Right now, aria2c prints diagnostics on stdout, which makes it more difficult to use in scripts (in particular, shell pipes). curl and wget print to stderr instead.

@tatsuhiro-t
Copy link
Collaborator

I'd like to make this WONTFIX. This is because if we change the device, the existing script might not work anymore.

@AladW
Copy link
Author

AladW commented Apr 26, 2016

As with #639, you could keep the existing behaviour by making the new one optional, e.g via --stderr.

@tatsuhiro-t
Copy link
Collaborator

Yes, we can send all stuff currently written to stdout to stderr with new --stderr option.
But it only happens after --stderr is parsed correctly. Before options are parsed, still stdout is used for backward compatibility.

@tatsuhiro-t
Copy link
Collaborator

Fix committed via db239c2

@tatsuhiro-t tatsuhiro-t added this to the v1.23.0 milestone Apr 26, 2016
@AladW
Copy link
Author

AladW commented Apr 26, 2016

Works for me, thanks. Though it seems the option is not included in aria2c --help.

edit: works with aria2c '-h#advanced'

@AladW AladW closed this as completed Apr 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants