Bug#831435: www.debian.org: Link to the patch tracker broken

[Boyuan Yang]

在 2018年1月16日星期二 CST 下午4:38:10，Rhonda D'Vine 写道：
> Hi,
> 
>  I currently can't deploy it but will keep it in mind for later.

That would be great.

Besides this sole problem, there are also many important (and one RC-level) 
bugs for packages.debian.org. I'm wondering if you could take a look at the 
running status of the website and investigate into them later.

> 
>  Though:
> 
> * Orestis Ioannou  [2016-07-16 00:36:03 CEST]:
> > The URL for the patch tracker changed slightly hence the links towards
> > sources.debian.net/patches are broken.
> > I.e when you are at https://packages.debian.org/stretch/python-pip
> > the link to the Debian Patch Tracker is
> > http://sources.debian.net/patches/summary/python-pip/8.1.2-2/
> > but the correct one is
> > http://sources.debian.net/patches/python-pip/8.1.2-2/
>  If such URL schemes are changed they break not only the packages site
> but all external links pointing to it.  It is good to get the links in
> the sites linking to it corrected - but it would be also very useful to
> add a redirect that handles that; from what I can see only the /summary/
> part of the URL got removed?  That should be doable in a single redirect
> line and unbreak all the external links, not only the packages website.
> 
>  I'm not questioning the need for the need in the change, there for sure
> is good reasons behind that, but please add defensive redirects for such
> cases. :)  Not sure if you are involved with the patch tracker, I
> dropped the note also in the #debian-qa IRC channel, from what I can
> tell the patch tracker is in the field of the QA team.

The patch tracker, as part of sources.debian.org (not sources.debian.net), is 
not handled by debian-www team. It is under QA Team so I'm CC-ing this mail to
debian-qa maillist too.

Regards,
Boyuan Yang

> 
>  Enjoy,
> Rhonda


signature.asc
Description: This is a digitally signed message part.

Bug#831435: www.debian.org: Link to the patch tracker broken

[Rhonda D'Vine]

Hi,

 I currently can't deploy it but will keep it in mind for later.

 Though:

* Orestis Ioannou  [2016-07-16 00:36:03 CEST]:
> The URL for the patch tracker changed slightly hence the links towards
> sources.debian.net/patches are broken.
> I.e when you are at https://packages.debian.org/stretch/python-pip
> the link to the Debian Patch Tracker is
> http://sources.debian.net/patches/summary/python-pip/8.1.2-2/
> but the correct one is http://sources.debian.net/patches/python-pip/8.1.2-2/

 If such URL schemes are changed they break not only the packages site
but all external links pointing to it.  It is good to get the links in
the sites linking to it corrected - but it would be also very useful to
add a redirect that handles that; from what I can see only the /summary/
part of the URL got removed?  That should be doable in a single redirect
line and unbreak all the external links, not only the packages website.

 I'm not questioning the need for the need in the change, there for sure
is good reasons behind that, but please add defensive redirects for such
cases. :)  Not sure if you are involved with the patch tracker, I
dropped the note also in the #debian-qa IRC channel, from what I can
tell the patch tracker is in the field of the QA team.

 Enjoy,
Rhonda
-- 
Fühlst du dich mutlos, fass endlich Mut, los  |
Fühlst du dich hilflos, geh raus und hilf, los| Wir sind Helden
Fühlst du dich machtlos, geh raus und mach, los   | 23.55: Alles auf Anfang
Fühlst du dich haltlos, such Halt und lass los|

Bug#831435: www.debian.org: Link to the patch tracker broken - will fix it

[Tim Landscheidt]

The source code is in a Git repository at
https://anonscm.debian.org/git/webwml/packages.git
(https://packages.debian.org/stretch/python-pip, "Learn more
about this site." => https://packages.debian.org/about/).
The patch is already committed
(5a638ca4744b3bb68db35a92b70c568223809afd), but it has not
been deployed yet?  Probably only the Debian Website Team
can deploy it (or confirm that is has been deployed, but it
is faulty).

Bug#831435: www.debian.org: Link to the patch tracker broken - will fix it

[Mechtilde Stehmann]

Hello

I want to fix it. Can someone give me a hint how to do it? Where can I
find the "source code" of that website and so on?

Where can I find a documentation or a howto?

Kind regards


-- 
Mechtilde Stehmann
## Debian Developer
## Loook, calender-exchange-provider, libreoffice-canzeley-client
## PGP encryption welcome
## F0E3 7F3D C87A 4998 2899  39E7 F287 7BBA 141A AD7F



signature.asc
Description: OpenPGP digital signature

Bug#831435: www.debian.org: Link to the patch tracker broken

[Tim Landscheidt]

AFAICT:

| diff --git a/templates/config.tmpl b/templates/config.tmpl
| index 3f64edf..f1f7c84 100644
| --- a/templates/config.tmpl
| +++ b/templates/config.tmpl
| @@ -22,7 +22,7 @@
| changelogs_url = 'http://ftp-master.metadata.debian.org/changelogs/'
| policy_url = 'https://www.debian.org/doc/debian-policy/'
| cn_help_url = project_homepage _ 'intro/cn'
| -   patch_tracking_url = 'http://sources.debian.net/patches/summary'
| +   patch_tracking_url = 'http://sources.debian.net/patches'
| screenshots_url = 'https://screenshots.debian.net/package/'
| screenshots_thumb_url = 
'https://screenshots.debian.net/thumbnail-with-version/'
| logo = {

should fix it.

Tim

Bug#831435: www.debian.org: Link to the patch tracker broken

[Orestis Ioannou]

Package: www.debian.org
Severity: normal
User: www.debian@packages.debian.org
Usertag: packages

Heya

The URL for the patch tracker changed slightly hence the links towards
sources.debian.net/patches are broken.
I.e when you are at https://packages.debian.org/stretch/python-pip
the link to the Debian Patch Tracker is
http://sources.debian.net/patches/summary/python-pip/8.1.2-2/
but the correct one is http://sources.debian.net/patches/python-pip/8.1.2-2/

I should probably be able to propose a patch for that but i ve been
thinking to
do this for a week so somebody might beat me at it :)

Cheers,

Orestis



-- System Information:
Debian Release: stretch/sid
  APT prefers testing
  APT policy: (750, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.5.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)