--- Begin Message ---
Subject: |
[PATCH 0/1] gnu: Add atftp. |
Date: |
Sun, 12 Mar 2023 21:41:38 -0700 |
Hi,
I am not sure whether the mirror:// URL is right. It works, but there are two
lint warnings:
gnu/packages/networking.scm:2946:15: atftp@0.8.0: URI
https://www.gnu.org/software/atftp/ not reachable: 404 ("Not Found")
gnu/packages/networking.scm:2921:12: atftp@0.8.0: source not archived on
Software Heritage and missing from the Disarchive database
Kind regards,
Felix Lechner
* * *
Felix Lechner (1):
gnu: Add atftp.
gnu/packages/networking.scm | 41 +++++++++++++++++++++++++++++++++++++
1 file changed, 41 insertions(+)
base-commit: 9584b1daa687d1212550841d1c5f62e7ffa44ffc
--
2.39.1
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#62156: [PATCH 0/1] gnu: Add atftp. |
Date: |
Mon, 27 Mar 2023 18:49:19 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi,
Felix Lechner <felix.lechner@lease-up.com> skribis:
> * gnu/packages/networking.scm (atftp): New variable.
>
> Thanks to Josselin Poiret and Bruno Victal for their generous and detailed
> reviews of this item!
I tweaked the description (to start with a full sentence) and passed it
through ‘guix style’.
> For the atftp package added in the preceeding commit, lint produced this
> warning:
>
> gnu/packages/networking.scm:2924:5: atftp@0.8.0:
> URI https://git.code.sf.net/p/atftp/code not reachable:
> 404 ("Not Found")
>
> Thanks to Sergey Trofimov for suggesting a fix! [1]
>
> [1] https://issues.guix.gnu.org/62156#3
>
> It was implemented here, and the warning disappeared.
>
> * guix/lint.scm (check-source): Append "/info/refs" to git-reference-url.
>
> Co-authored-by: Sergey Trofimov <sarg@sarg.org.ru>
Nice. I adjusted ‘tests/lint.scm’ accordingly.
Applied both; thank you, and thanks to the reviewers!
Ludo’.
--- End Message ---