[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [Repo-Management] ade6cc8: Fail only once all auditing is done
From:       Ben Cooksley <bcooksley () kde ! org>
Date:       2010-11-15 20:29:34
Message-ID: 20101115202934.1B917A60A6 () git ! kde ! org
[Download RAW message or body]

commit ade6cc840c0e386212c8d8caed987b8ce2cce380
branch master
Author: Ben Cooksley <bcooksley@kde.org>
Date:   Tue Nov 16 09:29:41 2010 +1300

    Fail only once all auditing is done

diff --git a/hooks/update.secondary b/hooks/update.secondary
index a55d5a9..a24e007 100755
--- a/hooks/update.secondary
+++ b/hooks/update.secondary
@@ -78,12 +78,16 @@ fi
 auditdir=$mgmtdir/repo-configs/audit/$repopath
 if [ ! -f $auditdir/skip-audit ]; then
     echo "Auditing commits, please wait ..."
+    auditfail=0
     for commit_id in $revlist ; do
         $mgmtdir/bin/audit-commit.pl ${commit_id}
         if [ $? -ne 0 ]; then
-            exit 1
+            auditfail=1
         fi
     done
+    if [ $auditfail -ne 0 ]; then
+        exit 1
+    fi
 fi
 
 # Are post-commit hooks disabled?
[prev in list] [next in list] [prev in thread] [next in thread]