Correctly trap errors in shell scripts (#1844)

graphql · May 2, 2019 · 316e24e · 316e24e
1 parent 22f0719
commit 316e24e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/resources/checkgit.sh b/resources/checkgit.sh
@@ -1,5 +1,5 @@
 # Exit immediately if any subcommand terminated
-trap "exit 1" TERM
+trap "exit 1" ERR
 
 #
 # This script determines if current git state is the up to date master. If so

diff --git a/resources/gitpublish.sh b/resources/gitpublish.sh
@@ -1,7 +1,7 @@
-#!/bin/sh -e
+#!/bin/sh
 
 # Exit immediately if any subcommand terminated
-trap "exit 1" TERM
+trap "exit 1" ERR
 
 # This script maintains a git branch which mirrors master but in a form that
 # what will eventually be deployed to npm, allowing npm dependencies to use:

diff --git a/resources/prepublish.sh b/resources/prepublish.sh
@@ -1,5 +1,5 @@
 # Exit immediately if any subcommand terminated
-trap "exit 1" TERM
+trap "exit 1" ERR
 
 # Publishing to NPM is currently supported by Travis CI, which ensures that all
 # tests pass first and the deployed module contains the correct file structure.