Infrastructure/fedora-infrastructure
1
0
Fork 0
Code Issues 70 Pull requests Projects Releases Packages Wiki Activity Actions

Fixes for latest python-bugzilla.

Browse source
This commit is contained in:
Jon Ciesla 2013-03-23 09:43:11 -05:00
parent 77c937eda1
commit 30a7b6c4b2
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
scripts/process-git-requests/process-git-requests
View file

@ -406,7 +406,7 @@ def process_no_request(bug, allcomments):
ok = raw_input("\nClear the fedora-cvs flag (y/n)?") ok = raw_input("\nClear the fedora-cvs flag (y/n)?")
if ok == 'y': if ok == 'y':
print "Clearing the flag..." print "Clearing the flag..."
bug.updateflags({'fedora-cvs':'X', 'nomail':1}) bug.updateflags({'fedora-cvs':'X'})
break break
elif ok == 'n': elif ok == 'n':
return True return True
@ -493,7 +493,7 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc
if frflag[0]['status'] != '+': if frflag[0]['status'] != '+':
warnings.append("WARNING: fedora-review flag not set to '+'") warnings.append("WARNING: fedora-review flag not set to '+'")
warned = True warned = True
if frflag[0]['setter_email'] == bug.reporter: if frflag[0]['setter'] == bug.reporter:
warnings.append("WARNING: fedora-review flag set by review submitter! Verify that review was approved by reviewer!") warnings.append("WARNING: fedora-review flag set by review submitter! Verify that review was approved by reviewer!")
warned = True warned = True
@ -537,7 +537,7 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc
ok = raw_input("\nClear the fedora-cvs flag (y/n)?") ok = raw_input("\nClear the fedora-cvs flag (y/n)?")
if ok == 'y': if ok == 'y':
print "Clearing the flag..." print "Clearing the flag..."
bug.updateflags({'fedora-cvs':'X', 'nomail':1}) bug.updateflags({'fedora-cvs':'X'})
return (False, True) return (False, True)
elif ok == 'e': elif ok == 'e':
req_string = edit_string(req_string) req_string = edit_string(req_string)
@ -572,7 +572,7 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc
print 'Updating bugzilla...' print 'Updating bugzilla...'
# XXX Need to handle errors here - might be done, limburgher 2012-09-05 # XXX Need to handle errors here - might be done, limburgher 2012-09-05
try: try:
bug.updateflags({'fedora-cvs':'+', 'nomail':1}) bug.updateflags({'fedora-cvs':'+'})
except Exception, e: except Exception, e:
print "Bugzilla call failed:" print "Bugzilla call failed:"
print e print e
@ -671,7 +671,7 @@ def process_change_request(bug, comment, allcomments, firstfound, pkgdb, branche
ok = raw_input("\nClear the fedora-cvs flag (y/n)?") ok = raw_input("\nClear the fedora-cvs flag (y/n)?")
if ok == 'y': if ok == 'y':
print "Clearing the flag..." print "Clearing the flag..."
bug.updateflags({'fedora-cvs':'X', 'nomail':1}) bug.updateflags({'fedora-cvs':'X'})
return (False, True) return (False, True)
elif ok == 'e': elif ok == 'e':
req_string = edit_string(req_string) req_string = edit_string(req_string)
@ -706,7 +706,7 @@ def process_change_request(bug, comment, allcomments, firstfound, pkgdb, branche
print 'Updating bugzilla...' print 'Updating bugzilla...'
# XXX Need to handle errors here - might be done, limburgher 2012-09-05 # XXX Need to handle errors here - might be done, limburgher 2012-09-05
try: try:
bug.updateflags({'fedora-cvs':'+', 'nomail':1}) bug.updateflags({'fedora-cvs':'+'})
except Exception, e: except Exception, e:
print "Bugzilla call failed:" print "Bugzilla call failed:"
print e print e