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

Consider stdout as (default?) output option #68

Closed
willcohen opened this issue Feb 27, 2022 · 2 comments
Closed

Consider stdout as (default?) output option #68

willcohen opened this issue Feb 27, 2022 · 2 comments

Comments

@willcohen
Copy link
Contributor

Right now, while the choices are stderr and syslog, it might be worth having output default to stdout, or at least have it be an option. When packaging tuntox for nix, (NixOS/nixpkgs#160548), lack of stdout has made it more difficult to quickly check for a working build.

Even if it's decided to keep default behavior as is, having a version check, likely with --version or -v, should default to stdout.

@gjedeer
Copy link
Owner

gjedeer commented Feb 28, 2022

Thanks. The reasoning behind using stderr is mostly because -W option:

    -W <remotehostname>:<remoteport> - forward <remotehostname>:<remoteport> to
                                       stdin/stdout (SSH ProxyCommand mode)

But for --version, --help etc you're right, it should to stdout, I just checked how the standard GNU utilities do it and they print such output to stdout.

Also, there's no --version / --v, that needs to be implemented.

@gjedeer
Copy link
Owner

gjedeer commented Jun 12, 2022

Alright:

$ ./tuntox -V     
Tuntox built from git commit 0ba3dc62293ece11400673965b89a99d91d7e8ed, toxcore version 0.2.13
$ 

--help now prints to stdout, too. Thank you for letting me know about this problem and feel free to open issues/PRs on problems with packaging.

@gjedeer gjedeer closed this as completed Jun 12, 2022
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

No branches or pull requests

2 participants