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
?