Move ARTIFACT_DIR to config script and source it

4 files changed, 22 insertions, 10 deletions

diff --git a/desktop/scripts/release/4-make-release b/desktop/scripts/release/4-make-release
index 71f52a2acd..7274e50056 100755
--- a/desktop/scripts/release/4-make-release
+++ b/desktop/scripts/release/4-make-release
@@ -27,10 +27,10 @@ fi
 
 PRODUCT_VERSION=$1
 
-ARTIFACT_DIR="./artifacts"
+# shellcheck source=desktop/scripts/release/release-config.sh
+source "$SCRIPT_DIR/release-config.sh"
 
-rm -rf $ARTIFACT_DIR
-mkdir -p $ARTIFACT_DIR
+rm -rf "$ARTIFACT_DIR" && mkdir -p "$ARTIFACT_DIR" || exit 1
 
 function publish_release {
     echo ">>> Downloading changelog"
@@ -85,5 +85,5 @@ function publish_release {
     echo "The above URL contains the text \"untagged\", but don't worry it is tagged properly and everything will look correct once it's published."
 }
 
-./download-release-artifacts "$PRODUCT_VERSION"
+./download-release-artifacts "$PRODUCT_VERSION" "$ARTIFACT_DIR"
 publish_release
diff --git a/desktop/scripts/release/5-update-and-publish-metadata b/desktop/scripts/release/5-update-and-publish-metadata
index ec385565f8..86f646b501 100755
--- a/desktop/scripts/release/5-update-and-publish-metadata
+++ b/desktop/scripts/release/5-update-and-publish-metadata
@@ -23,7 +23,8 @@ PRODUCT_VERSION=$1
 BUILDSERVER_HOST=$2
 METADATA_SERVER_HOST=$3
 
-ARTIFACT_DIR="./artifacts"
+# shellcheck source=desktop/scripts/release/release-config.sh
+source "$SCRIPT_DIR/release-config.sh"
 
 function publish_metadata {
     local platforms
@@ -69,6 +70,6 @@ function remove_release_artifacts {
     rm -r "$ARTIFACT_DIR"
 }
 
-./download-release-artifacts "$PRODUCT_VERSION"
+./download-release-artifacts "$PRODUCT_VERSION" "$ARTIFACT_DIR"
 publish_metadata
 remove_release_artifacts
diff --git a/desktop/scripts/release/download-release-artifacts b/desktop/scripts/release/download-release-artifacts
index 8e8a77f0ad..aec0fbfe3e 100755
--- a/desktop/scripts/release/download-release-artifacts
+++ b/desktop/scripts/release/download-release-artifacts
@@ -7,19 +7,22 @@ set -eu
 SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 cd "$SCRIPT_DIR"
 
-if [ $# -ne 1 ]; then
+if [ $# -ne 2 ]; then
     echo "Please provide the following arguments:"
     echo "    $(basename "$0") \\"
-    echo "        <product version>"
+    echo "        <product version> \\"
+    echo "        <artifact download directory> "
     exit 1
 fi
 
+# The app version to download
 PRODUCT_VERSION=$1
+# The directory where the artifacts will be downloaded to
+ARTIFACT_DIR=$2
 
-ARTIFACT_DIR="./artifacts"
 URL_BASE="https://releases.mullvad.net/desktop/releases"
 
-mkdir -p $ARTIFACT_DIR
+mkdir -p "$ARTIFACT_DIR"
 
 # The signer key file "mullvad-code-signing-key.asc" is expected to exist in the current working directory.
 SIGNER_KEY_FILE="./mullvad-code-signing-key.asc"
diff --git a/desktop/scripts/release/release-config.sh b/desktop/scripts/release/release-config.sh
new file mode 100644
index 0000000000..3e053da880
--- /dev/null
+++ b/desktop/scripts/release/release-config.sh
@@ -0,0 +1,8 @@
+#!/usr/bin/env bash
+
+# Configuration variables shared between the release scripts in this directory.
+
+# Where to download app installers locally during the release process.
+# This value is also hardcoded into the `mullvad-release` binary and
+# has to be in sync with that value
+export ARTIFACT_DIR="artifacts"