[master/rhel7] Don't allow related bugs without acks

Brian C. Lane bcl at redhat.com
Thu Oct 23 03:18:13 UTC 2014


A related bug shouldn't override bugs without acks, and it should also
have acks itself.
---
 scripts/makebumpver | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/scripts/makebumpver b/scripts/makebumpver
index 4ab94c4..941b509 100755
--- a/scripts/makebumpver
+++ b/scripts/makebumpver
@@ -355,11 +355,11 @@ class MakeBumpVer:
                                                               fixedIn) or \
                             not self._isRHELBugAcked(ckbug, commit, summary)):
                             bad = True
-                        elif valid and action == 'Related' and \
-                            self._isRHELBugAcked(ckbug, commit, summary):
-                            print "*** Bug %s Related commit %s is allowed\n" % (bug, commit)
-                            # Related bugs only need to be valid and have an ack
-                            bad = False
+                        elif valid and action == 'Related':
+                            if self._isRHELBugAcked(ckbug, commit, summary):
+                                print "*** Bug %s Related commit %s is allowed\n" % (bug, commit)
+                            else:
+                                bad = True
 
                 if len(rhbz) == 0 and not self.skip_all:
                     print "*** No bugs referenced in commit %s\n" % commit
-- 
1.9.3



More information about the anaconda-patches mailing list