[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