[PATCH] website: Link to the new packages browser

  • Done
  • quality assurance status badge
Details
3 participants
  • Ludovic Courtès
  • Luis Felipe
  • Christopher Baines
Owner
unassigned
Submitted by
Luis Felipe
Severity
normal
L
L
Luis Felipe wrote on 19 Nov 2022 14:47
(name . Luis Felipe via Guix-patches)(address . guix-patches@gnu.org)
zXAO_O-FJjvCko_ocjv5hT50oWHjDwH43RxS1X8AIPRO9Cx_3AsiBRCD29YO_4efJAJKJwGEba3uVos74d69mCx_6oJ6yVlekvKzNyaWDHk=@protonmail.com
Hi,

This is a patch for the guix-artwork repository.


---
Luis Felipe López Acevedo
Attachment: signature.asc
L
L
Ludovic Courtès wrote on 21 Nov 2022 12:05
(name . Luis Felipe)(address . luis.felipe.la@protonmail.com)(address . 59385@debbugs.gnu.org)
87pmdglg9b.fsf@gnu.org
Hi,

Luis Felipe <luis.felipe.la@protonmail.com> skribis:

Toggle quote (15 lines)
> From ad0effaab60acdb7bd0e533bd544b49a4bee8a2b Mon Sep 17 00:00:00 2001
> From: Luis Felipe <luis.felipe.la@protonmail.com>
> Date: Sat, 19 Nov 2022 08:21:04 -0500
> Subject: [PATCH] website: Link to the new packages browser.
>
> Update all package-related links to point to the newly deployed browser
> at https://packages.guix.gnu.org/.
>
> * website/apps/base/utils.scm (packages-url): New procedure.
> * website/apps/base/data.scm (contact-media): Use packages-url.
> * website/apps/base/templates/components.scm (navbar): Likewise.
> * website/apps/base/templates/contribute.scm (contribute-t): Likewise.
> * website/apps/base/templates/donate.scm (donate-t): Likewise.
> * website/apps/base/templates/home.scm (home-t): Likewise.

Awesome, pushed!

While we’re at it, can we also disable individual package page
generation? It takes a lot of time to build and uses quite a lot of
space as well; I’m eager to see them go. :-)

Maybe we can add a redirect from https://guix.gnu.org/en/packages
pages like this:
Toggle diff (18 lines)
diff --git a/hydra/modules/sysadmin/nginx.scm b/hydra/modules/sysadmin/nginx.scm
index 3c3456d..0a8b1a8 100644
--- a/hydra/modules/sysadmin/nginx.scm
+++ b/hydra/modules/sysadmin/nginx.scm
@@ -75,6 +75,13 @@ synonymous IETF language tags that should be mapped to the same $lang."
(define guix.gnu.org-redirect-locations
(list
+ (nginx-location-configuration
+ (uri "~ /packages/.*")
+ (body (list "return 301 https://packages.guix.gnu.org")))
+ (nginx-location-configuration
+ (uri "~ /([^/]+)/packages/.*")
+ (body (list "return 301 https://packages.guix.gnu.org")))
+
;; Short URL for the installation script
(redirect "/install.sh" "https://git.savannah.gnu.org/cgit/guix.git/plain/etc/guix-install.sh")
;; What follows is a list of redirects for URLs that used to be
If we’re serious about it, we can try to match the package/version in
and redirect to the right one. Not sure if that’s worth the trouble
though.

Thanks!

Ludo’.
L
L
Luis Felipe wrote on 21 Nov 2022 17:38
(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 59385@debbugs.gnu.org)
xq3lUDAYtRq0i4OWggpnfVz0qyr6hHo8A7U5zEmmDICBwYSApAOnPI73QsZATMcEXJkMuodHoarUx8ng8rTGpTlZhgQxSiJxb4-6WhbAAnc=@protonmail.com
Hi,

On Monday, November 21st, 2022 at 11:05, Ludovic Courtès <ludo@gnu.org> wrote:

Toggle quote (28 lines)
> Hi,
>

> Luis Felipe luis.felipe.la@protonmail.com skribis:
>

> > From ad0effaab60acdb7bd0e533bd544b49a4bee8a2b Mon Sep 17 00:00:00 2001
> > From: Luis Felipe luis.felipe.la@protonmail.com
> > Date: Sat, 19 Nov 2022 08:21:04 -0500
> > Subject: [PATCH] website: Link to the new packages browser.
> >

> > Update all package-related links to point to the newly deployed browser
> > at https://packages.guix.gnu.org/.
> >

> > * website/apps/base/utils.scm (packages-url): New procedure.
> > * website/apps/base/data.scm (contact-media): Use packages-url.
> > * website/apps/base/templates/components.scm (navbar): Likewise.
> > * website/apps/base/templates/contribute.scm (contribute-t): Likewise.
> > * website/apps/base/templates/donate.scm (donate-t): Likewise.
> > * website/apps/base/templates/home.scm (home-t): Likewise.
>

>

> Awesome, pushed!

Thanks :)


Toggle quote (4 lines)
> While we’re at it, can we also disable individual package page
> generation? It takes a lot of time to build and uses quite a lot of
> space as well; I’m eager to see them go. :-)

