Revert "Add --dest-file flag to config init command (#957)" (#1026)

This reverts commit 900654f.

Author: silvanocerza

cli/config/init.go

@@ -17,20 +17,16 @@ package config
 
 import (
 	"os"
+	"path/filepath"
 
 	"github.com/arduino/arduino-cli/cli/errorcodes"
 	"github.com/arduino/arduino-cli/cli/feedback"
-	paths "github.com/arduino/go-paths-helper"
 	"github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 )
 
-var (
-	destDir   string
-	destFile  string
-	overwrite bool
-)
+var destDir string
 
 const defaultFileName = "arduino-cli.yaml"
 
@@ -41,66 +37,39 @@ func initInitCommand() *cobra.Command {
 		Long:  "Creates or updates the configuration file in the data directory or custom directory with the current configuration settings.",
 		Example: "" +
 			"  # Writes current configuration to the configuration file in the data directory.\n" +
-			"  " + os.Args[0] + " config init" +
-			"  " + os.Args[0] + " config init --dest-dir /home/user/MyDirectory" +
-			"  " + os.Args[0] + " config init --dest-file /home/user/MyDirectory/my_settings.yaml",
+			"  " + os.Args[0] + " config init",
 		Args: cobra.NoArgs,
 		Run:  runInitCommand,
 	}
 	initCommand.Flags().StringVar(&destDir, "dest-dir", "", "Sets where to save the configuration file.")
-	initCommand.Flags().StringVar(&destFile, "dest-file", "", "Sets where to save the configuration file.")
-	initCommand.Flags().BoolVar(&overwrite, "overwrite", false, "Overwrite existing config file.")
 	return initCommand
 }
 
 func runInitCommand(cmd *cobra.Command, args []string) {
-	if destFile != "" && destDir != "" {
-		feedback.Errorf("Can't use both --dest-file and --dest-dir flags at the same time.")
-		os.Exit(errorcodes.ErrGeneric)
-	}
-
-	var configFileAbsPath *paths.Path
-	var absPath *paths.Path
-	var err error
-
-	switch {
-	case destFile != "":
-		configFileAbsPath, err = paths.New(destFile).Abs()
-		if err != nil {
-			feedback.Errorf("Cannot find absolute path: %v", err)
-			os.Exit(errorcodes.ErrGeneric)
-		}
-
-		absPath = configFileAbsPath.Parent()
-	case destDir == "":
+	if destDir == "" {
 		destDir = viper.GetString("directories.Data")
-		fallthrough
-	default:
-		absPath, err = paths.New(destDir).Abs()
-		if err != nil {
-			feedback.Errorf("Cannot find absolute path: %v", err)
-			os.Exit(errorcodes.ErrGeneric)
-		}
-		configFileAbsPath = absPath.Join(defaultFileName)
 	}
 
-	if !overwrite && configFileAbsPath.Exist() {
-		feedback.Error("Config file already exists, use --overwrite to discard the existing one.")
+	absPath, err := filepath.Abs(destDir)
+	if err != nil {
+		feedback.Errorf("Cannot find absolute path: %v", err)
 		os.Exit(errorcodes.ErrGeneric)
 	}
+	configFileAbsPath := filepath.Join(absPath, defaultFileName)
 
 	logrus.Infof("Writing config file to: %s", absPath)
-	if err := absPath.MkdirAll(); err != nil {
+
+	if err := os.MkdirAll(absPath, os.FileMode(0755)); err != nil {
 		feedback.Errorf("Cannot create config file directory: %v", err)
 		os.Exit(errorcodes.ErrGeneric)
 	}
 
-	if err := viper.WriteConfigAs(configFileAbsPath.String()); err != nil {
+	if err := viper.WriteConfigAs(configFileAbsPath); err != nil {
 		feedback.Errorf("Cannot create config file: %v", err)
 		os.Exit(errorcodes.ErrGeneric)
 	}
 
-	msg := "Config file written to: " + configFileAbsPath.String()
+	msg := "Config file written to: " + configFileAbsPath
 	logrus.Info(msg)
 	feedback.Print(msg)
 }

commands/daemon/settings_test.go

@@ -18,7 +18,6 @@ package daemon
 import (
 	"context"
 	"encoding/json"
-	"path/filepath"
 	"testing"
 
 	"github.com/spf13/viper"
@@ -31,12 +30,12 @@ import (
 var svc = SettingsService{}
 
 func init() {
-	configuration.Init(filepath.Join("testdata", "arduino-cli.yaml"))
+	configuration.Init("testdata")
 }
 
 func reset() {
 	viper.Reset()
-	configuration.Init(filepath.Join("testdata", "arduino-cli.yaml"))
+	configuration.Init("testdata")
 }
 
 func TestGetAll(t *testing.T) {

configuration/configuration.go

@@ -35,21 +35,15 @@ import (
 func Init(configPath string) {
 	// Config file metadata
 	jww.SetStdoutThreshold(jww.LevelFatal)
-
-	configDir := paths.New(configPath)
-	if configDir != nil && !configDir.IsDir() {
-		viper.SetConfigName(strings.TrimSuffix(configDir.Base(), configDir.Ext()))
-	} else {
-		viper.SetConfigName("arduino-cli")
-	}
+	viper.SetConfigName("arduino-cli")
 
 	// Get default data path if none was provided
 	if configPath == "" {
 		configPath = getDefaultArduinoDataDir()
 	}
 
 	// Add paths where to search for a config file
-	viper.AddConfigPath(filepath.Dir(configPath))
+	viper.AddConfigPath(configPath)
 
 	// Bind env vars
 	viper.SetEnvPrefix("ARDUINO")
@@ -191,39 +185,30 @@ func IsBundledInDesktopIDE() bool {
 }
 
 // FindConfigFile returns the config file path using the argument '--config-file' if specified or via the current working dir
-func FindConfigFile(args []string) string {
+func FindConfigFile() string {
+
 	configFile := ""
-	for i, arg := range args {
+	for i, arg := range os.Args {
 		// 0 --config-file ss
 		if arg == "--config-file" {
-			if len(args) > i+1 {
-				configFile = args[i+1]
+			if len(os.Args) > i+1 {
+				configFile = os.Args[i+1]
 			}
 		}
 	}
 
 	if configFile != "" {
-		return configFile
+		if fi, err := os.Stat(configFile); err == nil {
+			if fi.IsDir() {
+				return configFile
+			}
+			return filepath.Dir(configFile)
+		}
 	}
 
 	return searchCwdForConfig()
 }
 
-func searchCwdForConfig() string {
-	cwd, err := os.Getwd()
-
-	if err != nil {
-		return ""
-	}
-
-	configFile := searchConfigTree(cwd)
-	if configFile == "" {
-		return configFile
-	}
-
-	return configFile + string(os.PathSeparator) + "arduino-cli.yaml"
-}
-
 func searchConfigTree(cwd string) string {
 
 	// go back up to root and search for the config file
@@ -245,3 +230,13 @@ func searchConfigTree(cwd string) string {
 	}
 
 }
+
+func searchCwdForConfig() string {
+	cwd, err := os.Getwd()
+
+	if err != nil {
+		return ""
+	}
+
+	return searchConfigTree(cwd)
+}

configuration/configuration_test.go

@@ -30,12 +30,6 @@ func tmpDirOrDie() string {
 	if err != nil {
 		panic(fmt.Sprintf("error creating tmp dir: %v", err))
 	}
-	// Symlinks are evaluated becase the temp folder on Mac OS is inside /var, it's not writable
-	// and is a symlink to /private/var, we want the full path so we do this
-	dir, err = filepath.EvalSymlinks(dir)
-	if err != nil {
-		panic(fmt.Sprintf("error evaluating tmp dir symlink: %v", err))
-	}
 	return dir
 }
 
@@ -77,40 +71,3 @@ func BenchmarkSearchConfigTree(b *testing.B) {
 	}
 	result = s
 }
-
-func TestFindConfigFile(t *testing.T) {
-	configFile := FindConfigFile([]string{"--config-file"})
-	require.Equal(t, "", configFile)
-
-	configFile = FindConfigFile([]string{"--config-file", "some/path/to/config"})
-	require.Equal(t, "some/path/to/config", configFile)
-
-	configFile = FindConfigFile([]string{"--config-file", "some/path/to/config/arduino-cli.yaml"})
-	require.Equal(t, "some/path/to/config/arduino-cli.yaml", configFile)
-
-	configFile = FindConfigFile([]string{})
-	require.Equal(t, "", configFile)
-
-	// Create temporary directories
-	tmp := tmpDirOrDie()
-	defer os.RemoveAll(tmp)
-	target := filepath.Join(tmp, "foo", "bar", "baz")
-	os.MkdirAll(target, os.ModePerm)
-	require.Nil(t, os.Chdir(target))
-
-	// Create a config file
-	f, err := os.Create(filepath.Join(target, "..", "..", "arduino-cli.yaml"))
-	require.Nil(t, err)
-	f.Close()
-
-	configFile = FindConfigFile([]string{})
-	require.Equal(t, filepath.Join(tmp, "foo", "arduino-cli.yaml"), configFile)
-
-	// Create another config file
-	f, err = os.Create(filepath.Join(target, "arduino-cli.yaml"))
-	require.Nil(t, err)
-	f.Close()
-
-	configFile = FindConfigFile([]string{})
-	require.Equal(t, filepath.Join(target, "arduino-cli.yaml"), configFile)
-}

main.go

@@ -25,7 +25,7 @@ import (
 )
 
 func main() {
-	configuration.Init(configuration.FindConfigFile(os.Args))
+	configuration.Init(configuration.FindConfigFile())
 	i18n.Init()
 	arduinoCmd := cli.NewCommand()
 	if err := arduinoCmd.Execute(); err != nil {