fix IsBundledInDesktopIDE logic

Author: Massimiliano Pippi

configuration/configuration.go

@@ -151,25 +151,27 @@ func IsBundledInDesktopIDE() bool {
 
 	ideDir := filepath.Dir(executablePath)
 
-	// We check an arbitrary number of folders that are part of the IDE
-	// install tree
+	// To determine if the CLI is bundled with an IDE, We check an arbitrary
+	// number of folders that are part of the IDE install tree
 	tests := []string{
 		"tools-builder",
 		"examples/01.Basics/Blink",
-		"portable",
 	}
 
 	for _, test := range tests {
 		if _, err := os.Stat(filepath.Join(ideDir, test)); err != nil {
 			// the test folder doesn't exist or is not accessible
 			return viper.GetBool("IDE.Bundled")
 		}
+	}
 
-		if test == "portable" {
-			viper.Set("IDE.Portable", true)
-		}
+	// Check whether this is a portable install
+	if _, err := os.Stat(filepath.Join(ideDir, "portable")); err != nil {
+		viper.Set("IDE.Portable", true)
 	}
 
+	// Persist IDE-related config settings and return true
+	viper.Set("IDE.Bundled", false)
 	viper.Set("IDE.Directory", ideDir)
 	return true
 }