]> granicus.if.org Git - sysstat/commit
Option -V from sysstat commands now displays the version number on stdout
authorSebastien Godard <sysstat@orange.fr>
Sun, 27 Feb 2011 14:45:23 +0000 (15:45 +0100)
committerSebastien Godard <sysstat@orange.fr>
Sun, 27 Feb 2011 14:45:23 +0000 (15:45 +0100)
commit835274145aecbb82e4e09209a47cf5849dbc50d2
treee0703a251ab4c948e2b9ef79cbda1dd9a5ca0823
parentca8aece839f673bd9c7e1f46c813a6a66b541d71
Option -V from sysstat commands now displays the version number on stdout
and returns 0 for the exit code.

Option -V used to display the version number on stderr and returns 1 for
the exit code.
This is not the expected behavior as it has done everything we asked properly.
So change this: Display on stdout and returns 0.
The same change has been applied to sar's option -h, which displays a
help message.

Mail from Lodewijk Bonebakker <jlbonebakker@gmail.com> (15/02/2011):
Subject: Systat version reporting

Dear Sebastian,

I have question related to the way you report the version number in sysstat. At the moment, it seems that you write:

sysstat version <versionno>
(C) Sebastien Godard (sysstat <at> orange.fr)

to stderr, and set the error-code to 1.

Given the significant changes between 7/8 and 9, we have a tremendous headache in automatically dealing with the different sar data files, collected during the day on different machines (some which we prefer not to upgrade). Currently in our environment we can work with this way of reporting your version number, but I would like to make a suggestion:

It would greatly help us if the version command returns only "sysstat version <versionno>" to stdout and sets the return code to 0. Our reasoning behind this is that 'sar -V' should print the version number and exit 0, since it has done everything we have asked it to do correctly. A non-zero exit code is then reserved for error-conditions. This way we can efficiently get the version-number, check for proper installation/kernel versioning etc.

Thank you for your time in considering this suggestion,

Kind regards,

Lodewijk Bonebakker
CHANGES
CREDITS
common.c
sar.c