After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 751694 - Version string should be printed to stdout
Version string should be printed to stdout
Status: RESOLVED FIXED
Product: gnome-taquin
Classification: Other
Component: general
unspecified
Other Linux
: Normal minor
: ---
Assigned To: GNOME Taquin maintainer(s)
GNOME Taquin maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-06-30 04:25 UTC by Mohammed Sadiq
Modified: 2015-08-20 04:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Print version string to stdout (800 bytes, patch)
2015-06-30 04:25 UTC, Mohammed Sadiq
none Details | Review

Description Mohammed Sadiq 2015-06-30 04:25:12 UTC
Created attachment 306364 [details] [review]
Print version string to stdout

While running 'gnome-taquin --version', the version string is printed to stderr.
It should be printed to stdout.

By the way, my first bug report, and my first patch. :-)
Comment 1 Arnaud B. 2015-06-30 05:54:08 UTC
I mostly began adding the --version option on many binaries. Welcome here!

Looks like most of the games (at least) are printing the version on stderr; but I agree it is weird, I do not see any reason for that. I’m going to ask for a confirmation on IRC to other devs, and make all the games in a row (no need to open more bugs).

Thanks for reporting, I’ll update the bug status when all that’s done.
Comment 2 Michael Catanzaro 2015-06-30 15:35:36 UTC
My guess is that about half of GNOME apps use stdout and half use stderr. This is one of those things the UNIX community has never been able to standardize.

I would favor stdout, since it's not an error.
Comment 3 Mohammed Sadiq 2015-06-30 17:09:18 UTC
I don't know about Unix standards. But GNU standards recommends (mandates in GNU packages) to print version string to stdout.

Also, its very common to check version of a software using shell scripts, which 
usually checks for stdout, and not stderr.

See a relevant python bug report: https://bugs.python.org/issue18338
Comment 4 Mohammed Sadiq 2015-06-30 17:10:16 UTC
Relevant link to GNU standards:
https://www.gnu.org/prep/standards/html_node/_002d_002dversion.html
Comment 5 Michael Catanzaro 2015-06-30 18:15:02 UTC
I will use stdout from now on myself, then :)
Comment 6 Arnaud B. 2015-08-18 01:49:50 UTC
I pushed a patch for the 3.17.90 unstable release of Taquin, thanks for the bug report. I hope I’ll find the time to do all the other games.
Comment 7 Arnaud B. 2015-08-19 17:13:26 UTC
Stupid me, I was so tired yesterday that I forgot to use your patch, sorry for that. (I’m thinking of that because I maintain the tab open, to remember doing so everywhere!) But you’ve of course been named in the NEWS file. I’ll do better next time.
Comment 8 Mohammed Sadiq 2015-08-20 04:50:36 UTC
(In reply to Arnaud Bonatti from comment #7)
> Stupid me, I was so tired yesterday that I forgot to use your patch, sorry
> for that. (I’m thinking of that because I maintain the tab open, to remember
> doing so everywhere!) But you’ve of course been named in the NEWS file. I’ll
> do better next time.

Its OK. Happy to contribute to GNOME codebase, the first time.