New submission from Georg Brandl <[email protected]>:

What do others think?  Is this feasible?

(Rough patch attached; untested.)

----------
files: autonosy_rm.diff
messages: 1872
nosy: gbrandl
priority: wish
status: unread
title: auto-nosy release manager(s) on release blockers

_______________________________________________________
PSF Meta Tracker <[email protected]>
<http://psf.upfronthosting.co.za/roundup/meta/issue363>
_______________________________________________________
Index: python-dev/detectors/autonosy.py
===================================================================
--- python-dev/detectors/autonosy.py    (Revision 87094)
+++ python-dev/detectors/autonosy.py    (Arbeitskopie)
@@ -4,14 +4,16 @@
 # Python 2.3 ... 2.6 compatibility:
 from roundup.anypy.sets_ import set
 
+RELEASE_MANAGERS = {
+    'Python 2.6': 'barry',
+    'Python 2.7': 'benjamin.peterson',
+    'Python 3.1': 'benjamin.peterson',
+    'Python 3.2': 'georg.brandl',
+}
+
 def autonosy(db, cl, nodeid, newvalues):
+    components = newvalues.get('components', [])
 
-    if 'components' not in newvalues:
-        # Without components, nobody needs to be added as nosy
-        return
-    else:
-        components = newvalues['components']
-
     nosy = set()
     if 'nosy' in newvalues:
         new_nosy = newvalues.get('nosy', [])
@@ -26,6 +28,13 @@
         users = db.component.get(component, 'add_as_nosy')
         nosy |= set(users)
 
+    if 'priority' in newvalues:
+        if db.priority.get(newvalues['priority'], 'name') == 'release blocker':
+            for version in newvalues.get('versions', []):
+                name = db.version.get(version, 'name')
+                if name in RELEASE_MANAGERS:
+                    nosy.add(RELEASE_MANAGERS[name])
+
     newvalues['nosy'] = list(nosy)
 
 
_______________________________________________
Tracker-discuss mailing list
[email protected]
http://mail.python.org/mailman/listinfo/tracker-discuss

Reply via email to