I'm attaching a patch that removes the whole "(apps packages)" module.


Toggle quote (2 lines)
> Maybe we can add a redirect from https://guix.gnu.org/en/packages

That would be good.


Toggle quote (5 lines)
> If we’re serious about it, we can try to match the package/version in
> URLs like https://guix.gnu.org/en/packages/julia-mappedarrays-0.4.0/
> and redirect to the right one. Not sure if that’s worth the trouble
> though.

I'd say it is not necessary since people were avoiding linking to them because of the version. They were already considered ephemeral.
Attachment: signature.asc
L
L
Ludovic Courtès wrote on 25 Nov 2022 18:09
(name . Luis Felipe)(address . luis.felipe.la@protonmail.com)(address . 59385-done@debbugs.gnu.org)
87fse7arle.fsf_-_@gnu.org
Hola Luis,

Luis Felipe <luis.felipe.la@protonmail.com> skribis:

Toggle quote (13 lines)
> From 29723b3c711db2316e1fb66d252de769494b4a98 Mon Sep 17 00:00:00 2001
> From: Luis Felipe <luis.felipe.la@protonmail.com>
> Date: Mon, 21 Nov 2022 11:06:03 -0500
> Subject: [PATCH] website: Remove packages app.
>
> There is a new package browser at https://packages.guix.gnu.org/. It
> is not necessary to generate a static package catalog anymore.
>
> * website/apps/packages: Remove it.
> * website/tests/apps/packages: Likewise.
> * website/haunt.scm (site): Remove packages builder.
> * website/tests/all.scm: Don't import packages tests.

I realized we still need to build /sources.json and /packages.json, so I
kept these and removed all the rest (commit
412ac3958bf288c56fded35d0a632c530e57ef8f).

Maybe we’ll eventually build them on Cuirass or something.

Anyway ‘haunt build’ is soooo fast now! :-)

Thanks,
Ludo’.
Closed
C
C
Christopher Baines wrote on 25 Nov 2022 20:01
(name . Ludovic Courtès)(address . ludo@gnu.org)
8735a6zwce.fsf@cbaines.net
Ludovic Courtès <ludo@gnu.org> writes:

Toggle quote (23 lines)
> Hola Luis,
>
> Luis Felipe <luis.felipe.la@protonmail.com> skribis:
>
>> From 29723b3c711db2316e1fb66d252de769494b4a98 Mon Sep 17 00:00:00 2001
>> From: Luis Felipe <luis.felipe.la@protonmail.com>
>> Date: Mon, 21 Nov 2022 11:06:03 -0500
>> Subject: [PATCH] website: Remove packages app.
>>
>> There is a new package browser at https://packages.guix.gnu.org/. It
>> is not necessary to generate a static package catalog anymore.
>>
>> * website/apps/packages: Remove it.
>> * website/tests/apps/packages: Likewise.
>> * website/haunt.scm (site): Remove packages builder.
>> * website/tests/all.scm: Don't import packages tests.
>
> I realized we still need to build /sources.json and /packages.json, so I
> kept these and removed all the rest (commit
> 412ac3958bf288c56fded35d0a632c530e57ef8f).
>
> Maybe we’ll eventually build them on Cuirass or something.

Back before /packages.json was around, I was thinking the Guix Data
Service could provide this information. I think that might require
storing a bit more information on package sources, but it should be
quite possible (to serve both the sources.json and packages.json files).

Chris
-----BEGIN PGP SIGNATURE-----

