diff mbox series

[bug#42306] gnu: Add nfs4-acl-tools

Message ID 20200727071139.GB3225@zpidnp36
State Accepted
Headers show
Series [bug#42306] gnu: Add nfs4-acl-tools | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch fail View Laminar job

Commit Message

Lars-Dominik Braun July 27, 2020, 7:11 a.m. UTC 
Hey Marius,

> Sorry for the late reply...
no problem.

> The patch mostly LGTM, but no longer applies.  Can you rebase it on
> current master?  Meanwhile, a few comments:
Sure, see attached patch, which (hopefully) also improves the description.

I noticed linting fails with a pattern matching error, because git:// protocol
URI’s are not supported. The second patch fixes it, but I’m not sure that’s a
“proper” solution.

Cheers,
Lars
diff mbox series

Patch

From 5c9dddd0d4b54ca73f4fe4ebac37789bde0e54ce Mon Sep 17 00:00:00 2001
From: Lars-Dominik Braun <ldb@leibniz-psychology.org>
Date: Mon, 27 Jul 2020 09:09:24 +0200
Subject: [PATCH 2/2] =?UTF-8?q?guix:=20lint:=20Ignore=20unsupported=20sour?=
 =?UTF-8?q?ce=20URL=E2=80=99s.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* guix/lint.scm (check-source): Add match case for #f.
---
 guix/lint.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/guix/lint.scm b/guix/lint.scm
index 8a55f3e744..395630aa1c 100644
--- a/guix/lint.scm
+++ b/guix/lint.scm
@@ -794,6 +794,9 @@  descriptions maintained upstream."
            (#t
             ;; We found a working URL, so stop right away.
             '())
+           (#f
+            ;; Unsupported URL or other error, skip.
+            (loop rest '()))
            ((? lint-warning? warning)
             (loop rest (cons warning warnings))))))))
 
-- 
2.20.1