]> granicus.if.org Git - vnstat/commitdiff
use VERSION string from config.h instead of VNSTATVERSION from common.h
authorTeemu Toivola <git@humdi.net>
Wed, 21 Oct 2015 19:07:19 +0000 (22:07 +0300)
committerTeemu Toivola <git@humdi.net>
Wed, 21 Oct 2015 19:07:19 +0000 (22:07 +0300)
12 files changed:
Makefile.am
Makefile.in
src/cfg.c
src/common.c
src/common.h
src/daemon.c
src/dbjson.c
src/dbxml.c
src/vnstat.c
src/vnstatd.c
src/vnstati.c
tests/common_tests.c

index 962fd76952193f519d7b4c6cc712694ab79dcebd..bb11ab6b4bafdebce76e0a42710286dfaf435961 100644 (file)
@@ -88,4 +88,4 @@ profile:
 myclean: maintainer-clean
 
 clean-local:
-       -rm -fr coverage *.lcov *.gz test*.log test*.xml
+       -rm -fr coverage *.lcov *.gz test*.log test*.xml src/*.gc* tests/*.gc*
index 8c5eec819abf89e74ad4b72028384e28f1586b67..6523e18e7fd72050bb4f2100ce1c5a7ed994997c 100644 (file)
@@ -1625,7 +1625,7 @@ profile:
 myclean: maintainer-clean
 
 clean-local:
-       -rm -fr coverage *.lcov *.gz test*.log test*.xml
+       -rm -fr coverage *.lcov *.gz test*.log test*.xml src/*.gc* tests/*.gc*
 
 # Tell versions [3.59,3.63) of GNU make to not export all variables.
 # Otherwise a system limit (for SysV at least) may be exceeded.
index 1b4d1088eca9eea3231e45af3fd813728e693721..7bab5eb3cccab41019f547d8ed79b1f15cd6c8fc 100644 (file)
--- a/src/cfg.c
+++ b/src/cfg.c
@@ -5,7 +5,7 @@ void printcfgfile(void)
 {
        ibwnode *p = ifacebw;
 
-       printf("# vnStat %s config file\n", VNSTATVERSION);
+       printf("# vnStat %s config file\n", getversion());
        printf("##\n\n");
 
        printf("# default interface\n");
index 44b9cc1f78d25cf6462cedbb89bc550de2885415..7ff2862646aeb741609f7e67acad9e7f61868b43 100644 (file)
@@ -250,3 +250,16 @@ void panicexit(const char *sourcefile, const int sourceline)
        printe(PT_Error);
        exit(EXIT_FAILURE);
 }
+
+char *getversion(void)
+{
+       int i;
+       static char versionbuffer[16];
+       strncpy_nt(versionbuffer, VERSION, 16);
+       for (i=0; i<strlen(versionbuffer); i++) {
+               if (versionbuffer[i] == '_') {
+                       versionbuffer[i] = ' ';
+               }
+       }
+       return versionbuffer;
+}
index af4760149ef19aa18f3014e7b7005ec755ba5d7a..80a2056347475d52eb7d686530c6ac99c6108fde 100644 (file)
@@ -155,9 +155,6 @@ and most can be changed later from the config file.
 /* 1 = 1.0, 2 = 1.1-1.2, 3 = 1.3- */
 #define DBVERSION 3
 
-/* version string */
-#define VNSTATVERSION "1.15 beta"
-
 /* xml format version */
 /* 1 = 1.7- */
 #define XMLVERSION 1
@@ -306,6 +303,7 @@ uint64_t mbkbtokb(uint64_t mb, uint64_t kb);
 char *strncpy_nt(char *dest, const char *src, size_t n);
 int isnumeric(const char *s);
 void panicexit(const char *sourcefile, const int sourceline);
+char *getversion(void);
 
 /* global variables */
 DATA data;
