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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]