[ARVADOS] updated: 050a8d39b9ca831073520316dbc197270b11720d

git at public.curoverse.com git at public.curoverse.com
Thu Aug 28 13:33:36 EDT 2014


Summary of changes:
 sdk/cli/bin/crunch-job | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

       via  050a8d39b9ca831073520316dbc197270b11720d (commit)
      from  8350db2288e8f915bb70328e89c2717d69ba62ad (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 050a8d39b9ca831073520316dbc197270b11720d
Author: Tom Clegg <tom at curoverse.com>
Date:   Thu Aug 28 13:33:28 2014 -0400

    Complain and fail if exec() returns. No issue #

diff --git a/sdk/cli/bin/crunch-job b/sdk/cli/bin/crunch-job
index 4d2aad9..402c2de 100755
--- a/sdk/cli/bin/crunch-job
+++ b/sdk/cli/bin/crunch-job
@@ -1493,8 +1493,12 @@ for my $dir ($destdir, $task_work) {
 open L, ">", "$destdir.lock" or die "$destdir.lock: $!";
 flock L, LOCK_EX;
 if (readlink ("$destdir.commit") eq $commit && -d $destdir) {
-    exec(@ARGV) if @ARGV;
-    exit 0;
+    if (@ARGV) {
+        exec(@ARGV);
+        die "Cannot exec `@ARGV`: $!";
+    } else {
+        exit 0;
+    }
 }
 
 unlink "$destdir.commit";
@@ -1542,6 +1546,7 @@ close L;
 
 if (@ARGV) {
     exec(@ARGV);
+    die "Cannot exec `@ARGV`: $!";
 } else {
     exit 0;
 }

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list