Merge pull request #3741 from element-hq/feature/bma/releaseImprovement

Release script improvement

tools/github/download_all_github_artifacts.py

@@ -10,7 +10,7 @@ import argparse
 import hashlib
 import json
 import os
-# Run `pip3 install requests` if not installed yet
+# Run `pip3 install requests --break-system-packages` if not installed yet
 import requests
 # Run `pip3 install re` if not installed yet
 import re

tools/release/release.sh

@@ -185,11 +185,24 @@ targetPath="./tmp/Element/${version}"
 printf "\n================================================================================\n"
 printf "Downloading the artifacts...\n"
 
+ret=1
+
+while [[ $ret -ne 0 ]]; do
   python3 ./tools/github/download_all_github_artifacts.py \
      --token "${gitHubToken}" \
      --runUrl "${runUrl}" \
      --directory "${targetPath}"
 
+  ret=$?
+  if [[ $ret -ne 0 ]]; then
+    read -p "Error while downloading the artifacts. You may want to fix the issue and retry. Retry (yes/no) default to yes? " doRetry
+    doRetry=${doRetry:-yes}
+    if [ "${doRetry}" == "no" ]; then
+      exit 1
+    fi
+  fi
+done
+
 printf "\n================================================================================\n"
 printf "Unzipping the F-Droid artifact...\n"
 
@@ -312,8 +325,8 @@ printf "\n======================================================================
 printf "The file ${signedBundlePath} has been signed and can be uploaded to the PlayStore!\n"
 
 printf "\n================================================================================\n"
-read -p "Do you want to build the APKs from the app bundle? You need to do this step if you want to install the application to your device. (yes/no) default to yes " doBuildApks
+read -p "Do you want to build the APKs from the app bundle? You need to do this step if you want to install the application to your device. (yes/no) default to no " doBuildApks
-doBuildApks=${doBuildApks:-yes}
+doBuildApks=${doBuildApks:-no}
 
 if [ "${doBuildApks}" == "yes" ]; then
   printf "Building apks...\n"