From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp0 ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms0.migadu.com with LMTPS id sE3mGxMSmGEGCwAAgWs5BA (envelope-from ) for ; Fri, 19 Nov 2021 22:07:31 +0100 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp0 with LMTPS id sEaXFxMSmGHNEwAA1q6Kng (envelope-from ) for ; Fri, 19 Nov 2021 21:07:31 +0000 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 2E87628A2D for ; Fri, 19 Nov 2021 22:07:31 +0100 (CET) Received: from localhost ([::1]:53616 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1moB6Y-0004ai-6y for larch@yhetil.org; Fri, 19 Nov 2021 16:07:30 -0500 Received: from eggs.gnu.org ([209.51.188.92]:59560) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1moB67-0004aV-JL for guix-patches@gnu.org; Fri, 19 Nov 2021 16:07:03 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:57352) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1moB66-00031t-9j for guix-patches@gnu.org; Fri, 19 Nov 2021 16:07:03 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1moB66-0001IY-5O for guix-patches@gnu.org; Fri, 19 Nov 2021 16:07:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#51985] lint: Adjust patch file length check. Resent-From: Vagrant Cascadian Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Fri, 19 Nov 2021 21:07:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 51985 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: To: 51985@debbugs.gnu.org X-Debbugs-Original-To: guix-patches@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.16373559754917 (code B ref -1); Fri, 19 Nov 2021 21:07:01 +0000 Received: (at submit) by debbugs.gnu.org; 19 Nov 2021 21:06:15 +0000 Received: from localhost ([127.0.0.1]:40665 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1moB5L-0001HF-En for submit@debbugs.gnu.org; Fri, 19 Nov 2021 16:06:15 -0500 Received: from lists.gnu.org ([209.51.188.17]:55112) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1moB5K-0001H8-DJ for submit@debbugs.gnu.org; Fri, 19 Nov 2021 16:06:14 -0500 Received: from eggs.gnu.org ([209.51.188.92]:59452) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1moB5J-0004Qu-SD for guix-patches@gnu.org; Fri, 19 Nov 2021 16:06:14 -0500 Received: from cascadia.aikidev.net ([173.255.214.101]:35680) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1moB5E-0002p1-Ti for guix-patches@gnu.org; Fri, 19 Nov 2021 16:06:13 -0500 Received: from localhost (unknown [IPv6:2600:3c01:e000:21:21:21:0:100b]) (Authenticated sender: vagrant@cascadia.debian.net) by cascadia.aikidev.net (Postfix) with ESMTPSA id D2D6F1AA41 for ; Fri, 19 Nov 2021 13:06:01 -0800 (PST) From: Vagrant Cascadian Date: Fri, 19 Nov 2021 13:05:58 -0800 Message-ID: <875ysnsvnt.fsf@yucca> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="==-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" Received-SPF: none client-ip=173.255.214.101; envelope-from=vagrant@debian.org; helo=cascadia.aikidev.net X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action 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+larch=yhetil.org@gnu.org Sender: "Guix-patches" X-Migadu-Flow: FLOW_IN ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1637356051; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:list-id:list-help:list-unsubscribe:list-subscribe: list-post; bh=J1UONkvwVp3FWSLOzyZctVIZ5KI0pTYT1GoHnKC6CCs=; b=VQexD4+oFw636TBBwagelM6HxxykDfd00imD9Df1AI9Mg7s1IKdDvCrKAxxePlOEIfL7Ee BsA4DhM/W+VOoD+bgcrg2v9CfFp9hx4EsUh4AcwL+2p8ZjmdigxnCpuhFPfWZOtbTSHWb8 oa5p/X8fsPzkNdkt1IfoQ9lpaYRR0F2yX6bmjCTbysAfojXXPhfw07RsUDyiFCgrH63yM+ f0KQ/YLyvZ3qjmnNRUh77U/ob59iSkkrSSAaozwaBERxhmY0wPeGZACDep0o+IZhSNtg2Z Nf5J2Oph6Z5f2492d5mCg87Q78E9N57guDkPPY0CQu3yPl6PXbWIUjNHs+FjAg== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1637356051; a=rsa-sha256; cv=none; b=VAHba0+UmR50peL/+dQERQxPRpoVA6dkg3vdM9Ni3LNxXfZkVmuuQfvvqLm58Aa4u2IwNu n5XuSTWJuuxFLixnnGTumng3zAz7rMqJrIzrqSvVMeqt4sHkXAno63YCQOuQM8MPBvJsNa HZLDTNUpF5NGXvu9rkxS7eUyOGeSHqxeWIdW+ZYHck6wrtcl1GXck7UFL6gU659o3/7GKy zdqgJIJOQaU7QxHioj738WpFdVjBMZYjT/wFQMFLsoV1IR4Dxknfp6aiIGeuQZC8LC+cmt O1HSMIrS2d5OWvMHs2914U4uiHxnTF4mz+J5O7NQAnw6A16qgYNWOp8uSnyPZA== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Migadu-Spam-Score: -4.67 Authentication-Results: aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Migadu-Queue-Id: 2E87628A2D X-Spam-Score: -4.67 X-Migadu-Scanner: scn1.migadu.com X-TUID: tocGiGGKW9aZ --==-=-= Content-Type: multipart/mixed; boundary="=-=-=" --=-=-= Content-Type: text/plain The current guix lint check is a bit overly conservative, and reports several results which do not in practice actually cause issues. This patch proposes to reduce the size by two characters (leaving only two patches on guix master that need to be adjusted), uses a version string more like what actually might be included in a tarball built using "make dist", and adds a comment describing what the arbitrary string actually is supposed to represent. This should still even leave a little wiggle-room when guix hits version 100+ and/or 1000000+ commits, by which time hopefully guix has switched to a tarball format that doesn't have such a short arbitrary file length limit! live well, vagrant --=-=-= Content-Type: text/x-diff Content-Disposition: inline; filename=0001-lint-Adjust-patch-file-length-check.patch Content-Transfer-Encoding: quoted-printable From=206ad2050a8bbc308a328d30d4f66cb229d868b79d Mon Sep 17 00:00:00 2001 From: Vagrant Cascadian Date: Fri, 19 Nov 2021 12:14:19 -0800 Subject: [PATCH] lint: Adjust patch file length check. * guix/lint.scm (check-patch-file-names): Adjust margin used to check for patch file lengths. =2D-- guix/lint.scm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/guix/lint.scm b/guix/lint.scm index ac2e7b3841..39b4a2ae85 100644 =2D-- a/guix/lint.scm +++ b/guix/lint.scm @@ -957,7 +957,10 @@ patch could not be found." =20 ;; Check whether we're reaching tar's maximum file name length. (let ((prefix (string-length (%distro-directory))) =2D (margin (string-length "guix-2.0.0rc3-10000-1234567890/")) + ;; Margin approximating the largest path that "make dist" might + ;; create, with a release candidate version, 123456 commits, and + ;; git commit hash abcde0. + (margin (string-length "guix-12.0.0rc3-123456-abcde0/")) (max 99)) (filter-map (match-lambda ((? string? patch) =2D-=20 2.30.2 --=-=-=-- --==-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEARYKAB0WIQRlgHNhO/zFx+LkXUXcUY/If5cWqgUCYZgRtgAKCRDcUY/If5cW qup5AQD81KiqA/DfiPpt9xQTyb4qvkp++22qCv9Wgl0Di+JB3wD/TyO2cTC2XkoX dGk/JnNlOcLquHtSuQ4FmA0thjC/fgw= =qx7F -----END PGP SIGNATURE----- --==-=-=--