Always complete the old auto-upgrade procedure

This is required for clients upgrading from versions <=1.2.7

Author: cmaglie

updater/updater.go

@@ -22,7 +22,9 @@ import (
 	"fmt"
 	"io"
 	"net/http"
+	"path/filepath"
 	"runtime"
+	"strings"
 
 	log "github.com/sirupsen/logrus"
 )
@@ -77,3 +79,19 @@ func fetch(url string) (io.ReadCloser, error) {
 	}
 	return resp.Body, nil
 }
+
+// addTempSuffixToPath adds the "-temp" suffix to the path to an executable file (a ".exe" extension is replaced with "-temp.exe")
+func addTempSuffixToPath(path string) string {
+	if filepath.Ext(path) == "exe" {
+		path = strings.Replace(path, ".exe", "-temp.exe", -1)
+	} else {
+		path = path + "-temp"
+	}
+
+	return path
+}
+
+// removeTempSuffixFromPath removes "-temp" suffix from the path to an executable file (a "-temp.exe" extension is replaced with ".exe")
+func removeTempSuffixFromPath(path string) string {
+	return strings.Replace(path, "-temp", "", -1)
+}

updater/updater_darwin.go

@@ -53,6 +53,21 @@ import (
 )
 
 func start(src string) string {
+	if strings.Contains(src, "-temp") {
+		// This is required to transition from the previous auto-update system to the new one.
+		// Updating from version <=1.2.7 will produce the `ArduinoCreateAgent-temp` file and we should
+		// complete the upgrade by copying the `ArduinoCreateAgent-temp` executable back to the original.
+		//
+		// This procedure will be automatically skipped starting from version >1.2.7.
+
+		newPath := removeTempSuffixFromPath(src)
+		if err := copyExe(src, newPath); err != nil {
+			log.Println("Copy error: ", err)
+			panic(err)
+		}
+		return newPath
+	}
+
 	return ""
 }
 

updater/updater_default.go

@@ -111,22 +111,6 @@ func copyExe(from, to string) error {
 	return nil
 }
 
-// addTempSuffixToPath adds the "-temp" suffix to the path to an executable file (a ".exe" extension is replaced with "-temp.exe")
-func addTempSuffixToPath(path string) string {
-	if filepath.Ext(path) == "exe" {
-		path = strings.Replace(path, ".exe", "-temp.exe", -1)
-	} else {
-		path = path + "-temp"
-	}
-
-	return path
-}
-
-// removeTempSuffixFromPath removes "-temp" suffix from the path to an executable file (a "-temp.exe" extension is replaced with ".exe")
-func removeTempSuffixFromPath(path string) string {
-	return strings.Replace(path, "-temp", "", -1)
-}
-
 // Updater is the configuration and runtime data for doing an update.
 //
 // Note that ApiURL, BinURL and DiffURL should have the same value if all files are available at the same location.