rename release.sh to tarball.sh, more logging

"release" did not describe what it really did
node-gfx · Nov 1, 2018 · c6493c3 · c6493c3
1 parent 8b0e174
commit c6493c3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/ci/install.sh b/ci/install.sh
@@ -46,7 +46,7 @@ for ver in $NODEJS_VERSIONS; do
  if [ $? -eq 0 ]; then
  cd ..
  source ci/$OS/bundle.sh;
- source ci/release.sh;
+ source ci/tarball.sh;
  else
  echo "error building in nodejs version $ver"
  cd ..

diff --git a/ci/release.sh → ci/tarball.sh b/ci/release.sh → ci/tarball.sh
@@ -7,7 +7,11 @@ FILENAME=$(
 ).tar.gz;
 
 # Zip up the release
-tar -C node-canvas/build -czf $FILENAME Release/ || {
+tar -C node-canvas/build -czf $FILENAME Release/;
+
+if [ $? -ne 0 ]; then
  echo "failed to make tarball $FILENAME from node-canvas/build"
  exit 1;
-}
+else
+ echo "created release tarball $FILENAME";
+fi