[libvirt] [PATCH v2] maint: Switch to xz compressed PAX release archives

Wed Jun 15 14:31:16 UTC 2016

On 06/15/2016 09:03 AM, Andrea Bolognani wrote:
> This allows us to produce releases that are roughly a third in
> size, have no limitation on path length, and are still readable
> by all supported platforms.
> ---

I just want to point out that the tarfile is built every time you run 
"make rpm" (which I do quite a lot - I prefer installing rpms to the 
carnage created by make install), and this increases the time for make 
rpm on my system by 1min38sec. (jtomko may have something to say about 
that, since he's been interested in shaving fractions of a second off 
the build time in the last few days :-O)

Am I going to need to carry a local patch to revert this so that I don't 
get *even more* bored waiting for builds to complete? Or is there a 
reasonable way to make it easily configurable with a switch to autogen? 
(even then I would still need a patch to the specfile, unless we could 
make it happen based on some environment setting).

(Aside from that, I applied the patch and it does work).

> Changes from [v1]:
>
>    * Update all references to .tar.gz, including spec files,
>      scripts and documentation (thanks Dan)
>    * Fix typo lenght -> length (thanks Ján)
>
> [v1] https://www.redhat.com/archives/libvir-list/2016-June/msg00974.html
>
>   .gitignore             | 2 +-
>   Makefile.am            | 2 +-
>   autobuild.sh           | 2 +-
>   configure.ac           | 2 +-
>   docs/compiling.html.in | 2 +-
>   docs/downloads.html.in | 4 ++--
>   libvirt.spec.in        | 2 +-
>   mingw-libvirt.spec.in  | 2 +-
>   8 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/.gitignore b/.gitignore
> index 7fd9963..39c0423 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -88,7 +88,7 @@
>   /gnulib/tests/*
>   /include/libvirt/libvirt-common.h
>   /libtool
> -/libvirt-*.tar.gz
> +/libvirt-*.tar.xz
>   /libvirt-[0-9]*
>   /libvirt*.pc
>   /libvirt.spec
> diff --git a/Makefile.am b/Makefile.am
> index da07e6c..4b4aad3 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -63,7 +63,7 @@ $(top_srcdir)/HACKING: $(top_srcdir)/docs/hacking1.xsl \
>   	   > $@-t && mv $@-t $@ ; fi;
>   
>   rpm: clean
> -	@(unset CDPATH ; $(MAKE) dist && rpmbuild -ta $(distdir).tar.gz)
> +	@(unset CDPATH ; $(MAKE) dist && rpmbuild -ta $(distdir).tar.xz)
>   
>   check-local: all tests
>   
> diff --git a/autobuild.sh b/autobuild.sh
> index 4d9f616..2143479 100755
> --- a/autobuild.sh
> +++ b/autobuild.sh
> @@ -56,7 +56,7 @@ exec 3>&-
>   test "$st" = 0
>   test -x /usr/bin/lcov && make cov
>   
> -rm -f *.tar.gz
> +rm -f *.tar.xz
>   make dist
>   
>   if test -n "$AUTOBUILD_COUNTER" ; then
> diff --git a/configure.ac b/configure.ac
> index 0114149..7d16e5a 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -24,7 +24,7 @@ AH_BOTTOM([#include <config-post.h>])
>   AC_CONFIG_MACRO_DIR([m4])
>   dnl Make automake keep quiet about wildcards & other GNUmake-isms; also keep
>   dnl quiet about the fact that we intentionally cater to automake 1.9
> -AM_INIT_AUTOMAKE([-Wno-portability -Wno-obsolete tar-ustar subdir-objects])
> +AM_INIT_AUTOMAKE([-Wno-portability -Wno-obsolete tar-pax no-dist-gzip dist-xz subdir-objects])
>   dnl older automake's default of ARFLAGS=cru is noisy on newer binutils;
>   dnl we don't really need the 'u' even in older toolchains.  Then there is
>   dnl older libtool, which spelled it AR_FLAGS
> diff --git a/docs/compiling.html.in b/docs/compiling.html.in
> index aff5487..d2a3f15 100644
> --- a/docs/compiling.html.in
> +++ b/docs/compiling.html.in
> @@ -13,7 +13,7 @@
>       </p>
>   
>       <pre>
> -      $ gunzip -c libvirt-x.x.x.tar.gz | tar xvf -
> +      $ xz -c libvirt-x.x.x.tar.xz | tar xvf -
>         $ cd libvirt-x.x.x
>         $ ./configure</pre>
>   
> diff --git a/docs/downloads.html.in b/docs/downloads.html.in
> index f213b29..1a3400a 100644
> --- a/docs/downloads.html.in
> +++ b/docs/downloads.html.in
> @@ -28,8 +28,8 @@
>       </p>
>   
>       <ul>
> -      <li><a href="ftp://libvirt.org/libvirt/libvirt-git-snapshot.tar.gz">libvirt.org FTP server</a></li>
> -      <li><a href="http://libvirt.org/sources/libvirt-git-snapshot.tar.gz">libvirt.org HTTP server</a></li>
> +      <li><a href="ftp://libvirt.org/libvirt/libvirt-git-snapshot.tar.xz">libvirt.org FTP server</a></li>
> +      <li><a href="http://libvirt.org/sources/libvirt-git-snapshot.tar.xz">libvirt.org HTTP server</a></li>
>       </ul>
>   
>       <h2><a name="schedule">Primary release schedule</a></h2>
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index ee6162e..6b3e888 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -225,7 +225,7 @@ URL: http://libvirt.org/
>   %if %(echo %{version} | grep -o \\. | wc -l) == 3
>       %define mainturl stable_updates/
>   %endif
> -Source: http://libvirt.org/sources/%{?mainturl}libvirt-%{version}.tar.gz
> +Source: http://libvirt.org/sources/%{?mainturl}libvirt-%{version}.tar.xz
>   
>   Requires: libvirt-daemon = %{version}-%{release}
>   Requires: libvirt-daemon-config-network = %{version}-%{release}
> diff --git a/mingw-libvirt.spec.in b/mingw-libvirt.spec.in
> index 6f95832..e808839 100644
> --- a/mingw-libvirt.spec.in
> +++ b/mingw-libvirt.spec.in
> @@ -30,7 +30,7 @@ Summary:        MinGW Windows libvirt virtualization library
>   License:        LGPLv2+
>   Group:          Development/Libraries
>   URL:            http://libvirt.org/
> -Source0:        ftp://libvirt.org/libvirt/libvirt-%{version}.tar.gz
> +Source0:        ftp://libvirt.org/libvirt/libvirt-%{version}.tar.xz
>   
>   BuildRequires:  mingw32-filesystem >= 95
>   BuildRequires:  mingw64-filesystem >= 95