Re: Bad HTML syntax

[Tiago Ilieve <tiago.myhro@gmail.com>]

Hi Nicolas,

On 18 May 2016 at 04:03, Nicolas Quiniou-Briand
<nicolas.quiniou-briand@ch-redon.fr> wrote:
> I find a bad HTML syntax on pages like this : http://debian.univ-lorraine.fr/debian-cd/current/amd64/iso-cd/.
>
> <a href="https://wiki.debian.org/Teams/DebianCd";>Debian CD team<a>  <<<<<<<<<
>
> Correct syntax should be :
>
> <a href="https://wiki.debian.org/Teams/DebianCd";>Debian CD team</a> <<<<<<<<<
>
> I use a HTML parser inside a script to download latest ISO. With this bad HTML syntax, my script doesn't work.

Nice catch! Those hyperlinks are defined at the bottom of
"jessie/HEADER.html.in"[1] and "stretch/HEADER.html.in"[2] files under
Debian CD's "setup.git" repository[3].

Not sure about how to send a Pull Request to it, so I'm attaching a
patch (0001-Fix-a-HTML-tag-closing-in-headers.patch). Hope this helps
Steve to fix it.

Regards,
Tiago.

[1]: https://anonscm.debian.org/git/debian-cd/setup.git/tree/jessie/HEADER.html.in?id=a513068#n232
[2]: https://anonscm.debian.org/git/debian-cd/setup.git/tree/stretch/HEADER.html.in?id=a513068#n225
[3]: https://anonscm.debian.org/git/debian-cd/setup.git/

-- 
Tiago "Myhro" Ilieve
Blog: https://blog.myhro.info/
GitHub: https://github.com/myhro
LinkedIn: https://br.linkedin.com/in/myhro
Montes Claros - MG, Brasil

From a1d896419c76ad3a5f80787ef16b8af7366aec83 Mon Sep 17 00:00:00 2001
From: Tiago Ilieve <tiago.myhro@gmail.com>
Date: Wed, 18 May 2016 04:42:07 -0300
Subject: [PATCH] Fix '<a>' HTML tag closing in headers
To: debian-cd@lists.debian.org

Spotted by Nicolas Quiniou-Briand[1].

[1]: https://lists.debian.org/debian-cd/2016/05/msg00056.html
---
 jessie/HEADER.html.in  | 2 +-
 stretch/HEADER.html.in | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/jessie/HEADER.html.in b/jessie/HEADER.html.in
index d9b028c..f5440fb 100644
--- a/jessie/HEADER.html.in
+++ b/jessie/HEADER.html.in
@@ -229,5 +229,5 @@ cpp.
 for lots more information about Debian CDs and installation.</p>
 
 <p>The images here were put together by
-the <a href="https://wiki.debian.org/Teams/DebianCd";>Debian CD team<a>
+the <a href="https://wiki.debian.org/Teams/DebianCd";>Debian CD team</a>
 , using debian-cd and other software.</p>
diff --git a/stretch/HEADER.html.in b/stretch/HEADER.html.in
index 1f4d581..39f9bac 100644
--- a/stretch/HEADER.html.in
+++ b/stretch/HEADER.html.in
@@ -222,5 +222,5 @@ cpp.
 for lots more information about Debian CDs and installation.</p>
 
 <p>The images here were put together by
-the <a href="https://wiki.debian.org/Teams/DebianCd";>Debian CD team<a>
+the <a href="https://wiki.debian.org/Teams/DebianCd";>Debian CD team</a>
 , using debian-cd and other software.</p>
-- 
2.1.4