iQKlBAEBCgCPFiEEPonu50WOcg2XVOCyXiijOwuE9XcFAmOBEoFfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDNF
ODlFRUU3NDU4RTcyMEQ5NzU0RTBCMjVFMjhBMzNCMEI4NEY1NzcRHG1haWxAY2Jh
aW5lcy5uZXQACgkQXiijOwuE9XeJmg//Z53o7QuxRCLfivH3LqUhGBuy3OvP1r8O
h9p0mm0enLSAKbdB+YPN2b9HwvmMMd5Ykdip7OW5SWQH/K+gF9FxC4HlVsojqhcP
EyNsbiOyMGxqoAVs/PdFeW5luWkJXJubZp0/9V4XpeaZ4yTUfpqQYjl4dNAPbc9m
Wh0F8QUH0XD8UQqmBbxVmvWypOnnFdLhK+eLJzQLakgYK4HUjRcMokyGjM8yMttK
slYAGuvZBj0MftoQzUkt8sMXE7PhTKaz4BvFXd3PobzM6z+jVTJG89YILdMwfI21
Y+s50DPqpDQgiRnplz+R+mXjs3350dA4UIQ/YDpPfc/wf2x4vpSp9veVYqH+TasG
7qyjU4kRTiWkp7O+rITRUPs8DoOHGTESUu+05l9rT0D9Mp+auP+xccNbOY8/FxiO
bVlU9nJ6a8/v0EU2wD8eqF716jPv4I0lUzZ0tPpyUx4rV9GHryVUX19ww493k2uu
+YfIcuaaoZ8gtEWKnfzs1JJ5Ey391YV1DUZS4vedC/zcAZab2RCZqgBjruC1EY9s
hcvHK+PX5lOWsdOOgKAW8rlq7sQbzeb5rsbwbGT3IgRuRh1rDMAwP2+K/E1HB68B
QdS3UH43sLXX0T6y6kUffz0PDUxZJE7C34+X2RyuL/bPKB8V0qGXMlWSeK+J/1VR
R2Mb5W5Yo90=
=nVWz
-----END PGP SIGNATURE-----

L
L
Ludovic Courtès wrote on 25 Nov 2022 22:49
(name . Luis Felipe)(address . luis.felipe.la@protonmail.com)(address . 59385@debbugs.gnu.org)
87bkoubt6w.fsf_-_@gnu.org
Ludovic Courtès <ludo@gnu.org> skribis:

Toggle quote (2 lines)
> Maybe we can add a redirect from https://guix.gnu.org/en/packages

Done in maintenance commit e97f78bd61fd4e0c54607861360b18c2a5681131, now
deployed on berlin and bayfront (the web site is currently on the
latter).

Ludo’.
L
L
Ludovic Courtès wrote on 26 Nov 2022 17:22
(name . Christopher Baines)(address . mail@cbaines.net)(address . 59385@debbugs.gnu.org)
87zgcd65yn.fsf@chbouib.org
Hi,

Christopher Baines <mail@cbaines.net> skribis:

Toggle quote (30 lines)
> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Hola Luis,
>>
>> Luis Felipe <luis.felipe.la@protonmail.com> skribis:
>>
>>> From 29723b3c711db2316e1fb66d252de769494b4a98 Mon Sep 17 00:00:00 2001
>>> From: Luis Felipe <luis.felipe.la@protonmail.com>
>>> Date: Mon, 21 Nov 2022 11:06:03 -0500
>>> Subject: [PATCH] website: Remove packages app.
>>>
>>> There is a new package browser at https://packages.guix.gnu.org/. It
>>> is not necessary to generate a static package catalog anymore.
>>>
>>> * website/apps/packages: Remove it.
>>> * website/tests/apps/packages: Likewise.
>>> * website/haunt.scm (site): Remove packages builder.
>>> * website/tests/all.scm: Don't import packages tests.
>>
>> I realized we still need to build /sources.json and /packages.json, so I
>> kept these and removed all the rest (commit
>> 412ac3958bf288c56fded35d0a632c530e57ef8f).
>>
>> Maybe we’ll eventually build them on Cuirass or something.
>
> Back before /packages.json was around, I was thinking the Guix Data
> Service could provide this information. I think that might require
> storing a bit more information on package sources, but it should be
> quite possible (to serve both the sources.json and packages.json files).

Yes, it’d be more logical to produce it on the Data Service or similar.
Lemme know if you work on this!

Ludo’.
?
Your comment

This issue is archived.

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

To respond to this issue using the mumi CLI, first switch to it
mumi current 59385
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