fennel package not working with generic-html refresh updater

  • Done
  • quality assurance status badge
Details
2 participants
  • Léo Le Bouter
  • Ludovic Courtès
Owner
unassigned
Submitted by
Léo Le Bouter
Severity
normal
L
L
Léo Le Bouter wrote on 18 Mar 2021 10:05
(address . bug-guix@gnu.org)
0c7fa1ff58a9bb5b11fbcf1a5f11930974040547.camel@zaclys.net
After applying:

Toggle diff (25 lines)
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.
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEFIvLi9gL+xax3g6RRaix6GvNEKYFAmBTF+MACgkQRaix6GvN
EKYDXQ//fO0oQBSRBlR4XTCeTVqWsIOuFHt7cVuhtZJ7XZ52pUAE6OxBKDTgVpOS
cp9sNqR+3L/vqQebvGNIVPlm8kearzarB1maC1LtCbfsL0Sw4tEY5IwA/fYNtMS9
6kMCie0N4ak8ulqoQnJHH0WS8aP9Q0uyXTtyTvDWEcuBDhzwBSHXS3A4gj7dNzdi
UnavdK4Ntgpjo8VNqGymLTJT1zXTiwllA/U53kq6CH9HWAupYWqzQZ4u2FeNhkP/
rGMn+7ZkQmLf8ouMVWzXzDz9Ae9FpyP9hXiCCZPivinTrBUZgysw3b6x29PsbFom
a1XDnFfpTbXPckwdml3PiUPfxv+U49Rhx7UTSzwWjYuKX/FAA46oT/MEddLetWqD
u4IqKh+LZjXopgE771aaCS0daiBSQ5GeRxzLnVVVY5eO0xgvehndWGKt8JKnvuzV
kggZUTyVjjpJleYHMhE3qAY6Npm5lK+7eph7KqlC2J0vSnC/tvhCko0qQks6UHy0
Few84m7tbuzriCj26Pf67QZrl4CmJqFn68VnqFYNu4530wRdpppfgtMyysXMCXJb
Efh5jU4Ip6WgX7rOg9wdid4WMYYcmpga7H73ppAtceK+D+DVvHHzQfG8rupbgp9C
cPLBP0VtFSVDnyi8bFvPIc3i0SDHLLKQ3F5ONzIA8Ivwz9uMSSw=
=NYt5
-----END PGP SIGNATURE-----


L
L
Ludovic Courtès wrote on 18 Mar 2021 14:51
(name . Léo Le Bouter)(address . lle-bout@zaclys.net)(address . 47226@debbugs.gnu.org)
87zgz0o9in.fsf@gnu.org
Léo Le Bouter <lle-bout@zaclys.net> skribis:

Toggle quote (25 lines)
> 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’.
L
L
Ludovic Courtès wrote on 18 Mar 2021 21:42
control message for bug #47226
(address . control@debbugs.gnu.org)
87im5okxcu.fsf@gnu.org
tags 47226 notabug
close 47226
quit
?
Your comment

This issue is archived.

To comment on this conversation send an email to 47226@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 47226
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch