GNU bug report logs - #59385
[PATCH] website: Link to the new packages browser

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 59385 in the body.
You can then email your comments to 59385 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: Luis Felipe <luis.felipe.la <at> protonmail.com>
To: Luis Felipe via Guix-patches <guix-patches <at> gnu.org>
Subject: [PATCH] website: Link to the new packages browser
Date: Sat, 19 Nov 2022 13:47:40 +0000

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

Hi,

This is a patch for the guix-artwork repository.


---
Luis Felipe López Acevedo
https://luis-felipe.gitlab.io/

[0001-website-Link-to-the-new-packages-browser.patch (text/x-patch, attachment)]

[publickey - luis.felipe.la@protonmail.com - 0x12DE1598.asc (application/pgp-keys, attachment)]

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

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Luis Felipe <luis.felipe.la <at> protonmail.com>
Cc: 59385 <at> debbugs.gnu.org
Subject: Re: bug#59385: [PATCH] website: Link to the new packages browser
Date: Mon, 21 Nov 2022 12:05:20 +0100

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

Hi,

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

> From ad0effaab60acdb7bd0e533bd544b49a4bee8a2b Mon Sep 17 00:00:00 2001
> From: Luis Felipe <luis.felipe.la <at> 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:

[Message part 2 (text/x-patch, inline)]

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

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

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.

Thanks!

Ludo’.

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

From: Luis Felipe <luis.felipe.la <at> protonmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 59385 <at> debbugs.gnu.org
Subject: Re: bug#59385: [PATCH] website: Link to the new packages browser
Date: Mon, 21 Nov 2022 16:38:22 +0000

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

Hi,

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

> Hi,
> 

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

> > From ad0effaab60acdb7bd0e533bd544b49a4bee8a2b Mon Sep 17 00:00:00 2001
> > From: Luis Felipe luis.felipe.la <at> 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 :)


> 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.


> Maybe we can add a redirect from https://guix.gnu.org/en/packages

That would be good.


> 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.

[0001-website-Remove-packages-app.patch (text/x-patch, attachment)]

[publickey - luis.felipe.la@protonmail.com - 0x12DE1598.asc (application/pgp-keys, attachment)]

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

Message #16 received at 59385-done <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Luis Felipe <luis.felipe.la <at> protonmail.com>
Cc: 59385-done <at> debbugs.gnu.org
Subject: Re: bug#59385: [PATCH] website: Link to the new packages browser
Date: Fri, 25 Nov 2022 18:09:33 +0100

Hola Luis,

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

> From 29723b3c711db2316e1fb66d252de769494b4a98 Mon Sep 17 00:00:00 2001
> From: Luis Felipe <luis.felipe.la <at> 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’.

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

From: Christopher Baines <mail <at> cbaines.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 59385 <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: bug#59385: [PATCH] website: Link to the new packages browser
Date: Fri, 25 Nov 2022 19:01:38 +0000

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

Ludovic Courtès <ludo <at> gnu.org> writes:

> Hola Luis,
>
> Luis Felipe <luis.felipe.la <at> protonmail.com> skribis:
>
>> From 29723b3c711db2316e1fb66d252de769494b4a98 Mon Sep 17 00:00:00 2001
>> From: Luis Felipe <luis.felipe.la <at> 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

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

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Luis Felipe <luis.felipe.la <at> protonmail.com>
Cc: 59385 <at> debbugs.gnu.org
Subject: Re: bug#59385: [PATCH] website: Link to the new packages browser
Date: Fri, 25 Nov 2022 22:49:43 +0100

Ludovic Courtès <ludo <at> gnu.org> skribis:

> 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’.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 59385 <at> debbugs.gnu.org
Subject: Re: bug#59385: [PATCH] website: Link to the new packages browser
Date: Sat, 26 Nov 2022 17:22:40 +0100

Hi,

Christopher Baines <mail <at> cbaines.net> skribis:

> Ludovic Courtès <ludo <at> gnu.org> writes:
>
>> Hola Luis,
>>
>> Luis Felipe <luis.felipe.la <at> protonmail.com> skribis:
>>
>>> From 29723b3c711db2316e1fb66d252de769494b4a98 Mon Sep 17 00:00:00 2001
>>> From: Luis Felipe <luis.felipe.la <at> 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’.

Previous Next

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