[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#433894: marked as done (qa.debian.org: "Updating foo introduces new bugs:" links only to the first link)

Your message dated Wed, 22 Jul 2009 01:22:10 +0000
with message-id <E1MTQX4-0003EV-O6@alioth.debian.org>
and subject line qa.debian.org bug fixed in revision 2228
has caused the Debian Bug report #433894,
regarding qa.debian.org: "Updating foo introduces new bugs:" links only to the first link
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
433894: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=433894
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: qa.debian.org
Severity: minor
Tags: patch

When a package introduces new bugs to testing, the "Updating foo
introduces new bugs:" links only link to one bug of the whole lot.

An example is here:

http://packages.qa.debian.org/p/php4-ps.html

I've attached a quick and dirty and _untested_ patch that may work. I'm
not sure it is the right approach, making all lines with links in them
do something similar might be a better way to go so that all links in
the update_excuses.html file are preserved on the PTS page. If that is
wanted I can rework it.

I disclaim all copyright on this 11 line patch :)

-- 
bye,
pabs

http://wiki.debian.org/PaulWise

--- excuses_to_xml.py~	2007-07-20 07:09:44.000000000 +0000
+++ excuses_to_xml.py	2007-07-20 07:39:02.000000000 +0000
@@ -99,6 +99,17 @@
                 sub_elt = doc.createElement("item")
                 sub_elt.appendChild(doc.createTextNode(subline))
                 main_elt.appendChild(sub_elt) 
+            elif subline.find("introduces new bugs:") != -1:
+                subline = striphtml(subline)
+                subline = re.split('(#[0-9]+)', subline)
+                for item in subline:
+                    if len(item) and item[0] == '#':
+                        sub_elt = doc.createElement("item")
+                        sub_elt.setAttribute("url", "http://bugs.debian.org/"; + item[1:])
+	                sub_elt.appendChild(doc.createTextNode(item))
+                    else:
+                        sub_elt = doc.createTextNode(item)
+                    main_elt.appendChild(sub_elt) 
             else:
                 sub_elt = doc.createElement("item")
                 start = subline.find('href="http://')

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


--- End Message ---
--- Begin Message --- 
Version: 2228

This bug was closed by Stefano Zacchiroli (zack) in SVN revision 2228.
Note that it might take some time until the qa.debian.org code has
been updated and cronjobs have picked up changed data.

Commit message:

split BTS links for excuse "updating foo introduce new bugs"
(Closes: #433894)

--- End Message ---
Reply to: