Bug#730293: how-can-i-help: should also look at RFH

[Christophe Siraut]

Hi,

On Sat, Nov 23, 2013 at 07:09:22PM +0100, Lucas Nussbaum wrote:
> how-can-i-help currnetly looks at RFA, O, ITA. It should also look at
> RFH.
>
> Changes are needed on both the UDD side and the client side.

Please review my patches for adding RFH to how-can-i-help. Notet his is
my first look at the UDD and I have not tested the whole chain.

Thanks,
Christophe

diff --git a/udd/orphaned_packages_gatherer.py b/udd/orphaned_packages_gatherer.py
index a476718..23d2ccc 100644
--- a/udd/orphaned_packages_gatherer.py
+++ b/udd/orphaned_packages_gatherer.py
@@ -1,13 +1,13 @@
 #!/usr/bin/env python
 
 """
-This script imports the list of orphaned, ITAed and RFAed packages into the DB
+This script imports the list of orphaned, ITA, RFA and RFH packages into the DB
 """
 
 import aux
 from gatherer import gatherer
 import re
-from psycopg2 import IntegrityError 
+from psycopg2 import IntegrityError
 
 def get_gatherer(connection, config, source):
   return orphaned_packages_gatherer(connection, config, source)
@@ -17,10 +17,10 @@ class orphaned_packages_gatherer(gatherer):
     gatherer.__init__(self, connection, config, source)
     self.assert_my_config('bugs-path', 'table', 'unarchived-table')
 
-  title_re = re.compile('^(ITA|RFA|O): ([^\s]*)([ ]+[-]+[ ]+(.*))?$')
+  title_re = re.compile('^(ITA|RFA|RFH|O): ([^\s]*)([ ]+[-]+[ ]+(.*))?$')
   otime_re = re.compile('^<!-- time:([0-9]+) ')
   chtitle_re = re.compile('^<strong>Changed Bug title to `O:.*$')
-  
+
   def get_time_orphaned(self, bug):
     bug = str(bug)
     dir = bug[4:6]
@@ -45,7 +45,7 @@ class orphaned_packages_gatherer(gatherer):
     #for the new data:
     cur = self.cursor()
     cur2 = self.cursor()
-    cur.execute("SELECT id, title, arrival FROM %s WHERE package = 'wnpp' AND status != 'done' AND title ~* '^(ITA|RFA|O):' AND id NOT IN (SELECT id from %s WHERE id > merged_with)" % (self.my_config['unarchived-table'], self.my_config['unarchived-table'] + '_merged_with'))
+    cur.execute("SELECT id, title, arrival FROM %s WHERE package = 'wnpp' AND status != 'done' AND title ~* '^(ITA|RFA|RFH|O):' AND id NOT IN (SELECT id from %s WHERE id > merged_with)" % (self.my_config['unarchived-table'], self.my_config['unarchived-table'] + '_merged_with'))
     rows = cur.fetchall()
 
     cur2.execute("DELETE FROM %s" % self.my_config['table'])

diff --git a/bin/how-can-i-help b/bin/how-can-i-help
index 452b8ec..295fc62 100755
--- a/bin/how-can-i-help
+++ b/bin/how-can-i-help
@@ -143,6 +143,7 @@ notesting = helpitems_filtered.select { |e| e['type'] == 'no-testing' }
 def wnpptype(t)
   return 'O (Orphaned)' if t == 'O'
   return 'RFA (Maintainer looking for adopter)' if t == 'RFA'
+  return 'RFH (Maintainer looking for help)' if t == 'RFH'
   return 'ITA (Someone working on adoption)' if t == 'ITA'
 end
 

signature.asc
Description: Digital signature