From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:45178)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1cuCuI-0000GO-3X
	for guix-patches@gnu.org; Sat, 01 Apr 2017 02:53:07 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1cuCuE-0007va-Qs
	for guix-patches@gnu.org; Sat, 01 Apr 2017 02:53:06 -0400
Received: from debbugs.gnu.org ([208.118.235.43]:56781)
	by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)
	(Exim 4.71) (envelope-from <Debian-debbugs@debbugs.gnu.org>)
	id 1cuCuE-0007vJ-BK
	for guix-patches@gnu.org; Sat, 01 Apr 2017 02:53:02 -0400
Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1cuCuE-00031Y-4Y
	for guix-patches@gnu.org; Sat, 01 Apr 2017 02:53:02 -0400
Subject: bug#26225: [PATCH] guix: Compress and decompress xz archives
	in	parallel.
Resent-Message-ID: <handler.26225.B26225.149102955311591@debbugs.gnu.org>
From: Marius Bakke <mbakke@fastmail.com>
In-Reply-To: <20170330222235.GB10357@jasmine>
References: <E1cr0d6-0004an-8g@eggs.gnu.org>
	<5vvaqyii1c.fsf@fencepost.gnu.org> <20170330222235.GB10357@jasmine>
Date: Sat, 01 Apr 2017 08:52:28 +0200
Message-ID: <878tnkfwlv.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha512; protocol="application/pgp-signature"
List-Id: <guix-patches.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-patches/>
List-Post: <mailto:guix-patches@gnu.org>
List-Help: <mailto:guix-patches-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=subscribe>
Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org
Sender: "Guix-patches" <guix-patches-bounces+kyle=kyleam.com@gnu.org>
To: Leo Famulari <leo@famulari.name>, 26225@debbugs.gnu.org, efraim@flashner.co.il

--=-=-=
Content-Type: text/plain

Leo Famulari <leo@famulari.name> writes:

> Efraim Flashner wrote:
>> * guix/scripts/pack.scm (%compressors): Add flag '-T0' when calling "xz".
>> * guix/utils.scm (decompressed-port, compressed-port,
>> compressed-output-port): Same.
>
>> --- a/guix/utils.scm
>> +++ b/guix/utils.scm
>> @@ -154,7 +155,7 @@ a symbol such as 'xz."
>>    (match compression
>>      ((or #f 'none) (values input '()))
>>      ('bzip2        (filtered-port `(,%bzip2 "-dc") input))
>> -    ('xz           (filtered-port `(,%xz "-dc") input))
>> +    ('xz           (filtered-port `(,%xz "-dc -T0") input))
>>      ('gzip         (filtered-port `(,%gzip "-dc") input))
>>      (else          (error "unsupported compression scheme" compression))))
>
> Parallel decompression isn't implemented yet, but the extra '-T0' option
> doesn't seem to break anything.
>
> The changes LGTM.
>
> I sent a companion patch to <https://bugs.gnu.org/26316> that enables
> threaded compression when building source tarballs.

Is (parallel-job-count) accessible here? It would be nice to respect it.

--=-=-=
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAEBCgAdFiEEu7At3yzq9qgNHeZDoqBt8qM6VPoFAljfTiwACgkQoqBt8qM6
VPrBWQgApc4bdfvSCVtABSMaB+kRw9WSK5S/jcuYEIocUNxnqJJeQo+MF8nYNBPi
i0g1NlWrXr8nonRO0iuMIZLbi74iy4sv4ghGjRXIO4SZfWzsc8ESh+rZH9j2Wnq0
AEDn7oo5klBgWlHnHzsV5DabwQm/6fYFC0ozoBuZw6Xxhnxdg+FMUT8+7DhwexZG
tzfjOD6eoWGVA11N8693EW8KWEO+ViDZNteN5dmCtb7TN7h/1hw4jY3vOn78VLVn
p6ZuH5qxJYDc0e/jcR+PACdX4DZE3ucmt1YS6A0oj38vHPrmjbuRRY4g3+FVmVRk
wtgfbuGFw4Gv4kS/lmG3Uj3HJBnFeg==
=nXr1
-----END PGP SIGNATURE-----
--=-=-=--