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

Handle a change request with no new branches specified.

Browse source
This commit is contained in:
Jason Tibbitts 2010-08-03 18:04:23 -05:00
parent d5d7dc314b
commit f9b00c60dd
1 changed files with 11 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -216,6 +216,7 @@ def parse_prefixed_lines(s):
lastitem = '' lastitem = ''
items = {} items = {}
items['Branches'] = '' items['Branches'] = ''
items['New Branches'] = ''
lines = s.splitlines() lines = s.splitlines()
# Skip until the Request line # Skip until the Request line
@ -277,16 +278,15 @@ def clean_request(items):
items['Branches'] = branches items['Branches'] = branches
request['branches'] = branches.split() request['branches'] = branches.split()
if 'New Branches' in items: branches = items['New Branches'].strip()
branches = items['New Branches'].strip() branches = re.sub(r',', ' ', branches)
branches = re.sub(r',', ' ', branches) branches = re.sub(r'f', 'F', branches)
branches = re.sub(r'f', 'F', branches) branches = re.sub(r'F([1-9][0-9])', r'F-\1', branches)
branches = re.sub(r'F([1-9][0-9])', r'F-\1', branches) branches = re.sub(r'F-14', r'f14', branches)
branches = re.sub(r'F-14', r'f14', branches) branches = re.sub(r' +', ' ', branches)
branches = re.sub(r' +', ' ', branches) branches = branches.strip()
branches = branches.strip() items['New Branches'] = branches
items['New Branches'] = branches request['newbranches'] = branches.split()
request['newbranches'] = branches.split()
owners = items['Owners'].strip() owners = items['Owners'].strip()
owners = re.sub(r',', ' ', owners) owners = re.sub(r',', ' ', owners)
@ -517,7 +517,7 @@ def process_change_request(bug, comment, allcomments, firstfound, pkgdb, branche
if not 'Owners' in items: if not 'Owners' in items:
warnings.append("WARNING: No owners provided.") warnings.append("WARNING: No owners provided.")
warned = True warned = True
if not 'New Branches' in items: if not len(items['New Branches']):
warnings.append("WARNING: No new branches requested.") warnings.append("WARNING: No new branches requested.")
for i in request['branches']: for i in request['branches']:
if i not in branches: if i not in branches: