Load indexes first

Author: alessio-perugini

Diff for: commands/instances.go

@@ -270,31 +270,6 @@ func Init(req *rpc.InitRequest, responseCallback func(r *rpc.InitResponse)) erro
 		// even if it should not.
 		pmb, commitPackageManager := instance.pm.NewBuilder()
 
-		loadBuiltinTools := func() []error {
-			builtinPackage := pmb.GetOrCreatePackage("builtin")
-			return pmb.LoadToolsFromPackageDir(builtinPackage, pmb.PackagesDir.Join("builtin", "tools"))
-		}
-
-		// Load Platforms
-		if profile == nil {
-			for _, err := range pmb.LoadHardware() {
-				s := &arduino.PlatformLoadingError{Cause: err}
-				responseError(s.ToRPCStatus())
-			}
-		} else {
-			// Load platforms from profile
-			errs := pmb.LoadHardwareForProfile(
-				profile, true, downloadCallback, taskCallback,
-			)
-			for _, err := range errs {
-				s := &arduino.PlatformLoadingError{Cause: err}
-				responseError(s.ToRPCStatus())
-			}
-
-			// Load "builtin" tools
-			_ = loadBuiltinTools()
-		}
-
 		// Load packages index
 		urls := []string{globals.DefaultIndexURL}
 		if profile == nil {
@@ -335,6 +310,31 @@ func Init(req *rpc.InitRequest, responseCallback func(r *rpc.InitResponse)) erro
 			}
 		}
 
+		loadBuiltinTools := func() []error {
+			builtinPackage := pmb.GetOrCreatePackage("builtin")
+			return pmb.LoadToolsFromPackageDir(builtinPackage, pmb.PackagesDir.Join("builtin", "tools"))
+		}
+
+		// Load Platforms
+		if profile == nil {
+			for _, err := range pmb.LoadHardware() {
+				s := &arduino.PlatformLoadingError{Cause: err}
+				responseError(s.ToRPCStatus())
+			}
+		} else {
+			// Load platforms from profile
+			errs := pmb.LoadHardwareForProfile(
+				profile, true, downloadCallback, taskCallback,
+			)
+			for _, err := range errs {
+				s := &arduino.PlatformLoadingError{Cause: err}
+				responseError(s.ToRPCStatus())
+			}
+
+			// Load "builtin" tools
+			_ = loadBuiltinTools()
+		}
+
 		// We load hardware before verifying builtin tools are installed
 		// otherwise we wouldn't find them and reinstall them each time
 		// and they would never get reloaded.