From 7dd42c510dc8ab10f52dcc2d20dc8af412be0744 Mon Sep 17 00:00:00 2001 From: David Oberhollenzer Date: Tue, 19 Jul 2016 15:06:58 +0200 Subject: Unify version string printing When a program does sophisticated enough command line processing (i.e. getopt), make sure it responds to -V and --version. When a program prints a version string, make sure it uses the common_print_version macro to print out its name, that it is part of mtd-utils and the mtd-utils version from the build system in a fashion similar to common program packages like the GNU coreutils. When a program responds to -V/--version or -h/--help, make sure it reports success exit status. Signed-off-by: David Oberhollenzer Signed-off-by: Richard Weinberger --- tests/fs-tests/integrity/integck.c | 5 ++--- tests/jittertest/JitterTest.c | 8 +++----- tests/jittertest/plotJittervsFill.c | 6 ++++-- 3 files changed, 9 insertions(+), 10 deletions(-) (limited to 'tests') diff --git a/tests/fs-tests/integrity/integck.c b/tests/fs-tests/integrity/integck.c index c6319fb..67384fa 100644 --- a/tests/fs-tests/integrity/integck.c +++ b/tests/fs-tests/integrity/integck.c @@ -37,7 +37,6 @@ #include #include -#define PROGRAM_VERSION "1.1" #define PROGRAM_NAME "integck" #include "common.h" #include "libubi.h" @@ -2992,7 +2991,7 @@ static void get_tested_fs_info(void) pid, fsinfo.fstype, fsinfo.mount_point); } -static const char doc[] = PROGRAM_NAME " version " PROGRAM_VERSION +static const char doc[] = PROGRAM_NAME " version " VERSION " - a stress test which checks the file-system integrity.\n" "\n" "The test creates a directory named \"integck_test_dir_\", where where\n" @@ -3077,7 +3076,7 @@ static int parse_opts(int argc, char * const argv[]) args.verbose = 1; break; case 'V': - fprintf(stderr, "%s\n", PROGRAM_VERSION); + common_print_version(); exit(EXIT_SUCCESS); case 'h': diff --git a/tests/jittertest/JitterTest.c b/tests/jittertest/JitterTest.c index 76371e8..1d74b8a 100644 --- a/tests/jittertest/JitterTest.c +++ b/tests/jittertest/JitterTest.c @@ -92,7 +92,8 @@ #include #include - +#define PROGRAM_NAME "JitterTest" +#include "common.h" /**************************** Enumerations ****************************/ enum timerActions @@ -165,9 +166,6 @@ enum timerActions /************************** Module Variables **************************/ -/* version identifier (value supplied by CVS)*/ -static const char Version[] = "$Id: JitterTest.c,v 1.13 2005/11/07 11:15:20 gleixner Exp $"; - static char OutFileName[MAX_FILE_NAME_LEN+1]; /* output file name */ static char LogFile[MAX_FILE_NAME_LEN+1] = "/dev/console"; /* default */ static char ReadFile[MAX_FILE_NAME_LEN+1]; /* This file is read. Should @@ -988,7 +986,7 @@ void SetSchedulerPriority( ***********************************************************************/ void PrintVersionInfo(void) { - printf("JitterTest version %s\n", Version); + common_print_version(); printf("Copyright (c) 2001, Daniel Industries, Inc.\n"); return; } diff --git a/tests/jittertest/plotJittervsFill.c b/tests/jittertest/plotJittervsFill.c index 99e627b..f9427f1 100644 --- a/tests/jittertest/plotJittervsFill.c +++ b/tests/jittertest/plotJittervsFill.c @@ -62,7 +62,9 @@ #include #include -static char Version_string[] = "$Id: plotJittervsFill.c,v 1.6 2005/11/07 11:15:21 gleixner Exp $"; +#define PROGRAM_NAME "plotJittervsFill" +#include "common.h" + static char LogFile[250] = "InputLogFile.log"; static int JitterThreshold_ms = 1000; @@ -103,7 +105,7 @@ void HandleCmdLineArgs( if ((strcmp(argv[argNum],"--version") == 0) || (strcmp(argv[argNum],"-v") == 0)) { /* Print version information and exit. */ - printf("%s\n", Version_string); + common_print_version(); exit(0); } -- cgit v1.2.3