[ARVADOS] updated: 1bae4ec29e4181bfb41f1bf828f4764758e0026f

[git at public.curoverse.com]

Summary of changes:
 sdk/cli/bin/crunch-job | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

       via  1bae4ec29e4181bfb41f1bf828f4764758e0026f (commit)
      from  1f4000e57a6c0e82ed2d7de311f0137833006120 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 1bae4ec29e4181bfb41f1bf828f4764758e0026f
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Thu Sep 25 14:16:55 2014 -0400

    3859: Fix checking for errors in eval{}

diff --git a/sdk/cli/bin/crunch-job b/sdk/cli/bin/crunch-job
index 4fe12c9..c04e2df 100755
--- a/sdk/cli/bin/crunch-job
+++ b/sdk/cli/bin/crunch-job
@@ -282,7 +282,10 @@ my $jobmanager_id;
 if ($job_has_uuid)
 {
   # Claim this job, and make sure nobody else does
-  if (eval {$arv->{'jobs'}->{'lock'}->execute('uuid' => $Job->{'uuid'})}) {
+  eval {
+    $arv->{'jobs'}->{'lock'}->execute('uuid' => $Job->{'uuid'})
+  };
+  if ($@) {
     Log(undef, "Error while updating / locking job, exiting ".EX_TEMPFAIL);
     exit EX_TEMPFAIL;    
   };

-----------------------------------------------------------------------


hooks/post-receive
--