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

[Raphael Geissert]

Author: geissert
Date: 2009-09-27 00:36:57 +0000 (Sun, 27 Sep 2009)
New Revision: 1594

Modified:
   ddpo-by-mail/Makefile
   ddpo-by-mail/reminder-mail.rb
Log:
Know about unfixed security issues


Modified: ddpo-by-mail/Makefile
===================================================================
--- ddpo-by-mail/Makefile	2009-09-27 00:36:03 UTC (rev 1593)
+++ ddpo-by-mail/Makefile	2009-09-27 00:36:57 UTC (rev 1594)
@@ -1,6 +1,6 @@
 #!/usr/bin/make -f
 
-UPDATE_TARGETS:=ddpo_packages ddpo_maintainers testing-status.raw dehs.txt ood_excuses.txt pts_subscriptions.txt wnpp_rm lintian-list.txt debian-installer/all bugs_stable.txt bugs_unstable.txt
+UPDATE_TARGETS:=ddpo_packages ddpo_maintainers testing-status.raw dehs.txt ood_excuses.txt pts_subscriptions.txt wnpp_rm lintian-list.txt debian-installer/all bugs_stable.txt bugs_unstable.txt security-packages.txt
 ARCHITECTURES:=alpha,amd64,armel,hppa,hurd-i386,i386,ia64,mips,mipsel,powerpc,s390,sparc
 #ARCHITECTURES:=$(shell dpkg-architecture -L | tr "\n" ",")
 LOCAL_MIRROR:=/org/ftp.debian.org/ftp
@@ -50,6 +50,9 @@
 lintian-list.txt: $(FORCE)
 	$(WGET) http://qa.debian.org/data/$@
 
+security-packages.txt: $(FORCE)
+	$(WGET) http://alioth.debian.org/~geissert/$@
+
 debian-installer/binary-%: $(FORCE)
 	mkdir -p `dirname "$@"`
 	( cd `dirname "$@"` && $(WGET) $(MIRROR)/$(DI_PATH)/binary-$* ; ) || touch $@

Modified: ddpo-by-mail/reminder-mail.rb
===================================================================
--- ddpo-by-mail/reminder-mail.rb	2009-09-27 00:36:03 UTC (rev 1593)
+++ ddpo-by-mail/reminder-mail.rb	2009-09-27 00:36:57 UTC (rev 1594)
@@ -95,6 +95,13 @@
   lintian[pkg] = [errors.to_i, warnings.to_i]
 end
 
+# get security issues
+security = {}
+IO::read('data/security-packages.txt').each_line do |l|
+  pkg, issues = l.chomp.split(':')
+  security[pkg] = issues.to_i
+end
+
 bugs = parse_bugs
 testingissues = testing_issues(dipkgs)
 dehs = get_dehs
@@ -134,6 +141,10 @@
     pissues << [ :lintian, lintian[pkg][0], lintian[pkg][1] ]
     #pserious ||= lintian[pkg][0]
   end
+  if security[pkg]
+    pissues << [ :security, security[pkg] ]
+    pserious = true
+  end
   unless pissues.empty?
     issues[pkg] = pissues 
     serious[pkg] = pserious
@@ -153,6 +164,7 @@
 T_TESTING = "testing"
 T_LINTIAN = "lintian"
 T_DEHS = "dehs"
+T_SECURITY = "security"
 
 puts "Preparing mail for each maintainer..."
 # prepare mail for each maintainer
@@ -297,6 +309,16 @@
         end
         text[pkg][T_LINTIAN] += ", you should consider fixing them.\n"
         text[pkg][T_LINTIAN] += " See http://lintian.debian.org/maintainer/#{maintainer}.html##{pkg}\n"
+      elsif iss[0] == :security
+        text[pkg][T_INCLUDE] = true
+        text[pkg][T_SECURITY] = "= There "
+        if iss[1] > 1
+          text[pkg][T_SECURITY] += "are"
+        else
+          text[pkg][T_SECURITY] += "is"
+        end
+        text[pkg][T_SECURITY] += " #{iss[1]} unfixed security issue(s), please fix them.\n"
+        text[pkg][T_SECURITY] += " See http://security-tracker.debian.net/tracker/source-package/#{pkg}\n"
       else
         raise "Unknown issue #{iss[0]}!"
       end
@@ -306,6 +328,7 @@
     end
     if text[pkg][T_INCLUDE]
       gtext += text[pkg][T_HEADER]
+      gtext += text[pkg][T_SECURITY] if text[pkg][T_SECURITY]
       gtext += text[pkg][T_BUGS] if text[pkg][T_BUGS]
       gtext += text[pkg][T_BUILDS] if text[pkg][T_BUILDS]
       gtext += text[pkg][T_TESTING] if text[pkg][T_TESTING]