all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Luis Felipe via Guix-patches via <guix-patches@gnu.org>
To: "pelzflorian (Florian Pelz)" <pelzflorian@pelzflorian.de>
Cc: 49431@debbugs.gnu.org
Subject: [bug#49431] [PATCH maintenance.git] nginx: berlin: Redirect old video URLs for each language.
Date: Wed, 07 Jul 2021 17:51:54 +0000	[thread overview]
Message-ID: <hAneyXwK4xvLI1Ru5-hseApZGByUZzjVYxydzD-ogm2CaXhA-fJvfdOws3woIjcN5spXTAQZ3wJRFdu6w0IWMpScnZywQ1mW7pyAV8GQYeY=@protonmail.com> (raw)
In-Reply-To: <20210706065304.6kqbwqlcf6calkh7@pelzflorian.localdomain>

Hi Florian,


On Tuesday, July 6th, 2021 at 6:53 AM, pelzflorian (Florian Pelz) <pelzflorian@pelzflorian.de> wrote:

> This patch adds redirects for the guix website from the old video URLs
>
> like
>
> /LANG/videos/everyday-use-of-gnu-guix,-part-one/index.html
>
> /LANG/videos/everyday-use-of-gnu-guix,-part-one/
>
> /LANG/videos/everyday-use-of-gnu-guix,-part-one
>
> to the new
>
> /LANG/videos/2020/everyday-use-of-gnu-guix-part-one/
>
> but maybe the patch is too ugly because that’s code duplication to
>
> have three added redirections for each video. Note that my old
>
> videos page actually linked to
>
> </LANG/videos/everyday-use-of-gnu-guix,-part-one/index.html> and not
>
> </LANG/videos/everyday-use-of-gnu-guix,-part-one/> as usual.
>
> Shall I push it as-is to guix/maintenance.git,
>
> or do you think it would be better to rewrite the redirect procedure
>
> to return a list to redirect every URL with suffixes
>
> ""
>
> "/"
>
> "/index.html"
>
> and change guix.gnu.org-locations accordingly to flatten the location
>
> list?

I took a look at the patch, but I'm not familiar with the server-side of things, so there's not much I can say except that I trust your judgement on this :)

Thanks for taking care of it.




  reply	other threads:[~2021-07-07 17:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-06  6:53 [bug#49431] [PATCH maintenance.git] nginx: berlin: Redirect old video URLs for each language pelzflorian (Florian Pelz)
2021-07-07 17:51 ` Luis Felipe via Guix-patches via [this message]
2021-07-08 13:53 ` Ludovic Courtès
2021-07-09  7:15   ` pelzflorian (Florian Pelz)
2021-07-09 19:38     ` pelzflorian (Florian Pelz)
2021-07-10 13:41       ` Ludovic Courtès
2021-07-10 10:27     ` Ludovic Courtès
2021-07-10 19:28       ` bug#49431: " pelzflorian (Florian Pelz)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='hAneyXwK4xvLI1Ru5-hseApZGByUZzjVYxydzD-ogm2CaXhA-fJvfdOws3woIjcN5spXTAQZ3wJRFdu6w0IWMpScnZywQ1mW7pyAV8GQYeY=@protonmail.com' \
    --to=guix-patches@gnu.org \
    --cc=49431@debbugs.gnu.org \
    --cc=luis.felipe.la@protonmail.com \
    --cc=pelzflorian@pelzflorian.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.