From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::225]) by sourceware.org (Postfix) with ESMTPS id 8AB8E39730FB for ; Thu, 17 Nov 2022 10:39:33 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 8AB8E39730FB Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=seketeli.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=seketeli.org Received: (Authenticated sender: dodji@seketeli.org) by mail.gandi.net (Postfix) with ESMTPSA id 1272E1C000A; Thu, 17 Nov 2022 10:39:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=seketeli.org; s=gm1; t=1668681572; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Q9WQFz4fzuzlftlYLnj/MOCynhMvCNdTfib1bM38/tc=; b=C0saclYMlqGUHND3MZosonv6gTM/aJqCIMLIILgVEmtIBDbTHew2la+fAvXzrkwgkO+9v6 yXKBmMRuSAdj+JAi0rsoexCD/j8HzCX/thL2W2464jHwV89CWpqTypg/zj6lV4RUMiFuXt ZAd+2ZXspKwMhYxXAeZdPW59YOGD85ZtR2ZCBaRO1Lr5c3jjxeztveo5jtQN24gV/RB+KS 1sK3PhbYKTJoZPfsxKqTY40OcnVOu3cUwxRUzC3qHeJr4XzstF0lBd82UiamJw1/4PO1HQ iqxToF7zQP+CWUtT+TNq7uJn80U0MSs/zlVLBD6pp19Psdk8n3BZyP42hB7Sew== Received: by localhost (Postfix, from userid 1000) id 407FD581C59; Thu, 17 Nov 2022 11:39:31 +0100 (CET) From: Dodji Seketeli To: Sam James via Libabigail Cc: Sam James Subject: Re: [PATCH] configure.ac: add dist-xz Organization: Me, myself and I References: <20221108073503.2609200-1-sam@gentoo.org> <9B665B0E-2AE8-471F-9B35-D2809DDEA35E@gentoo.org> X-Operating-System: Fedora 38 X-URL: http://www.seketeli.net/~dodji Date: Thu, 17 Nov 2022 11:39:31 +0100 In-Reply-To: <9B665B0E-2AE8-471F-9B35-D2809DDEA35E@gentoo.org> (Sam James via Libabigail's message of "Tue, 8 Nov 2022 08:04:38 +0000") Message-ID: <874juxoof0.fsf@seketeli.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-9.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,JMQ_SPF_NEUTRAL,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hello Sam, Sam James via Libabigail a =C3=A9crit: >> On 8 Nov 2022, at 07:35, Sam James wrote: >>=20 >> On master at 9db716e3b15912b7162def1faa704eb7823bbf34: >> .rw-r--r-- 406M sam 8 Nov 07:27 =EF=90=90 libabigail-2.2.tar.gz >> .rw-r--r-- 348M sam 8 Nov 07:26 =EF=90=90 libabigail-2.2.tar.xz >>=20 >> xz is quite a common format for software to be distributed >> in nowadays because of its size reductions. This is a good idea! Thank you for suggesting this and coming up with this patch. I just have a few comments below. [...] >> diff --git a/ChangeLog b/ChangeLog >> index 33944561..f7f8d6f8 100644 >> --- a/ChangeLog >> +++ b/ChangeLog >> @@ -1,3 +1,11 @@ >> +2022-11-08 Sam James >> + >> + build: Create an xz distribution tarball in addition to gzip. >> + >> + * configure.ac: add dist-xz to AM_INIT_AUTOMAKE. >> + * Makefile.am: adjust $(TARBALL) to new extension. >> + * Makefile.am: pass XZ_OPT=3D"-0" for distcheck-fast. >> + >> 2022-09-24 Dodji Seketeli >>=20 >> ir: Avoid cancelling a "confirmed" propagated canonical type In the format of the patches sent to the list, what is going to be added to the ChangeLog is just put in the commit log. That part is later automatically extracted and added to the ChangeLog at release time. In other words, you don't have to add a ChangeLog entry in the patch. This is explained at https://sourceware.org/git/?p=3Dlibabigail.git;a=3Dblob_pla= in;f=3DCOMMIT-LOG-GUIDELINES;hb=3DHEAD. So, I have edited the patch accordingly to show you what I would apply instead. The amended patch can be found at the end of this message. >> diff --git a/Makefile.am b/Makefile.am >> index 65a1b121..a0310368 100644 >> --- a/Makefile.am >> +++ b/Makefile.am >> @@ -53,7 +53,7 @@ check-valgrind-recursive: >> update-changelog: >> python $(srcdir)/gen-changelog.py > $(srcdir)/ChangeLog >>=20 >> -TARBALL =3D $(PACKAGE_NAME)-$(VERSION).tar.gz >> +TARBALL =3D $(PACKAGE_NAME)-$(VERSION).tar.xz >> RELEASED_BRANCH =3D master > > I should've just changed this to take two so we keep gzip (as make > will produce one anyway), but I'll wait until feedback about if > we want to do this at all first. Right, I was wondering if there could be an option to chose between .tar and .xz for the compression format. We would then keep .xz as the default format. What do you think? In any case, thanks a lot for doing this. It is appreciated. Please find below the format of this patch that I would apply to the tree if it was to get in at this point. Cheers >From e1b055c43b07fa07e738dc39645b6825aa88834d Mon Sep 17 00:00:00 2001 From: Sam James Date: Tue, 8 Nov 2022 07:35:03 +0000 Subject: [PATCH] Use xz as the default tarball compression format MIME-Version: 1.0 Content-Type: text/plain; charset=3DUTF-8 Content-Transfer-Encoding: 8bit xz is quite a common format for software to be distributed in nowadays because of its size reductions. With this patch, here is the result on the master branch at db716e3b15912b7162def1faa704eb7823bbf34: .rw-r--r-- 406M sam 8 Nov 07:27 =EF=90=90 libabigail-2.2.tar.gz .rw-r--r-- 348M sam 8 Nov 07:26 =EF=90=90 libabigail-2.2.tar.xz * configure.ac: add dist-xz to AM_INIT_AUTOMAKE. * Makefile.am: adjust $(TARBALL) to new extension. * Makefile.am: pass XZ_OPT=3D"-0" for distcheck-fast. Signed-off-by: Sam James Signed-off-by: Dodji Seketeli --- Makefile.am | 4 ++-- configure.ac | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile.am b/Makefile.am index 65a1b121..a0310368 100644 --- a/Makefile.am +++ b/Makefile.am @@ -53,7 +53,7 @@ check-valgrind-recursive: update-changelog: python $(srcdir)/gen-changelog.py > $(srcdir)/ChangeLog =20 -TARBALL =3D $(PACKAGE_NAME)-$(VERSION).tar.gz +TARBALL =3D $(PACKAGE_NAME)-$(VERSION).tar.xz RELEASED_BRANCH =3D master =20 $(TARBALL): distcheck @@ -86,7 +86,7 @@ tag-and-all: distcheck check-self-compare # the default for GZIP_ENV is --best, which is pretty slow for check runs # distcheck-fast therefore compresses with --fast instead distcheck-fast: - $(MAKE) distcheck GZIP_ENV=3D"--fast" + $(MAKE) distcheck GZIP_ENV=3D"--fast" XZ_OPT=3D"-0" =20 # This makes us compare libabigail.so against its own ABIXML # representation. It's super slow (more than 5 minutes on one of my diff --git a/configure.ac b/configure.ac index 5aafed37..a2e5b33d 100644 --- a/configure.ac +++ b/configure.ac @@ -57,7 +57,7 @@ dnl This one is to be able to run "make check-valgrind" dnl and have unit tests run under der Valgrind. m4_include([autoconf-archive/ax_valgrind_check.m4]) =20 -AM_INIT_AUTOMAKE([1.11.1 foreign subdir-objects tar-ustar parallel-tests]) +AM_INIT_AUTOMAKE([1.11.1 foreign subdir-objects dist-xz tar-ustar parallel= -tests]) AM_MAINTAINER_MODE([enable]) =20 AM_SILENT_RULES([yes]) --=20 2.38.0 --=20 Dodji