GNU bug report logs - #47226
fennel package not working with generic-html refresh updater

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 47226 in the body.
You can then email your comments to 47226 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Léo Le Bouter <lle-bout <at> zaclys.net>
To: bug-guix <at> gnu.org
Subject: fennel package not working with generic-html refresh updater
Date: Thu, 18 Mar 2021 10:05:39 +0100

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

After applying:

diff --git a/gnu/packages/lua.scm b/gnu/packages/lua.scm
index edb3f85109..36fd1eb066 100644
--- a/gnu/packages/lua.scm
+++ b/gnu/packages/lua.scm
@@ -1175,6 +1175,8 @@ enabled.")
               (snippet
                '(begin
                   (delete-file "fennelview.lua") #t))))
+    (properties
+     '((release-monitoring-url . "https://fennel-lang.org/downloads/")
))
     (build-system gnu-build-system)
     (arguments
      '(#:make-flags (list (string-append "PREFIX=" (assoc-ref %outputs
"out")))

$ ./pre-inst-env guix refresh fennel
gnu/packages/lua.scm:1162:2: warning: no updater for fennel

It should otherwise clearly work since 
https://fennel-lang.org/downloads/ is a classic directory listing like
the generic-html updater should support.

I think it may be because this package uses a git-reference rather than
url-fetch.

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

Message #8 received at 47226 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Léo Le Bouter <lle-bout <at> zaclys.net>
Cc: 47226 <at> debbugs.gnu.org
Subject: Re: bug#47226: fennel package not working with generic-html refresh
 updater
Date: Thu, 18 Mar 2021 14:51:12 +0100

Léo Le Bouter <lle-bout <at> zaclys.net> skribis:

> After applying:
>
> diff --git a/gnu/packages/lua.scm b/gnu/packages/lua.scm
> index edb3f85109..36fd1eb066 100644
> --- a/gnu/packages/lua.scm
> +++ b/gnu/packages/lua.scm
> @@ -1175,6 +1175,8 @@ enabled.")
>                (snippet
>                 '(begin
>                    (delete-file "fennelview.lua") #t))))
> +    (properties
> +     '((release-monitoring-url . "https://fennel-lang.org/downloads/")
> ))
>      (build-system gnu-build-system)
>      (arguments
>       '(#:make-flags (list (string-append "PREFIX=" (assoc-ref %outputs
> "out")))
>
> $ ./pre-inst-env guix refresh fennel
> gnu/packages/lua.scm:1162:2: warning: no updater for fennel
>
> It should otherwise clearly work since 
> https://fennel-lang.org/downloads/ is a classic directory listing like
> the generic-html updater should support.

‘generic-html’ is for packages that use ‘url-fetch’ and a tarball
fetched over HTTP(S).  It does not apply to Fennel.

Here we should use the relevant sr.ht HTTP API, similar to what (guix
import github) does, or possibly a generic ‘git-tag’ updater as you
proposed.

Ludo’.

Previous Next

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