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

Bug#537930: marked as done (qa.debian.org: [PTS] Packages are listed as being incorrectly listed as NEW)

Your message dated Tue, 21 Jul 2009 21:14:51 +0000
with message-id <E1MTMfj-0001H2-Jm@alioth.debian.org>
and subject line qa.debian.org bug fixed in revision 2225
has caused the Debian Bug report #537930,
regarding qa.debian.org: [PTS] Packages are listed as being incorrectly listed as NEW
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.)


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

Currently, the PTS lists any package which it finds in new.822 as being
in NEW.  new.822 lists information for packages in both the NEW and
accepted (aka, incoming) queues.  Attached patch only uses info for
packages where the Queue field is 'new'.

-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (100, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.30-1-686 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Index: www/bin/other_to_xml.py
===================================================================
--- www/bin/other_to_xml.py	(revision 2224)
+++ www/bin/other_to_xml.py	(working copy)
@@ -116,12 +116,14 @@
 def read_NEW(fname):
     new_contents = {}
     for stanza in deb822.Sources.iter_paragraphs(file(os.path.join(dir, 'new.822'))):
-        if stanza.has_key('source') and stanza.has_key('version'):
-            # store only the most recent version in NEW
-            if not new_contents.has_key(stanza['source']) or \
-                    debian_support.version_compare( \
+        if stanza.has_key('source') and stanza.has_key('version') and \
+                stanza.has_key('queue'):
+            # store only the most recent version in NEW (ignore accepted)
+            if stanza['queue'] == 'new' and \
+                    (not new_contents.has_key(stanza['source']) or \
+                     debian_support.version_compare( \
                                     new_contents[stanza['source']],
-                                    stanza['version']) < 0:
+                                    stanza['version']) < 0):
                 new_contents[stanza['source']] = stanza['version']
     return new_contents

--- End Message ---
--- Begin Message --- 
Version: 2225

This bug was closed by Stefano Zacchiroli (zack) in SVN revision 2225.
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:

Do not list accepted packages as being in NEW (Closes: #537930)

Patch from James Vega.

--- End Message ---
Reply to: