GNU logs - #24737, boring messages


Message sent to bug-guix@HIDDEN:


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'.




Message sent:


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


Message sent to bug-guix@HIDDEN:


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.




Message sent to bug-guix@HIDDEN:


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


--=-=-=--




Message sent to bug-guix@HIDDEN:


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.




Message sent to bug-guix@HIDDEN:


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.




Message received at control <at> debbugs.gnu.org:


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





Last modified: Mon, 25 Nov 2019 12:00:02 UTC

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997 nCipher Corporation Ltd, 1994-97 Ian Jackson.