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

Error messages should go to STDERR, not STDOUT #3

Closed
rpdelaney opened this issue Dec 13, 2013 · 4 comments
Closed

Error messages should go to STDERR, not STDOUT #3

rpdelaney opened this issue Dec 13, 2013 · 4 comments

Comments

@rpdelaney
Copy link

This would facilitate a wrapper script to pipe errors and so on. See http://google-styleguide.googlecode.com/svn/trunk/shell.xml#STDOUT_vs_STDERR

@Hasimir
Copy link

Hasimir commented Oct 10, 2014

I'd need to double-check the spec, but I'm pretty sure UCI only deals with STDIN and STDOUT, not STDERR (possibly short-sighted, but not likely to be changed). In which case it may be necessary for errors to go to stdout as a result.

@rpdelaney
Copy link
Author

That's a bummer. There are some possible hacks to work-around both standards, such as reading environment variables or parameters that instruct the script to implement non-standard behavior (in either direction). For example TB_USE_STDERR if non-zero would print errors to STDERR only, and TB_USE_ALL if non-zero would print errors to both STDOUT and STDERR. Default behaviors (if neither is set) would be to print all output to STDOUT.

But would it be necessary for the tb generation scripts to follow the UCI protocols in their output? I would think that would not be a concern in any way, unless there are GUI applications that invoke them to generate tablebases expecting a UCI interface...?

@Hasimir
Copy link

Hasimir commented Oct 19, 2014

One of the reasons UCI has gained popularity in recent years is that it is a very simple and easy to understand interface. It's just text input and output which is, more or less, in plain English. It's not a massive protocol either , you should be able to read through the whole thing in under an hour.

@syzygy1
Copy link
Owner

syzygy1 commented Mar 15, 2015

Fixed.

@syzygy1 syzygy1 closed this as completed Mar 15, 2015
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

3 participants