From fff5874451cebde040c932faf1a38636a1f1b47b Mon Sep 17 00:00:00 2001 From: Jason Tibbitts Date: Mon, 15 Nov 2010 09:28:42 -0600 Subject: [PATCH] Fix possible ignoring of owner or CC warnings. If the only warnings were due to bad owners or CC's, the script failed to force acceptance of them before proceeding. --- scripts/process-git-requests/process-git-requests | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/scripts/process-git-requests/process-git-requests b/scripts/process-git-requests/process-git-requests index 0c09363..568bd94 100755 --- a/scripts/process-git-requests/process-git-requests +++ b/scripts/process-git-requests/process-git-requests @@ -445,7 +445,10 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc items = parse_prefixed_lines(comment['text']) request = clean_request(items) - warnings.extend(check_owners(fas, request['owner'], request['comaintainers'], request['cc_list'])) + w = check_owners(fas, request['owner'], request['comaintainers'], request['cc_list']) + if len(w): + warnings.extend(w) + warned = True if not 'Owners' in items: warnings.append("WARNING: No owners provided.")