From e86e9c4f234da49334a16ea6a93da0f1d519a688 Mon Sep 17 00:00:00 2001
From: Pierre Labastie <pierre.labastie@neuf.fr>
Date: Wed, 15 Jul 2026 23:45:46 +0200
Subject: [PATCH] Make wget-list.xsl and md5sums.xsl more robust

Those stylesheets selected download urls by seeking some pattern
in their content. Those patterns may need to be adjusted if a new
type of file is downloaded.

We prefer to seek urls inside <para> containing "Download",
which is the norm in chapter 3. The condition is also that those
<para> are in a <listitem>.
---
 stylesheets/md5sum.xsl    | 8 ++------
 stylesheets/wget-list.xsl | 8 ++------
 2 files changed, 4 insertions(+), 12 deletions(-)

diff --git a/stylesheets/md5sum.xsl b/stylesheets/md5sum.xsl
index 1e428262b..4f2e8e22f 100644
--- a/stylesheets/md5sum.xsl
+++ b/stylesheets/md5sum.xsl
@@ -8,18 +8,14 @@
   <xsl:output method="text"/>
 
   <xsl:template match="/">
-    <xsl:apply-templates select="//ulink"/>
+    <xsl:apply-templates select="//listitem/para[contains(string(),'Download')]/ulink"/>
   </xsl:template>
 
   <xsl:template match="ulink">
       <!-- If some package don't have the predefined strings in their
       name, the next test must be fixed to match it also. Skip possible
       duplicated URLs that may be split for PDF output -->
-    <xsl:if test="( (contains( @url, '.tar.' ) and not( ancestor::note ) ) or
-                    contains( @url, '.zip'   )                             or
-                    contains( @url, '.patch' ) 
-                  ) and 
-                    not( ancestor-or-self::*/@condition = 'pdf' )" >
+    <xsl:if test="not( ancestor-or-self::*/@condition = 'pdf' )" >
       <!-- Get the md5sum -->
       <xsl:value-of select="../../para/literal"/>
 
diff --git a/stylesheets/wget-list.xsl b/stylesheets/wget-list.xsl
index f9f17b08b..272b66df1 100644
--- a/stylesheets/wget-list.xsl
+++ b/stylesheets/wget-list.xsl
@@ -9,18 +9,14 @@
   <xsl:output method="text"/>
 
   <xsl:template match="/">
-    <xsl:apply-templates select="//ulink"/>
+    <xsl:apply-templates select="//listitem/para[contains(string(),'Download')]/ulink"/>
   </xsl:template>
 
   <xsl:template match="ulink">
       <!-- If some package don't have the predefined strings in their
       name, the next test must be fixed to match it also. Skip possible
       duplicated URLs due that may be split for PDF output -->
-    <xsl:if test="( (contains(@url, '.tar.') and not( ancestor::note ) ) or
-                    contains(@url, '.zip'  )                             or
-                    contains(@url, '.patch')
-                  ) and
-                    not(ancestor-or-self::*/@condition = 'pdf')">
+    <xsl:if test="not(ancestor-or-self::*/@condition = 'pdf')">
       <xsl:choose>
         <xsl:when test="contains(@url,'?download')">
           <xsl:value-of select="substring-before(@url,'?download')"/>