X-Loop: help-debbugs@HIDDEN Subject: bug#24737: `guix lint` should not check patch-file-names on inherited sources Resent-From: Marius Bakke <mbakke@HIDDEN> Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> Resent-CC: bug-guix@HIDDEN Resent-Date: Wed, 19 Oct 2016 15:33:01 +0000 Resent-Message-ID: <handler.24737.B.14768911684945 <at> debbugs.gnu.org> Resent-Sender: help-debbugs@HIDDEN X-GNU-PR-Message: report 24737 X-GNU-PR-Package: guix X-GNU-PR-Keywords: To: 24737 <at> debbugs.gnu.org X-Debbugs-Original-To: bug-guix@HIDDEN Received: via spool by submit <at> debbugs.gnu.org id=B.14768911684945 (code B ref -1); Wed, 19 Oct 2016 15:33:01 +0000 Received: (at submit) by debbugs.gnu.org; 19 Oct 2016 15:32:48 +0000 Received: from localhost ([127.0.0.1]:39930 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1bwsrH-0001Hh-Q4 for submit <at> debbugs.gnu.org; Wed, 19 Oct 2016 11:32:48 -0400 Received: from eggs.gnu.org ([208.118.235.92]:45567) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <mbakke@HIDDEN>) id 1bwrHw-000705-0Q for submit <at> debbugs.gnu.org; Wed, 19 Oct 2016 09:52:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <mbakke@HIDDEN>) id 1bwrHq-0000yZ-0h for submit <at> debbugs.gnu.org; Wed, 19 Oct 2016 09:52:06 -0400 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on eggs.gnu.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,FREEMAIL_FROM, T_DKIM_INVALID autolearn=disabled version=3.3.2 Received: from lists.gnu.org ([2001:4830:134:3::11]:49735) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from <mbakke@HIDDEN>) id 1bwrHp-0000yP-Sv for submit <at> debbugs.gnu.org; Wed, 19 Oct 2016 09:52:05 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36500) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from <mbakke@HIDDEN>) id 1bwrHo-0007b9-1Z for bug-guix@HIDDEN; Wed, 19 Oct 2016 09:52:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <mbakke@HIDDEN>) id 1bwrHk-0000ti-5Q for bug-guix@HIDDEN; Wed, 19 Oct 2016 09:52:04 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:57043) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from <mbakke@HIDDEN>) id 1bwrHk-0000tF-1b for bug-guix@HIDDEN; Wed, 19 Oct 2016 09:52:00 -0400 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailout.nyi.internal (Postfix) with ESMTP id 7B97A2064E for <bug-guix@HIDDEN>; Wed, 19 Oct 2016 09:51:59 -0400 (EDT) Received: from frontend2 ([10.202.2.161]) by compute4.internal (MEProxy); Wed, 19 Oct 2016 09:51:59 -0400 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=fastmail.com; h= content-type:date:from:message-id:mime-version:subject:to :x-sasl-enc:x-sasl-enc; s=mesmtp; bh=5LX/cTbyHdBYllSPrVCpk6Ty7qM =; b=WK5PbxDbOlmuPVAik+etqtJY0xP+Cwn/X42opYaJIlmW20ae+eho2kXjfQl bC/AgqXeRT589pqinniJeqWAm77bF29k8GRVBWqnPTjdN3Yg7jEUQox+yUd/3q/H lziHoyQLN51jBj1sRkorZjTUWytjsdCPVh5xhm2H7yBkiVh0= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=content-type:date:from:message-id :mime-version:subject:to:x-sasl-enc:x-sasl-enc; s=smtpout; bh=5L X/cTbyHdBYllSPrVCpk6Ty7qM=; b=h31QhKyCykKCoNqqLH2P4vVw0fwHIdfbrz C3YUxLq6f9jug2p2QxX5gUIEAuGY/E6y9eNbYZoSM0Rq35kECeNpOxj1lqbqIPTi Gl5wcClWQryWKI8HC9IsLw3ExMNVe5FPAj+xKmPQX0h9sLHh0MymsXTz6HOi2BgG aQLTgqInQ= X-Sasl-enc: Xy89qIbuNPujDAS8Wvg9TfJH/65dkSXU3ijAiMvG2EzO 1476885119 Received: from localhost (169.243.187.81.in-addr.arpa [81.187.243.169]) by mail.messagingengine.com (Postfix) with ESMTPA id 1C3D0CC0C0 for <bug-guix@HIDDEN>; Wed, 19 Oct 2016 09:51:59 -0400 (EDT) From: Marius Bakke <mbakke@HIDDEN> Date: Wed, 19 Oct 2016 14:51:54 +0100 Message-ID: <87zim05tvp.fsf@HIDDEN> MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 2001:4830:134:3::11 X-Spam-Score: -4.4 (----) X-Mailman-Approved-At: Wed, 19 Oct 2016 11:32:47 -0400 X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -4.4 (----) After patching 'notmuch', `guix lint -c patch-file-names` does not pass for 'python-notmuch' which inherits the source from 'notmuch'.
Content-Disposition: inline Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Mailer: MIME-tools 5.505 (Entity 5.505) Content-Type: text/plain; charset=utf-8 X-Loop: help-debbugs@HIDDEN From: help-debbugs@HIDDEN (GNU bug Tracking System) To: Marius Bakke <mbakke@HIDDEN> Subject: bug#24737: Acknowledgement (`guix lint` should not check patch-file-names on inherited sources) Message-ID: <handler.24737.B.14768911684945.ack <at> debbugs.gnu.org> References: <87zim05tvp.fsf@HIDDEN> X-Gnu-PR-Message: ack 24737 X-Gnu-PR-Package: guix Reply-To: 24737 <at> debbugs.gnu.org Date: Wed, 19 Oct 2016 15:33:01 +0000 Thank you for filing a new bug report with debbugs.gnu.org. This is an automatically generated reply to let you know your message has been received. Your message is being forwarded to the package maintainers and other interested parties for their attention; they will reply in due course. Your message has been sent to the package maintainer(s): bug-guix@HIDDEN If you wish to submit further information on this problem, please send it to 24737 <at> debbugs.gnu.org. Please do not send mail to help-debbugs@HIDDEN unless you wish to report a problem with the Bug-tracking system. --=20 24737: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=3D24737 GNU Bug Tracking System Contact help-debbugs@HIDDEN with problems
X-Loop: help-debbugs@HIDDEN Subject: bug#24737: `guix lint` should not check patch-file-names on inherited sources Resent-From: ludo@HIDDEN (Ludovic =?UTF-8?Q?Court=C3=A8s?=) Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> Resent-CC: bug-guix@HIDDEN Resent-Date: Wed, 19 Oct 2016 19:52:02 +0000 Resent-Message-ID: <handler.24737.B24737.147690671130168 <at> debbugs.gnu.org> Resent-Sender: help-debbugs@HIDDEN X-GNU-PR-Message: followup 24737 X-GNU-PR-Package: guix X-GNU-PR-Keywords: To: Marius Bakke <mbakke@HIDDEN> Cc: 24737 <at> debbugs.gnu.org Received: via spool by 24737-submit <at> debbugs.gnu.org id=B24737.147690671130168 (code B ref 24737); Wed, 19 Oct 2016 19:52:02 +0000 Received: (at 24737) by debbugs.gnu.org; 19 Oct 2016 19:51:51 +0000 Received: from localhost ([127.0.0.1]:40077 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1bwwty-0007qW-VY for submit <at> debbugs.gnu.org; Wed, 19 Oct 2016 15:51:51 -0400 Received: from eggs.gnu.org ([208.118.235.92]:46818) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <ludo@HIDDEN>) id 1bwwtx-0007qK-4j for 24737 <at> debbugs.gnu.org; Wed, 19 Oct 2016 15:51:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <ludo@HIDDEN>) id 1bwwto-0006yH-2r for 24737 <at> debbugs.gnu.org; Wed, 19 Oct 2016 15:51:44 -0400 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on eggs.gnu.org X-Spam-Level: X-Spam-Status: No, score=-0.3 required=5.0 tests=BAYES_40,RP_MATCHES_RCVD autolearn=disabled version=3.3.2 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:58364) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from <ludo@HIDDEN>) id 1bwwtn-0006yD-Vt; Wed, 19 Oct 2016 15:51:40 -0400 Received: from reverse-83.fdn.fr ([80.67.176.83]:38912 helo=pluto) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from <ludo@HIDDEN>) id 1bwwtn-0001nV-8k; Wed, 19 Oct 2016 15:51:39 -0400 From: ludo@HIDDEN (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <87zim05tvp.fsf@HIDDEN> X-URL: http://www.fdn.fr/~lcourtes/ X-Revolutionary-Date: 28 =?UTF-8?Q?Vend=C3=A9miaire?= an 225 de la =?UTF-8?Q?R=C3=A9volution?= X-PGP-Key-ID: 0x090B11993D9AEBB5 X-PGP-Key: http://www.fdn.fr/~lcourtes/ludovic.asc X-PGP-Fingerprint: 3CE4 6455 8A84 FDC6 9DB4 0CFB 090B 1199 3D9A EBB5 X-OS: x86_64-unknown-linux-gnu Date: Wed, 19 Oct 2016 21:51:32 +0200 In-Reply-To: <87zim05tvp.fsf@HIDDEN> (Marius Bakke's message of "Wed, 19 Oct 2016 14:51:54 +0100") Message-ID: <87eg3c5d8b.fsf@HIDDEN> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 2001:4830:134:3::e X-Spam-Score: -5.3 (-----) X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -5.3 (-----) Marius Bakke <mbakke@HIDDEN> skribis: > After patching 'notmuch', `guix lint -c patch-file-names` does not pass > for 'python-notmuch' which inherits the source from 'notmuch'. I agree but that=E2=80=99s not quite possible: the =E2=80=9Cinheritance=E2= =80=9D relation (which is really just a copy of a record) is not known at run time. So we=E2=80=99d need another trick to guess whether a patch is coming from elsewhere and should consequently be ignored by =E2=80=98lint=E2=80=99. Thanks, Ludo=E2=80=99.
X-Loop: help-debbugs@HIDDEN Subject: bug#24737: `guix lint` should not check patch-file-names on inherited sources Resent-From: Marius Bakke <mbakke@HIDDEN> Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> Resent-CC: bug-guix@HIDDEN Resent-Date: Sat, 22 Oct 2016 18:54:02 +0000 Resent-Message-ID: <handler.24737.B24737.147716238620397 <at> debbugs.gnu.org> Resent-Sender: help-debbugs@HIDDEN X-GNU-PR-Message: followup 24737 X-GNU-PR-Package: guix X-GNU-PR-Keywords: To: Ludovic =?UTF-8?Q?Court=C3=A8s?= <ludo@HIDDEN> Cc: 24737 <at> debbugs.gnu.org Received: via spool by 24737-submit <at> debbugs.gnu.org id=B24737.147716238620397 (code B ref 24737); Sat, 22 Oct 2016 18:54:02 +0000 Received: (at 24737) by debbugs.gnu.org; 22 Oct 2016 18:53:06 +0000 Received: from localhost ([127.0.0.1]:46563 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1by1Pl-0005Iv-Ns for submit <at> debbugs.gnu.org; Sat, 22 Oct 2016 14:53:05 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:58553) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <mbakke@HIDDEN>) id 1by1Pk-0005In-FK for 24737 <at> debbugs.gnu.org; Sat, 22 Oct 2016 14:53:04 -0400 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailout.nyi.internal (Postfix) with ESMTP id E0582205D8; Sat, 22 Oct 2016 14:53:03 -0400 (EDT) Received: from frontend1 ([10.202.2.160]) by compute4.internal (MEProxy); Sat, 22 Oct 2016 14:53:03 -0400 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=fastmail.com; h=cc :content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to:x-sasl-enc:x-sasl-enc; s=mesmtp; bh=ZN/+w zj7siUROYnMm5s0wrikGeE=; b=Ycd6dinRS9CWemZoAau+0+worJBysQBJItdZG ZYduZbYxbScLFssnphnyCsyBJM96/P3wCwNKwBj5dnDC/2h9i5LWFI3Xflixg+UT thMawMWwcLcQuyAgmK4bqioGFIro/PdXF8KaLcbYqGkZYexSM9XVrGralTC9An6m zItk14= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-sasl-enc :x-sasl-enc; s=smtpout; bh=ZN/+wzj7siUROYnMm5s0wrikGeE=; b=GI1Ga 109EdtoXhlZLQObOBNQVoZ7/dG6JgRTCmDRLlYY7+TRVpDtPoTJBq/QRNbl2QfoU M2YevfydpFJoHBGjPujKcCe+j64Ts0VyYOl+qz9YASttxbwxQsYSVlgQJmVhwF2J GBoz9/4u9cUPgD0Bdx1goNLKDKD+qz+qe4JXZ0= X-Sasl-enc: +/ACXg8s28LsWFKO0vAFe4/MsoXY/6eGTLSNj3uY38Cx 1477162383 Received: from localhost (169.243.187.81.in-addr.arpa [81.187.243.169]) by mail.messagingengine.com (Postfix) with ESMTPA id 8B69CF29CB; Sat, 22 Oct 2016 14:53:03 -0400 (EDT) From: Marius Bakke <mbakke@HIDDEN> In-Reply-To: <87eg3c5d8b.fsf@HIDDEN> References: <87zim05tvp.fsf@HIDDEN> <87eg3c5d8b.fsf@HIDDEN> Date: Sat, 22 Oct 2016 19:53:02 +0100 Message-ID: <87insk5i7l.fsf@HIDDEN> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-Spam-Score: -0.7 (/) X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -0.7 (/) --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Ludovic Court=C3=A8s <ludo@HIDDEN> writes: > Marius Bakke <mbakke@HIDDEN> skribis: > >> After patching 'notmuch', `guix lint -c patch-file-names` does not pass >> for 'python-notmuch' which inherits the source from 'notmuch'. > > I agree but that=E2=80=99s not quite possible: the =E2=80=9Cinheritance= =E2=80=9D relation (which > is really just a copy of a record) is not known at run time. > > So we=E2=80=99d need another trick to guess whether a patch is coming from > elsewhere and should consequently be ignored by =E2=80=98lint=E2=80=99. Here is a "RFC" patch that thwarts the warning if the source file name is different from the package name. Not sure how to properly make it part of the procedure, so that the checks are actually skipped as well. --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=0001-lint-Skip-patch-file-names-on-inherited-sources.patch From 160132bdc23b34c6331adf00af46af19dd8d737c Mon Sep 17 00:00:00 2001 From: Marius Bakke <mbakke@HIDDEN> Date: Sat, 22 Oct 2016 19:12:00 +0100 Subject: [PATCH] lint: Skip 'patch-file-names' on inherited sources. * guix/scripts/lint.scm (check-patch-file-names): Only report when the source file name matches the package name. --- guix/scripts/lint.scm | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/guix/scripts/lint.scm b/guix/scripts/lint.scm index d6281ea..ba1bbc3 100644 --- a/guix/scripts/lint.scm +++ b/guix/scripts/lint.scm @@ -497,10 +497,13 @@ patch could not be found." (_ #f)) ;must be an <origin> or something like that. (or (and=> (package-source package) origin-patches) '())) - (emit-warning - package - (_ "file names of patches should start with the package name") - 'patch-file-names)))) + ;; Skip report when the source file name differs (i.e. inherited). + (and (string-prefix? (package-name package) + (origin-actual-file-name (package-source package))) + (emit-warning + package + (_ "file names of patches should start with the package name") + 'patch-file-names))))) (define (escape-quotes str) "Replace any quote character in STR by an escaped quote character." -- 2.10.1 --=-=-=--
X-Loop: help-debbugs@HIDDEN Subject: bug#24737: `guix lint` should not check patch-file-names on inherited sources Resent-From: Marius Bakke <mbakke@HIDDEN> Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> Resent-CC: bug-guix@HIDDEN Resent-Date: Sat, 22 Oct 2016 19:08:02 +0000 Resent-Message-ID: <handler.24737.B24737.147716324021799 <at> debbugs.gnu.org> Resent-Sender: help-debbugs@HIDDEN X-GNU-PR-Message: followup 24737 X-GNU-PR-Package: guix X-GNU-PR-Keywords: To: Ludovic =?UTF-8?Q?Court=C3=A8s?= <ludo@HIDDEN> Cc: 24737 <at> debbugs.gnu.org Received: via spool by 24737-submit <at> debbugs.gnu.org id=B24737.147716324021799 (code B ref 24737); Sat, 22 Oct 2016 19:08:02 +0000 Received: (at 24737) by debbugs.gnu.org; 22 Oct 2016 19:07:20 +0000 Received: from localhost ([127.0.0.1]:46586 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1by1dY-0005fX-Ev for submit <at> debbugs.gnu.org; Sat, 22 Oct 2016 15:07:20 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:49699) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <mbakke@HIDDEN>) id 1by1dW-0005fQ-UT for 24737 <at> debbugs.gnu.org; Sat, 22 Oct 2016 15:07:19 -0400 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailout.nyi.internal (Postfix) with ESMTP id 5E93320566; Sat, 22 Oct 2016 15:07:17 -0400 (EDT) Received: from frontend1 ([10.202.2.160]) by compute4.internal (MEProxy); Sat, 22 Oct 2016 15:07:17 -0400 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=fastmail.com; h=cc :content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-sasl-enc :x-sasl-enc; s=mesmtp; bh=A12rhqGbVxBicQQ9PRH5tb931nE=; b=d0TVdD fJaxDyTCkYusrnN9YK0Rpots7qkmX3V951D8GnAxlEczTxVcmE56JE613/MyW852 k/1yiytRvKyOwYy1fW8jcdyk4b6aPk4x3PMksTgt7U+ebliUGIeJAxKpvB4ewWim ZhhtBKUAL9h6EMBnLfLywo3tGtU/PuJIHMHh4= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-sasl-enc:x-sasl-enc; s=smtpout; bh=A12rhqGbVxBicQQ 9PRH5tb931nE=; b=P1HRYf5AbcVMsR06b6co8nGOtKgm3OayvLnVTsgPfd4H7mn WIOA+uM4xLKjMIyiWbWKMNB//+BOf0GnKnjxYim0xbyQBjbfGytqpPoTzioYsBa3 PdQQERgm/rk99bQynaRK/7DoDHHIacre0JPwzrY1bjR5F6M2US2K0GAt4Wf4= X-Sasl-enc: O9bp0xpwJKfbJoBjLNBBcXu0NeGR6IOtxJSIfwSaqhwT 1477163237 Received: from localhost (169.243.187.81.in-addr.arpa [81.187.243.169]) by mail.messagingengine.com (Postfix) with ESMTPA id 0ED4CF29CB; Sat, 22 Oct 2016 15:07:16 -0400 (EDT) From: Marius Bakke <mbakke@HIDDEN> In-Reply-To: <87insk5i7l.fsf@HIDDEN> References: <87zim05tvp.fsf@HIDDEN> <87eg3c5d8b.fsf@HIDDEN> <87insk5i7l.fsf@HIDDEN> Date: Sat, 22 Oct 2016 20:07:15 +0100 Message-ID: <87funo5hjw.fsf@HIDDEN> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Spam-Score: -0.7 (/) X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -0.7 (/) Marius Bakke <mbakke@HIDDEN> writes: > Ludovic Court=C3=A8s <ludo@HIDDEN> writes: > >> Marius Bakke <mbakke@HIDDEN> skribis: >> >>> After patching 'notmuch', `guix lint -c patch-file-names` does not pass >>> for 'python-notmuch' which inherits the source from 'notmuch'. >> >> I agree but that=E2=80=99s not quite possible: the =E2=80=9Cinheritance= =E2=80=9D relation (which >> is really just a copy of a record) is not known at run time. >> >> So we=E2=80=99d need another trick to guess whether a patch is coming fr= om >> elsewhere and should consequently be ignored by =E2=80=98lint=E2=80=99. > > Here is a "RFC" patch that thwarts the warning if the source file name > is different from the package name. Not sure how to properly make it > part of the procedure, so that the checks are actually skipped as well. I just realized this approach will skip this check completely, if there are no packages that are named the same as origin (e.g. in the case of the soon-to-be-added avro, where the source is shared between the various avro-{c,python,java} etc packages.) The best approach is probably to check patch-file-names against (origin-actual-file-name (package-source package)), assuming one can extract the "base" name of origin-actual-file-name reliably.
X-Loop: help-debbugs@HIDDEN Subject: bug#24737: `guix lint` should not check patch-file-names on inherited sources Resent-From: ludo@HIDDEN (Ludovic =?UTF-8?Q?Court=C3=A8s?=) Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> Resent-CC: bug-guix@HIDDEN Resent-Date: Mon, 24 Oct 2016 20:05:02 +0000 Resent-Message-ID: <handler.24737.B24737.147733944531042 <at> debbugs.gnu.org> Resent-Sender: help-debbugs@HIDDEN X-GNU-PR-Message: followup 24737 X-GNU-PR-Package: guix X-GNU-PR-Keywords: To: Marius Bakke <mbakke@HIDDEN> Cc: 24737 <at> debbugs.gnu.org Received: via spool by 24737-submit <at> debbugs.gnu.org id=B24737.147733944531042 (code B ref 24737); Mon, 24 Oct 2016 20:05:02 +0000 Received: (at 24737) by debbugs.gnu.org; 24 Oct 2016 20:04:05 +0000 Received: from localhost ([127.0.0.1]:54910 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1bylTY-00084c-TF for submit <at> debbugs.gnu.org; Mon, 24 Oct 2016 16:04:05 -0400 Received: from eggs.gnu.org ([208.118.235.92]:49625) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <ludo@HIDDEN>) id 1bylTX-000849-Lw for 24737 <at> debbugs.gnu.org; Mon, 24 Oct 2016 16:04:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <ludo@HIDDEN>) id 1bylTP-00071U-DU for 24737 <at> debbugs.gnu.org; Mon, 24 Oct 2016 16:03:58 -0400 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on eggs.gnu.org X-Spam-Level: X-Spam-Status: No, score=-0.6 required=5.0 tests=BAYES_50,RP_MATCHES_RCVD autolearn=disabled version=3.3.2 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:55944) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from <ludo@HIDDEN>) id 1bylTP-00071O-9b; Mon, 24 Oct 2016 16:03:55 -0400 Received: from reverse-83.fdn.fr ([80.67.176.83]:55714 helo=pluto) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from <ludo@HIDDEN>) id 1bylTO-0002be-OI; Mon, 24 Oct 2016 16:03:55 -0400 From: ludo@HIDDEN (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <87zim05tvp.fsf@HIDDEN> <87eg3c5d8b.fsf@HIDDEN> <87insk5i7l.fsf@HIDDEN> <87funo5hjw.fsf@HIDDEN> X-URL: http://www.fdn.fr/~lcourtes/ X-Revolutionary-Date: 3 Brumaire an 225 de la =?UTF-8?Q?R=C3=A9volution?= X-PGP-Key-ID: 0x090B11993D9AEBB5 X-PGP-Key: http://www.fdn.fr/~lcourtes/ludovic.asc X-PGP-Fingerprint: 3CE4 6455 8A84 FDC6 9DB4 0CFB 090B 1199 3D9A EBB5 X-OS: x86_64-unknown-linux-gnu Date: Mon, 24 Oct 2016 22:03:52 +0200 In-Reply-To: <87funo5hjw.fsf@HIDDEN> (Marius Bakke's message of "Sat, 22 Oct 2016 20:07:15 +0100") Message-ID: <871sz5345z.fsf@HIDDEN> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 2001:4830:134:3::e X-Spam-Score: -6.4 (------) X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -6.4 (------) Marius Bakke <mbakke@HIDDEN> skribis: > Marius Bakke <mbakke@HIDDEN> writes: > >> Ludovic Court=C3=A8s <ludo@HIDDEN> writes: >> >>> Marius Bakke <mbakke@HIDDEN> skribis: >>> >>>> After patching 'notmuch', `guix lint -c patch-file-names` does not pass >>>> for 'python-notmuch' which inherits the source from 'notmuch'. >>> >>> I agree but that=E2=80=99s not quite possible: the =E2=80=9Cinheritance= =E2=80=9D relation (which >>> is really just a copy of a record) is not known at run time. >>> >>> So we=E2=80=99d need another trick to guess whether a patch is coming f= rom >>> elsewhere and should consequently be ignored by =E2=80=98lint=E2=80=99. >> >> Here is a "RFC" patch that thwarts the warning if the source file name >> is different from the package name. Not sure how to properly make it >> part of the procedure, so that the checks are actually skipped as well. > > I just realized this approach will skip this check completely, if there > are no packages that are named the same as origin (e.g. in the case of > the soon-to-be-added avro, where the source is shared between the > various avro-{c,python,java} etc packages.) > > The best approach is probably to check patch-file-names against > (origin-actual-file-name (package-source package)), assuming one can > extract the "base" name of origin-actual-file-name reliably. (=E2=80=98origin-actual-file-name=E2=80=99 already returns a basename.) Could you check whether the patch your proposed works well for some of the annoying cases we currently have, and also adds those as test cases in =E2=80=98tests/lint.scm=E2=80=99? (See the manual on how to run the tes= ts (info "(guix) Running the Test Suite").) If that works well enough, we should go for it. The only 100% reliable way to address this, I think, would be to build a patch to package mapping, and then make sure that for each patch, at least one of the corresponding packages has a matching name. The problem is that =E2=80=98lint=E2=80=99 is currently designed to work one pa= ckage at a time. Thanks! Ludo=E2=80=99.
Received: (at control) by debbugs.gnu.org; 2 Nov 2016 22:18:12 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Wed Nov 02 18:18:12 2016 Received: from localhost ([127.0.0.1]:40937 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1c23rH-0002JR-Vg for submit <at> debbugs.gnu.org; Wed, 02 Nov 2016 18:18:12 -0400 Received: from eggs.gnu.org ([208.118.235.92]:33610) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <ludo@HIDDEN>) id 1c23rG-0002JG-Nr for control <at> debbugs.gnu.org; Wed, 02 Nov 2016 18:18:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <ludo@HIDDEN>) id 1c23r9-0006MD-0g for control <at> debbugs.gnu.org; Wed, 02 Nov 2016 18:18:05 -0400 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on eggs.gnu.org X-Spam-Level: X-Spam-Status: No, score=-2.7 required=5.0 tests=BAYES_20,RP_MATCHES_RCVD autolearn=disabled version=3.3.2 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:41697) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from <ludo@HIDDEN>) id 1c23r8-0006M9-Tc for control <at> debbugs.gnu.org; Wed, 02 Nov 2016 18:18:02 -0400 Received: from reverse-83.fdn.fr ([80.67.176.83]:36650 helo=pluto) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from <ludo@HIDDEN>) id 1c23r8-0004lA-AP for control <at> debbugs.gnu.org; Wed, 02 Nov 2016 18:18:02 -0400 Date: Wed, 02 Nov 2016 23:18:00 +0100 Message-Id: <87h97ple5j.fsf@HIDDEN> To: control <at> debbugs.gnu.org From: ludo@HIDDEN (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: control message for bug #24737 MIME-version: 1.0 Content-type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 2001:4830:134:3::e X-Spam-Score: -7.7 (-------) X-Debbugs-Envelope-To: control X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -7.7 (-------) severity 24737 minor
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997 nCipher Corporation Ltd,
1994-97 Ian Jackson.