From fe057c77ae70a6d1c08119379d2599ba5a623504 Mon Sep 17 00:00:00 2001 From: Jon Ciesla Date: Thu, 16 Aug 2012 14:42:26 -0500 Subject: [PATCH] Robusticize fedora-review flag check. --- scripts/process-git-requests/process-git-requests | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/scripts/process-git-requests/process-git-requests b/scripts/process-git-requests/process-git-requests index 3f06298..afceae8 100755 --- a/scripts/process-git-requests/process-git-requests +++ b/scripts/process-git-requests/process-git-requests @@ -485,9 +485,13 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc warned = True frflag = bug.get_flags('fedora-review') - if frflag[0]['status'] != '+': - warnings.append("WARNING: fedora-review flag not set to '+'") + if not frflag[0]['status']: + warnings.append("WARNING: fedora-review flag not set") warned = True + else: + if frflag[0]['status'] != '+': + warnings.append("WARNING: fedora-review flag not set to '+'") + warned = True if frflag[0]['setter_email'] == bug.reporter: warnings.append("WARNING: fedora-review flag set by review submitter! Verify that review was approved by reviewer!") warned = True