[Collab-qa-commits] r687 - ddpo-by-mail

[lucas at alioth.debian.org]

Author: lucas
Date: 2008-01-31 13:08:25 +0000 (Thu, 31 Jan 2008)
New Revision: 687

Modified:
   ddpo-by-mail/ddpo-config.rb
   ddpo-by-mail/reminder-mail.rb
   ddpo-by-mail/update-data.rb
Log:
ddpomail: bugfixes

Modified: ddpo-by-mail/ddpo-config.rb
===================================================================
--- ddpo-by-mail/ddpo-config.rb	2008-01-31 12:29:55 UTC (rev 686)
+++ ddpo-by-mail/ddpo-config.rb	2008-01-31 13:08:25 UTC (rev 687)
@@ -49,7 +49,13 @@
 # (Date::today - Date::parse('2007-04-08')).to_i
 
 # Release Goals
-# FIXME add other release goals
+# FIXME missing tags: ipv6 i18n-debconf lfs nfs-v4 goal-enriched-chroot ddtp dep-init ftbfs-gcc
 RGTAGS = {
-  'goal-dash' => 'dash as /bin/sh'
+  'goal-dash' => 'switch /bin/sh to dash',
+  'qa:qa-doublebuild' => 'double compilation support',
+  'goal-recommends' => 'no unmet recommends',
+  'debmake' => 'drop debmake from Debian',
+  'utf8-control' => 'UTF-8 changelog and control',
+  'piuparts-stable-upgrade' => 'piuparts-clean archive',
+  'goal-python2.5' => 'python2.5 support'
 }

Modified: ddpo-by-mail/reminder-mail.rb
===================================================================
--- ddpo-by-mail/reminder-mail.rb	2008-01-31 12:29:55 UTC (rev 686)
+++ ddpo-by-mail/reminder-mail.rb	2008-01-31 13:08:25 UTC (rev 687)
@@ -6,7 +6,7 @@
 require 'update-data'
 
 # fetch updated files from www
-#update_data FIXME
+update_data
 
 # parse debian-installer packages
 pkg = nil
@@ -118,46 +118,46 @@
   mpkgs.keys.sort.each do |pkg|
     next if issues[pkg].nil?
     # skip if excluded
-    next if EXC_EMAILS_PKGS[email].include?(p)
-    if not mpkgs[p]
+    next if EXC_EMAILS_PKGS[email].include?(pkg)
+    if not mpkgs[pkg]
       comaint = ' (you co-maintain this package)'
-      only_uploader = false if haslist[p].nil?
+      only_uploader = false if haslist[pkg].nil? and serious[pkg]
     else
       comaint = ''
-      only_uploader = false
+      only_uploader = false if serious[pkg]
     end
+    mserious = mserious || serious[pkg]
     ptext = "=== #{pkg}:#{comaint}\n"
     testingpl = false
     issues[pkg].each do |iss|
       if iss[0] == :bugs
-        ptext += "= This package has #{iss[1].length} bug(s) that are important for the next release:\n"
+        ptext += "= This package has #{iss[1].length} bug(s) that shoud really be fixed for the next release:\n"
         iss[1].each do |b|
           ptext += "- ##{b[0]} <http://bugs.debian.org/#{b[0]}>\n"
           ptext += "  #{bugtitles[b[0]]}\n"
-          if b[1] == "RC"
+          if b[2] == "RC"
             ptext += "  (Release-Critical bug!)\n"
           else
-            ptext += "  Bug part of a release goal: #{b[1]}\n"
+            ptext += "  Bug part of a release goal: #{b[2]}\n"
           end
         end
       elsif iss[0] == :not_testing
-        ptext += "= This package has not been in testing for #{iss[1]} days.\n"
+        ptext += "= This package has not been in testing for #{iss[2]} days.\n"
         ptext += "  If things don't change, it won't be part of lenny!\n"
         testingpl = true
       elsif iss[0] == :no_migration
         ptext += "= This package has not been able to migrate from unstable\n"
-        ptext += "  to testing for #{iss[1]} days.\n"
+        ptext += "  to testing for #{iss[2]} days.\n"
         testingpl = true
       else
         raise "Unknown issue #{iss[0]}!"
       end
     end
     if testingpl
-      ptext += "  See <http://release.debian.org/migration/testing.pl?package=#{p}>\n"
+      ptext += "  See <http://release.debian.org/migration/testing.pl?package=#{pkg}>\n"
     end
     gtext += ptext
     gtext += "\n"
-    mserious = mserious || serious[pkg]
   end
   if not mserious
     puts "Skipping mail for #{email}, has only non-serious issues."

Modified: ddpo-by-mail/update-data.rb
===================================================================
--- ddpo-by-mail/update-data.rb	2008-01-31 12:29:55 UTC (rev 686)
+++ ddpo-by-mail/update-data.rb	2008-01-31 13:08:25 UTC (rev 687)
@@ -1,10 +1,8 @@
 require 'ddpo-config.rb'
 
-IGNFIXED="&ignotherfixed=on"
 def update_data
-  system("rm -f buglist ddpo_packages ddpo_maintainers popcon_by_inst")
+  system("rm -f bts2ldap_fullindex ddpo_packages ddpo_maintainers testing-status.raw dehs.txt")
   system("wget -q -O bts2ldap_fullindex http://qa.debian.org/data/bts2ldap/fullindex")
-  system("wget -q -O buglist 'http://bts.turmzimmer.net/details.php?bydist=sid&sortby=packages&ignnew=on#{IGNFIXED}&new=#{BUG_AGE}&refresh=1800'")
   system("wget -q -O ddpo_packages http://qa.debian.org/data/ddpo/results/ddpo_packages")
   system("wget -q -O ddpo_maintainers http://qa.debian.org/data/ddpo/results/ddpo_maintainers")
   system("wget -q -O testing-status.raw http://qa.debian.org/~lucas/testing-status.raw")