build: Don't add version extra string to the version number.

If this string was non-empty, then it broke a lot of things because
autoconf and other parts of the build system were expecting this
string to contain a simple MAJOR.MINOR.RELEASE version number.

This requires changes to guestfish and guestmount so they use the
guestfs_version API to fetch the version from the library.  (The
Perl tools were already doing it this way).  In a way this is more
accurate, because it's no longer hard-coded in the binary, but
fetched from the dynamically linked libguestfs.so.
This commit is contained in:
Richard Jones
2010-08-27 13:38:49 +01:00
parent 1c523760d3
commit 4932fdca3c
3 changed files with 13 additions and 5 deletions

View File

@@ -22,7 +22,7 @@ m4_define([libguestfs_release], [5])
# extra can be any string
m4_define([libguestfs_extra], [])
AC_INIT([libguestfs],libguestfs_major.libguestfs_minor.libguestfs_release[]libguestfs_extra)
AC_INIT([libguestfs],libguestfs_major.libguestfs_minor.libguestfs_release)
AC_CONFIG_AUX_DIR([build-aux])
AM_INIT_AUTOMAKE([foreign])

View File

@@ -20,6 +20,7 @@
#include <stdio.h>
#include <stdlib.h>
#include <inttypes.h>
#include <string.h>
#include <unistd.h>
#include <fcntl.h>
@@ -381,9 +382,12 @@ main (int argc, char *argv[])
guestfs_set_verbose (g, verbose);
break;
case 'V':
printf ("%s %s\n", program_name, PACKAGE_VERSION);
case 'V': {
struct guestfs_version *v = guestfs_version (g);
printf ("%s %"PRIi64".%"PRIi64".%"PRIi64"%s\n", program_name,
v->major, v->minor, v->release, v->extra);
exit (EXIT_SUCCESS);
}
case 'x':
guestfs_set_trace (g, 1);

View File

@@ -29,6 +29,7 @@
#include <stdio.h>
#include <stdlib.h>
#include <stdint.h>
#include <inttypes.h>
#include <string.h>
#include <unistd.h>
#include <getopt.h>
@@ -1074,9 +1075,12 @@ main (int argc, char *argv[])
guestfs_set_verbose (g, verbose);
break;
case 'V':
printf ("%s %s\n", program_name, PACKAGE_VERSION);
case 'V': {
struct guestfs_version *v = guestfs_version (g);
printf ("%s %"PRIi64".%"PRIi64".%"PRIi64"%s\n", program_name,
v->major, v->minor, v->release, v->extra);
exit (EXIT_SUCCESS);
}
case HELP_OPTION:
usage (EXIT_SUCCESS);