GNU bug report logs - #42615
[PATCH] guix: lint: Ignore unsupported source URL’s.

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Mathieu Othacehe <othacehe <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#42615: closed ([PATCH] guix: lint: Ignore unsupported source
 URL’s.)
Date: Sat, 01 Aug 2020 06:45:01 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sat, 01 Aug 2020 08:44:45 +0200
with message-id <87zh7eamqq.fsf <at> gnu.org>
and subject line Re: [bug#42615] [PATCH] guix: lint: Ignore unsupported source URL’s.
has caused the debbugs.gnu.org bug report #42615,
regarding [PATCH] guix: lint: Ignore unsupported source URL’s.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
42615: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=42615
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Lars-Dominik Braun <ldb <at> leibniz-psychology.org>
To: guix-patches <at> gnu.org
Subject: [PATCH] guix: lint: Ignore unsupported source URL’s.
Date: Thu, 30 Jul 2020 09:05:18 +0200

[Message part 3 (text/plain, inline)]

Hi,

in https://issues.guix.gnu.org/issue/42306 I noticed git:// URI’s in source
urls (see package nfs4-acl-tools) cause `guix lint` to fail currently. Attached
is a patch that fixes the issue, but I’m not sure that’s the “correct” way to
do it.

Cheers,
Lars

[0001-guix-lint-Ignore-unsupported-source-URL-s.patch (text/x-diff, attachment)]

[signature.asc (application/pgp-signature, inline)]

[Message part 6 (message/rfc822, inline)]

From: Mathieu Othacehe <othacehe <at> gnu.org>
To: Lars-Dominik Braun <ldb <at> leibniz-psychology.org>
Cc: 42615-done <at> debbugs.gnu.org
Subject: Re: [bug#42615] [PATCH] guix: lint: Ignore unsupported source
 URL’s.
Date: Sat, 01 Aug 2020 08:44:45 +0200

Hello,

> in https://issues.guix.gnu.org/issue/42306 I noticed git:// URI’s in source
> urls (see package nfs4-acl-tools) cause `guix lint` to fail currently. Attached
> is a patch that fixes the issue, but I’m not sure that’s the “correct” way to
> do it.

I considered adding a warning saying that URI validation failed, but
I think it can be misleading and let the user think that something is
wrong with the package definition.

Anyway looks fine, pushed!

Thanks,

Mathieu

Previous Next

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