ncp-update: fail if version cant be parsedv1.3.1

2 files changed, 18 insertions, 15 deletions

diff --git a/bin/ncp-update b/bin/ncp-update
index a68afc26..93c11408 100755
--- a/bin/ncp-update
+++ b/bin/ncp-update
@@ -9,7 +9,7 @@
   [[ "$BRANCH" != "master" ]] && echo "INFO: updating to development branch '$BRANCH'"
 
   TMPDIR="$( mktemp -d /tmp/ncp-update.XXXXXX || ( echo "Failed to create temp dir. Exiting" >&2; exit 1 ) )"
-  trap "rm -rf \"${TMPDIR}\"; exit 0" 0 1 2 3 15
+  trap "cd /; rm -rf \"${TMPDIR}\"; exit 0" 0 1 2 3 15
 
   echo -e "Downloading updates"
   git clone --depth 20 -b "$BRANCH" -q https://github.com/nextcloud/nextcloudpi.git "$TMPDIR" || {
@@ -24,18 +24,19 @@
 
   cd "$TMPDIR"
   VER=$( git describe --always --tags | grep -oP "v\d+\.\d+\.\d+" )
-  grep -qP "v\d+\.\d+\.\d+" <<< "$VER" && {       # check format
-    echo "$VER" > /usr/local/etc/ncp-version
-    echo "$VER" > /var/run/.ncp-latest-version
-
-    # write changelog
-    git log --graph --oneline --decorate \
-      --pretty=format:"[%D] %s" --date=short | \
-      grep 'tag: v' | \
-      sed '/HEAD ->\|origin/s|\[.*\(tag: v[0-9]\+\.[0-9]\+\.[0-9]\+\).*\]|[\1]|' | \
-      sed 's|* \[tag: |[|' > /usr/local/etc/ncp-changelog
-  }
-  cd /
+
+  # check format
+  grep -qP "v\d+\.\d+\.\d+" <<< "$VER" || { "Error: missing version"; exit 1; }
+
+  echo "$VER" > /usr/local/etc/ncp-version
+  echo "$VER" > /var/run/.ncp-latest-version
+
+  # write changelog
+  git log --graph --oneline --decorate \
+    --pretty=format:"[%D] %s" --date=short | \
+    grep 'tag: v' | \
+    sed '/HEAD ->\|origin/s|\[.*\(tag: v[0-9]\+\.[0-9]\+\.[0-9]\+\).*\]|[\1]|' | \
+    sed 's|* \[tag: |[|' > /usr/local/etc/ncp-changelog
 
   echo -e "NextCloudPi updated to version $VER"
   exit 0
diff --git a/changelog.md b/changelog.md
index 2fce85e8..3dcc2a70 100644
--- a/changelog.md
+++ b/changelog.md
@@ -1,7 +1,9 @@
 
-[v1.3.0](https://github.com/nextcloud/nextcloudpi/commit/8a49536) (2019-01-12) upgrade to NC15.0.2
+[v1.3.1](https://github.com/nextcloud/nextcloudpi/commit/bb69e8d) (2019-01-13) ncp-update: fail if version cant be parsed
 
-[v1.2.0](https://github.com/nextcloud/nextcloudpi/commit/9eaab31) (2019-01-08) add NCP Nextcloud app
+[v1.3.0  ](https://github.com/nextcloud/nextcloudpi/commit/2c943b7) (2019-01-12) upgrade to NC15.0.2
+
+[v1.2.0  ](https://github.com/nextcloud/nextcloudpi/commit/9eaab31) (2019-01-08) add NCP Nextcloud app
 
 [v1.1.3  ](https://github.com/nextcloud/nextcloudpi/commit/d21592c) (2019-01-11) nc-update-nextcloud: only try to restore on reboot once