Skip to content
Permalink
Browse files

Operate on all open FTBFS bugs we find, not just the first one.

  • Loading branch information...
ralphbean committed Sep 22, 2015
1 parent 4648ed3 commit a64d7d338954f288b9e1d7bd27f0ade80810d31b
Showing with 5 additions and 4 deletions.
  1. +2 −2 hotness/bz.py
  2. +3 −2 hotness/consumers.py
@@ -127,7 +127,7 @@ def attach_patch(self, filename, description, bug):
self.log.info("Attached patch to bug: %s" % bug.weburl)

def ftbfs_bug(self, name):
""" Return the first FTBFS bug we find for a package """
""" Return all FTBFS bugs we find for a package """
short_desc_pattern = '%s: FTBFS in rawhide' % name
query = {
'component': name,
@@ -144,7 +144,7 @@ def ftbfs_bug(self, name):
# currently not recognized by bugzilla. Therefore this test is
# required:
if bug.short_desc.startswith(short_desc_pattern):
return bug
yield bug

def exact_bug(self, **package):
""" Return a particular upstream release ticket for a package. """
@@ -300,9 +300,10 @@ def handle_buildsys_real(self, msg):
return

self.log.info("Handling koji build msg %r" % msg.get('msg_id', None))
bugs = [

# Search for all FTBFS bugs and any upstream bugs we filed earlier.
bugs = list(self.bugzilla.ftbfs_bug(name=package)) + [
self.bugzilla.exact_bug(name=package, upstream=version),
self.bugzilla.ftbfs_bug(name=package),
]
# Filter out None values
bugs = [bug for bug in bugs if bug]

0 comments on commit a64d7d3

Please sign in to comment.
You can’t perform that action at this time.