From patchwork Fri Jan 5 19:52:52 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Maxim Cournoyer X-Patchwork-Id: 58396 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id 6C3BC27BBEA; Fri, 5 Jan 2024 19:54:46 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-2.7 required=5.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FROM,MAILING_LIST_MULTI, SPF_HELO_PASS autolearn=unavailable autolearn_force=no version=3.4.6 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTPS id 6379027BBE2 for ; Fri, 5 Jan 2024 19:54:45 +0000 (GMT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rLqGb-00011N-4l; Fri, 05 Jan 2024 14:54:05 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rLqGZ-00010y-61 for guix-patches@gnu.org; Fri, 05 Jan 2024 14:54:03 -0500 Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rLqGY-0007d4-OA; Fri, 05 Jan 2024 14:54:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1rLqGZ-0000eD-Dt; Fri, 05 Jan 2024 14:54:03 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#68242] [PATCH core-updates v2 3/5] packages: Repack patched source archives via zstd by default. Resent-From: Maxim Cournoyer Original-Sender: "Debbugs-submit" Resent-CC: guix@cbaines.net, dev@jpoiret.xyz, ludo@gnu.org, othacehe@gnu.org, rekado@elephly.net, zimon.toutoune@gmail.com, me@tobias.gr, guix-patches@gnu.org Resent-Date: Fri, 05 Jan 2024 19:54:03 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 68242 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 68242@debbugs.gnu.org Cc: Maxim Cournoyer , Christopher Baines , Josselin Poiret , Ludovic =?utf-8?q?Court=C3=A8s?= , Mathieu Othacehe , Ricardo Wurmus , Simon Tournier , Tobias Geerinckx-Rice X-Debbugs-Original-Xcc: Christopher Baines , Josselin Poiret , Ludovic =?utf-8?q?Court=C3=A8s?= , Mathieu Othacehe , Ricardo Wurmus , Simon Tournier , Tobias Geerinckx-Rice Received: via spool by 68242-submit@debbugs.gnu.org id=B68242.17044844212423 (code B ref 68242); Fri, 05 Jan 2024 19:54:03 +0000 Received: (at 68242) by debbugs.gnu.org; 5 Jan 2024 19:53:41 +0000 Received: from localhost ([127.0.0.1]:57966 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rLqGC-0000cw-Kb for submit@debbugs.gnu.org; Fri, 05 Jan 2024 14:53:41 -0500 Received: from mail-qk1-x72e.google.com ([2607:f8b0:4864:20::72e]:58371) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rLqG8-0000c0-Dy for 68242@debbugs.gnu.org; Fri, 05 Jan 2024 14:53:38 -0500 Received: by mail-qk1-x72e.google.com with SMTP id af79cd13be357-781edcbb495so122176385a.0 for <68242@debbugs.gnu.org>; Fri, 05 Jan 2024 11:53:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1704484406; x=1705089206; darn=debbugs.gnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=WY2zrLmbm2SWhH3iTQSppY95X01nFe/0vLW/hINWGpY=; b=RslV6L+NmmK55zwrHqfyRAQAn6iNRQw6UzBh7HwUUmH2YTBEjV7rSemKjdXscRGl+j vaAMHpYr599RMWr6c09i/EWh4Xxr9C/6aVibU/BHyBwCspUtCdCaszbMrX+5o4sdmTyc 4eqMQ51D2eNGwNo6CPflV4IOhKPREbm1haKdotvarQhXA1NcIWvh6wwDZNp0H8kCcAEC lcyS8QDlt+n+ZZG0Zphtb/CA1Gi7rlXOJjdWCmeRA+6WmE5uEY7zvxL/B6hTPNUtc4KS ohlxByM5624U0Txt0Eet5fnIcpCM9aLfx/e4Q1IQnUPKo3nHPm78xR3i8msqW92hO7Oz JCeQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1704484406; x=1705089206; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=WY2zrLmbm2SWhH3iTQSppY95X01nFe/0vLW/hINWGpY=; b=p5v7UfjU59dsmdM7tXwP3+RKJqIV2bTNMJiN6Yimj1i1T4+FBB9nDGWHp4/a54hwku kxjrcXNUHBr6k3TcUbw3t+Kl95Tp1W6MXgQ2gwvrgkcByrOjSCxKezemNRttcEZowVx6 /XGgbx7eibI8cHB+qAy/7H7WJSXXWyxNia9/E6EpgGm5PjR3YxvM7xuxpv3Fy6d6Y+vy cq69GejHLznQfZc267NtLLsUiACzZQNw8GJdMxOXr6/NrAi7R7IfwhAs6Lj5TzL25ehi trC102T0eSyzAWdVFDsyINrRQVQlVQzpVNxScAipBhePS19qq/uTLkQkvAe/B4RmGwS+ UQLg== X-Gm-Message-State: AOJu0YwY8L6d58B6iBZARbfaDUTYpu3lTYxol9q9NdDtn4e+6x9bD8o4 j457GRHWML7SgUPsXaMizJiWem8TVa8= X-Google-Smtp-Source: AGHT+IGhUk0H4MjbPJhSlibPFyo4UzPrv900qr1r0x/K7mJjRFamDtAyijA1UqIyEDyYspIZdEYqIg== X-Received: by 2002:a05:620a:6019:b0:783:7df:7c7c with SMTP id dw25-20020a05620a601900b0078307df7c7cmr1771022qkb.67.1704484406124; Fri, 05 Jan 2024 11:53:26 -0800 (PST) Received: from localhost.localdomain (dsl-10-135-125.b2b2c.ca. [72.10.135.125]) by smtp.gmail.com with ESMTPSA id c22-20020a05620a11b600b00781e1c466e7sm823178qkk.24.2024.01.05.11.53.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Jan 2024 11:53:25 -0800 (PST) From: Maxim Cournoyer Date: Fri, 5 Jan 2024 14:52:52 -0500 Message-ID: X-Mailer: git-send-email 2.41.0 In-Reply-To: References: MIME-Version: 1.0 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org X-getmail-retrieved-from-mailbox: Patches * guix/build/utils.scm (compressor): Register zst file name extension. * guix/packages.scm (%standard-patch-inputs): Add zstd. (patch-and-repack): Rename tarxz-name nested procedure to tar-file-name, and accept a new 'ext' argument; adjust accordingly. Add zstd binding, and replace the XZ_DEFAULTS environment variable with ZSTD_NBTHREADS. Fallback to xz when zstd is not available. Change-Id: I614a6be8c87a4a0858eadce616c51d8e9b9fc020 --- (no changes since v1) guix/build/utils.scm | 1 + guix/packages.scm | 50 +++++++++++++++++++++++++------------------- 2 files changed, 30 insertions(+), 21 deletions(-) diff --git a/guix/build/utils.scm b/guix/build/utils.scm index e87066cc02..9c1e19f6d8 100644 --- a/guix/build/utils.scm +++ b/guix/build/utils.scm @@ -177,6 +177,7 @@ (define (compressor file-name) ((string-suffix? "lz" file-name) "lzip") ((string-suffix? "zip" file-name) "unzip") ((string-suffix? "xz" file-name) "xz") + ((string-suffix? "zst" file-name) "zstd") (else #f))) ;no compression used/unknown file extension (define (tarball? file-name) diff --git a/guix/packages.scm b/guix/packages.scm index cb8db925f8..ce1ba7c53a 100644 --- a/guix/packages.scm +++ b/guix/packages.scm @@ -5,7 +5,7 @@ ;;; Copyright © 2016 Alex Kost ;;; Copyright © 2017, 2019, 2020, 2022 Efraim Flashner ;;; Copyright © 2019 Marius Bakke -;;; Copyright © 2020, 2021 Maxim Cournoyer +;;; Copyright © 2020, 2021, 2024 Maxim Cournoyer ;;; Copyright © 2021 Chris Marusich ;;; Copyright © 2022 Maxime Devos ;;; Copyright © 2022 jgart @@ -862,6 +862,7 @@ (define (%standard-patch-inputs system) (module-ref (resolve-interface module) var)))))) `(("tar" ,(ref '(gnu packages base) 'tar)) ("xz" ,(ref '(gnu packages compression) 'xz)) + ("zstd" ,(ref '(gnu packages compression) 'zstd)) ("bzip2" ,(ref '(gnu packages compression) 'bzip2)) ("gzip" ,(ref '(gnu packages compression) 'gzip)) ("lzip" ,(ref '(gnu packages compression) 'lzip)) @@ -926,31 +927,35 @@ (define* (patch-and-repack source patches ;; Return true if DIRECTORY is a checkout (git, svn, etc). (string-suffix? "-checkout" directory)) - (define (tarxz-name file-name) - ;; Return a '.tar.xz' file name based on FILE-NAME. + (define (tar-file-name file-name ext) + ;; Return a '$filename.tar.$ext' file name based on FILE-NAME and EXT. (let ((base (if (numeric-extension? file-name) original-file-name (file-sans-extension file-name)))) (string-append base (if (equal? (file-extension base) "tar") - ".xz" - ".tar.xz")))) + (string-append "." ext) + (string-append ".tar." ext))))) (define instantiate-patch (match-lambda - ((? string? patch) ;deprecated + ((? string? patch) ;deprecated (local-file patch #:recursive? #t)) - ((? struct? patch) ;origin, local-file, etc. + ((? struct? patch) ;origin, local-file, etc. patch))) - (let ((tar (lookup-input "tar")) - (gzip (lookup-input "gzip")) - (bzip2 (lookup-input "bzip2")) - (lzip (lookup-input "lzip")) - (xz (lookup-input "xz")) - (patch (lookup-input "patch")) - (comp (and=> (compressor source-file-name) lookup-input)) - (patches (map instantiate-patch patches))) + (let* ((tar (lookup-input "tar")) + (gzip (lookup-input "gzip")) + (bzip2 (lookup-input "bzip2")) + (lzip (lookup-input "lzip")) + (xz (lookup-input "xz")) + (zstd (or (lookup-input "zstd") + ;; Fallback to xz in case zstd is not available, such as + ;; for bootstrap packages. + xz)) + (patch (lookup-input "patch")) + (comp (and=> (compressor source-file-name) lookup-input)) + (patches (map instantiate-patch patches))) (define build (with-imported-modules '((guix build utils)) #~(begin @@ -1028,12 +1033,12 @@ (define* (patch-and-repack source patches locale (system-error-errno args))))) (setenv "PATH" - (string-append #+xz "/bin" + (string-append #+zstd "/bin" (if #+comp (string-append ":" #+comp "/bin") ""))) - (setenv "XZ_DEFAULTS" (string-join (%xz-parallel-args))) + (setenv "ZSTD_NBTHREADS" (number->string (parallel-job-count))) ;; SOURCE may be either a directory, a tarball or a simple file. (let ((name (strip-store-file-name #+source)) @@ -1088,10 +1093,13 @@ (define* (patch-and-repack source patches (else ;single uncompressed file (copy-file file #$output))))))) - (let ((name (if (or (checkout? original-file-name) - (not (compressor original-file-name))) - original-file-name - (tarxz-name original-file-name)))) + (let* ((ext (if zstd + "zst" ;usual case + "xz")) ;zstd-less bootstrap-origin + (name (if (or (checkout? original-file-name) + (not (compressor original-file-name))) + original-file-name + (tar-file-name original-file-name ext)))) (gexp->derivation name build #:graft? #f #:system system