From f9a30c8cda0bbeaddd6570bfa2b172f01e780882 Mon Sep 17 00:00:00 2001
From: CURZOLA Pierre <pierre.curzola@gmail.com>
Date: Tue, 26 Sep 2023 15:38:28 +0200
Subject: [PATCH 1/2] fix(install): remove wget and sha256sum dependencies

---
 dists/install.sh | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/dists/install.sh b/dists/install.sh
index b5d9a7b54..99b15ee45 100755
--- a/dists/install.sh
+++ b/dists/install.sh
@@ -1,5 +1,9 @@
 #!/bin/bash
 
+function sha256sum() {
+  shasum -a 256 "$@"
+}
+
 main() {
   status() {
     echo -en "-----> $*"
@@ -17,14 +21,6 @@ main() {
     echo -en " /!\\   $*"
   }
 
-  check_command_exists() {
-    if ! command -v $1 &> /dev/null
-    then
-        error "$1 command could not be found, please install it first"
-        exit 1
-    fi
-  }
-
   clean_install() {
     tmpdir=$1
 
@@ -139,14 +135,11 @@ main() {
   fi
   echo "DONE"
 
-  check_command_exists sha256sum
-  check_command_exists wget
-
   status "Verifying the checksum...  "
   checksums_url="https://github.com/Scalingo/cli/releases/download/${version}/checksums.txt"
   # Use cut's short parameter to be compatible with MacOS: https://ss64.com/osx/cut.html
   checksum_computed=$(sha256sum ${tmpdir}/${archive_name} | cut -d" " -f1)
-  checksum_expected=$(wget --quiet --output-document - $checksums_url | grep $archive_name | cut -d" " -f1)
+  checksum_expected=$(curl --silent --location "$checksums_url" | grep "$archive_name" | cut -d" " -f1)
   if [[ "$checksum_computed" != "$checksum_expected" ]]; then
     echo "INVALID"
     error "Checksums don't match.\n"

From 359547a07af193e34a67d14a727b6041439ba8a1 Mon Sep 17 00:00:00 2001
From: CURZOLA Pierre <pierre.curzola@gmail.com>
Date: Tue, 26 Sep 2023 15:42:35 +0200
Subject: [PATCH 2/2] style(install): linter offenses

---
 dists/install.sh | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/dists/install.sh b/dists/install.sh
index 99b15ee45..b5355d7ed 100755
--- a/dists/install.sh
+++ b/dists/install.sh
@@ -24,9 +24,9 @@ main() {
   clean_install() {
     tmpdir=$1
 
-    rm -r $tmpdir
+    rm -r "$tmpdir"
     # If installed through one line install, remove script
-    if [ "x$0" = "xinstall" ] ; then
+    if [ "$0" = "install" ] ; then
       rm "$0"
     fi
   }
@@ -35,7 +35,7 @@ main() {
     info "$* [Y/n] "
     while true; do
         read answer
-        case $(echo "$answer" | tr "[A-Z]" "[a-z]") in
+        case $(echo "$answer" | tr '[:upper:]' '[:lower:]') in
         y|yes|"" ) return 0;;
         n|no     ) return 1;;
         esac
@@ -54,14 +54,14 @@ main() {
     echo
   }
 
-  if [ "x$DEBUG" = "xtrue" ] ; then
+  if [ -n "$DEBUG" ] ; then
     set -x
   fi
 
   uname -a | grep -qi 'Linux' ; is_linux=$?
   uname -a | grep -qi 'Darwin' ; is_darwin=$?
 
-  os=$(uname -s | tr '[A-Z]' '[a-z]')
+  os=$(uname -s | tr '[:upper:]' '[:lower:]')
   ext='tar.gz'
   if [ "$os" != "linux" ] && [ "$os" != "darwin" ]; then
     echo "Unsupported OS: $(uname -s)"
@@ -127,8 +127,8 @@ main() {
   url="https://github.com/Scalingo/cli/releases/download/${version}/${archive_name}"
 
   status "Downloading Scalingo client...  "
-  curl --silent --fail --location --output ${tmpdir}/${archive_name} ${url}
-  if [ ! -f ${tmpdir}/${archive_name} ]; then
+  curl --silent --fail --location --output "${tmpdir}/${archive_name}" "$url"
+  if [ ! -f "${tmpdir}/${archive_name}" ]; then
     echo ""
     error "Fail to download the CLI archive\n"
     exit 1
@@ -138,7 +138,7 @@ main() {
   status "Verifying the checksum...  "
   checksums_url="https://github.com/Scalingo/cli/releases/download/${version}/checksums.txt"
   # Use cut's short parameter to be compatible with MacOS: https://ss64.com/osx/cut.html
-  checksum_computed=$(sha256sum ${tmpdir}/${archive_name} | cut -d" " -f1)
+  checksum_computed=$(sha256sum "${tmpdir}/${archive_name}" | cut -d" " -f1)
   checksum_expected=$(curl --silent --location "$checksums_url" | grep "$archive_name" | cut -d" " -f1)
   if [[ "$checksum_computed" != "$checksum_expected" ]]; then
     echo "INVALID"
@@ -166,7 +166,7 @@ main() {
   target_dir="${target_dir:-$default_target_dir}"
   target="$target_dir/scalingo"
 
-  if [ -x "$target" -a -z "$yes_to_overwrite" ] ; then
+  if [ -x "$target" ] && [ -z "$yes_to_overwrite" ] ; then
     export DISABLE_UPDATE_CHECKER=true
     # Use cut's short parameter to be compatible with MacOS: https://ss64.com/osx/cut.html
     new_version=$($exe_path --version | cut -d' ' -f4)
@@ -175,7 +175,7 @@ main() {
 
     if ! ask "Do you want to replace it with version ${new_version}?"  ; then
       status "Aborting...\n"
-      exit -1
+      exit 1
     fi
   fi
 
@@ -194,7 +194,7 @@ main() {
     fi
   fi
 
-  $sudo mv $exe_path "$target" ; rc=$?
+  $sudo mv "$exe_path" "$target" ; rc=$?
 
   if [ $rc -ne 0 ] ; then
     error "Fail to install Scalingo client (return $rc)\n"