index 406a80a978f003f37d8b6ced069dc0757d7667c7..35c0fb1bc66a51a7c8b2d0718ebface6aca7a63d 100644 (file)
@@ -30,7 +30,7 @@ void daemonize(void)
        setsid(); /* obtain a new process group */
 
        if (cfg.uselogging) {
-               snprintf(errorstring, 512, "vnStat daemon %s started. (uid:%d gid:%d)", VNSTATVERSION, (int)getuid(), (int)getgid());
+               snprintf(errorstring, 512, "vnStat daemon %s started. (uid:%d gid:%d)", VERSION, (int)getuid(), (int)getgid());
                if (!printe(PT_Info)) {
                        printf("Error: Unable to use logfile. Exiting.\n");
                        exit(EXIT_FAILURE);
index a4ae9345d3180b20ac024a243c966bb7216392c4..2d4a5c1a9650dd4f74d8544ff77ce2f575987107 100644 (file)
@@ -148,7 +148,7 @@ void jsondate(time_t *date, int type)
 
 void jsonheader(void)
 {
-       printf("{\"vnstatversion\":\"%s\",\"jsonversion\":\"%d\",\"interfaces\":[", VNSTATVERSION, JSONVERSION);
+       printf("{\"vnstatversion\":\"%s\",\"jsonversion\":\"%d\",\"interfaces\":[", getversion(), JSONVERSION);
 }
 
 void jsonfooter(void)
index f52c12b82e9b2216f6eb35a271b1939548a0afb2..b3eddf7b8e7ad86ca8c4b4847ecb93cf8f4d4947 100644 (file)
@@ -124,7 +124,7 @@ void xmldate(time_t *date, int type)
 
 void xmlheader(void)
 {
-       printf("<vnstat version=\"%s\" xmlversion=\"%d\">\n", VNSTATVERSION, XMLVERSION);
+       printf("<vnstat version=\"%s\" xmlversion=\"%d\">\n", getversion(), XMLVERSION);
 }
 
 void xmlfooter(void)
index da04e68403df6d336d35a3f1f5880bc124aef711..d92bb6cd177122c4d2c13dc17d871873b6ebe739 100644 (file)
@@ -305,7 +305,7 @@ int main(int argc, char *argv[]) {
                        free(p.ifacelist);
                        return 0;
                } else if ((strcmp(argv[currentarg],"-v")==0) || (strcmp(argv[currentarg],"--version")==0)) {
-                       printf("vnStat %s by Teemu Toivola <tst at iki dot fi>\n", VNSTATVERSION);
+                       printf("vnStat %s by Teemu Toivola <tst at iki dot fi>\n", getversion());
                        return 0;
                } else if ((strcmp(argv[currentarg],"-r")==0) || (strcmp(argv[currentarg],"--reset")==0)) {
                        p.reset=1;
@@ -442,7 +442,7 @@ int synccounters(const char *iface, const char *dirname)
 
 void showhelp(PARAMS *p)
 {
-                       printf(" vnStat %s by Teemu Toivola <tst at iki dot fi>\n\n", VNSTATVERSION);
+                       printf(" vnStat %s by Teemu Toivola <tst at iki dot fi>\n\n", getversion());
 
                        printf("         -q,  --query          query database\n");
                        printf("         -h,  --hours          show hours\n");
@@ -464,7 +464,7 @@ void showhelp(PARAMS *p)
 
 void showlonghelp(PARAMS *p)
 {
-                       printf(" vnStat %s by Teemu Toivola <tst at iki dot fi>\n\n", VNSTATVERSION);
+                       printf(" vnStat %s by Teemu Toivola <tst at iki dot fi>\n\n", getversion());
 
                        printf("   Query:\n");
                        printf("         -q, --query           query database\n");
index 679440470d9313517a1e0ba119f9b233c4a94e5c..7ef043b71b5347f69323ffa8dd1c631a3612b04b 100644 (file)
@@ -108,7 +108,7 @@ int main(int argc, char *argv[])
                } else if (strcmp(argv[currentarg],"--noadd")==0) {
                        s.noadd = 1;
                } else if ((strcmp(argv[currentarg],"-v")==0) || (strcmp(argv[currentarg],"--version")==0)) {
-                       printf("vnStat daemon %s by Teemu Toivola <tst at iki dot fi>\n", VNSTATVERSION);
+                       printf("vnStat daemon %s by Teemu Toivola <tst at iki dot fi>\n", getversion());
                        return 0;
                } else if ((strcmp(argv[currentarg],"-p")==0) || (strcmp(argv[currentarg],"--pidfile")==0)) {
                        if (currentarg+1<argc) {
@@ -212,7 +212,7 @@ int main(int argc, char *argv[])
 
 void showhelp(void)
 {
-       printf(" vnStat daemon %s by Teemu Toivola <tst at iki dot fi>\n\n", VNSTATVERSION);
+       printf(" vnStat daemon %s by Teemu Toivola <tst at iki dot fi>\n\n", getversion());
        printf("         -d, --daemon         fork process to background\n");
        printf("         -n, --nodaemon       stay in foreground attached to the terminal\n\n");
        printf("         -s, --sync           sync interface counters on first update\n");
index a8a777f0d72c41e13bdd6c892ec5b08a13df01e7..52f70101ba957f6261642bc9f0e758be26ac9611 100644 (file)
@@ -213,7 +213,7 @@ int main(int argc, char *argv[])
                                        printf("Rateunit changed: %d\n", cfg.rateunit);
                        }
                } else if ((strcmp(argv[currentarg],"-v")==0) || (strcmp(argv[currentarg],"--version"))==0) {
-                       printf("vnStat image output %s by Teemu Toivola <tst at iki dot fi>\n", VNSTATVERSION);
+                       printf("vnStat image output %s by Teemu Toivola <tst at iki dot fi>\n", getversion());
                        return 0;
                } else {
                        printf("Unknown arg \"%s\". Use --help for help.\n",argv[currentarg]);
@@ -258,7 +258,7 @@ void initiparams(IPARAMS *p)
 
 void showihelp(IPARAMS *p)
 {
-       printf(" vnStat image output %s by Teemu Toivola <tst at iki dot fi>\n\n", VNSTATVERSION);
+       printf(" vnStat image output %s by Teemu Toivola <tst at iki dot fi>\n\n", getversion());
        printf("         -h,  --hours          output hours\n");
        printf("         -d,  --days           output days\n");
        printf("         -m,  --months         output months\n");
index cf01a4a1af942125d825479684f947b51338aa81..04c50d574282c21669f751100aa3fd2459cbc8a4 100644 (file)
@@ -368,6 +368,14 @@ START_TEST(isnumeric_it_is_not)
 }
 END_TEST
 
+START_TEST(getversion_returns_a_version)
+{
+       ck_assert_int_gt((int)strlen(getversion()), 1);
+       ck_assert(strchr(getversion(), '_') == NULL);
+       ck_assert(strchr(getversion(), '.') != NULL);
+}
+END_TEST
+
 void add_common_tests(Suite *s)
 {
        TCase *tc_common = tcase_create("Common");
@@ -395,5 +403,6 @@ void add_common_tests(Suite *s)
        tcase_add_test(tc_common, isnumeric_empty);
        tcase_add_test(tc_common, isnumeric_it_is);
        tcase_add_test(tc_common, isnumeric_it_is_not);
+       tcase_add_test(tc_common, getversion_returns_a_version);
        suite_add_tcase(s, tc_common);
 }