[libvirt] boostrap: gzip version check problem on FreeBSD
Eric Blake
eblake at redhat.com
Fri Nov 12 20:50:43 UTC 2010
[adding bug-gnulib]
On 11/12/2010 01:24 PM, Matthias Bolte wrote:
> boostrap.conf lists gzip as build dependency. bootstrap then tries to
> get it's version number using a get_version() function that executes
> 'gzio --version' and tries to parse the result.
>
> The sed expression expects the version number to contain a dot as gzip
> outputs it on Linux:
>
> $ gzip --version
> gzip 1.3.12
>
> But on FreeBSD it outputs this:
>
> # gzip --version
> FreeBSD gzip 20100407
>
> Therefore, the sed expression should be relaxed, or bootstrap should
> not try to parse the version number when it doesn't need it because
> gzip is listed with '-' as version requirement.
I'm pushing this to fix the gnulib side of things; a libvirt patch to
use the latest gnulib will follow shortly.
diff --git i/ChangeLog w/ChangeLog
index e348fe2..8a9d7b6 100644
--- i/ChangeLog
+++ w/ChangeLog
@@ -1,5 +1,11 @@
2010-11-12 Eric Blake <eblake at redhat.com>
+ bootstrap: allow FreeBSD gzip
+ * build-aux/bootstrap (get_version): Parse FreeBSD gzip version,
+ which has no '.' and goes to stderr.
+ * build-aux/bootstrap.conf (buildreq): Improve the sample file.
+ Reported by Matthias Bolte.
+
maintainer-makefile: check for i18n setup
* top/maint.mk (sc_bindtextdomain): Check for evidence that _()
will likely work.
diff --git i/build-aux/bootstrap w/build-aux/bootstrap
index 8c8d777..5278e8e 100755
--- i/build-aux/bootstrap
+++ w/build-aux/bootstrap
@@ -1,6 +1,6 @@
#! /bin/sh
# Print a version string.
-scriptversion=2010-10-08.16; # UTC
+scriptversion=2010-11-12.20; # UTC
# Bootstrap this package from checked-out sources.
@@ -345,11 +345,11 @@ get_version() {
$app --version 2>&1 |
sed -n '# extract version within line
- s/.*[v ]\{1,\}\([0-9]\{1,\}\.[.a-z0-9-]*\).*/\1/
+ s/.*[v ]\{1,\}\([0-9]\{1,\}\(\.[.a-z0-9-]*\)*\).*/\1/
t done
# extract version at start of line
- s/^\([0-9]\{1,\}\.[.a-z0-9-]*\).*/\1/
+ s/^\([0-9]\{1,\}\(\.[.a-z0-9-]*\)\)*.*/\1/
t done
d
diff --git i/build-aux/bootstrap.conf w/build-aux/bootstrap.conf
index 75cf6ff..2bb60e2 100644
--- i/build-aux/bootstrap.conf
+++ w/build-aux/bootstrap.conf
@@ -57,3 +57,11 @@ if test $gettext_external = 1; then
m4/xsize.m4
'
fi
+
+# Build prerequisites
+buildreq="\
+autoconf 2.59
+automake 1.9.6
+git 1.5.5
+tar -
+"
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20101112/6da12cce/attachment-0001.sig>
More information about the libvir-list
mailing list