fix --conv-yaml func

engine/config.go

@@ -112,8 +112,8 @@ func InitConf(specPath string) (*Config, error) {
 	cf := viper.ConfigFileUsed()
 	log.Println("[config] selected config file: ", cf)
 	if !configExists || dirChanged {
-		c.WriteYaml()
-		log.Println("[config] config file written: ", cf, "exists:", configExists, "dirchanged", dirChanged)
+		c.WriteDefault()
+		log.Println("[config] config file updated: ", cf, "exists:", configExists, "dirchanged", dirChanged)
 	}
 
 	return c, nil
@@ -221,8 +221,12 @@ func (c *Config) SyncViper(nc Config) {
 	}
 }
 
-func (c *Config) WriteYaml() error {
+func (c *Config) WriteDefault() error {
 	cf := viper.ConfigFileUsed()
+	return c.WriteYaml(cf)
+}
+
+func (c *Config) WriteYaml(cf string) error {
 	d, err := yaml.Marshal(c)
 	if err != nil {
 		return err

server/server.go

@@ -158,13 +158,17 @@ func (s *Server) Run(version string) error {
 	// write cloud-torrent.yaml at the same dir with -c conf and exit
 	if s.ConvYAML {
 		cf := viper.ConfigFileUsed()
-		log.Println("[config] current file path: ", cf)
-		// replace orig config file ext with ".yaml"
-		ymlcf := cf[:len(cf)-len(path.Ext(cf))] + ".yaml"
-		if err := viper.WriteConfigAs(ymlcf); err != nil {
-			return err
+		if !strings.HasSuffix(cf, ".yaml") {
+			log.Println("[config] current file path: ", cf)
+
+			// replace orig config file ext with ".yaml"
+			ymlcf := cf[:len(cf)-len(path.Ext(cf))] + ".yaml"
+			if err := c.WriteYaml(ymlcf); err != nil {
+				return err
+			}
+			return fmt.Errorf("config file converted and written to: %s", ymlcf)
 		}
-		return fmt.Errorf("ERROR: Config file converted and written to: %s", ymlcf)
+		return fmt.Errorf("config file is already yaml: %s", cf)
 	}
 
 	if err := detectDiskStat(c.DownloadDirectory); err != nil {

server/server_api.go

@@ -230,7 +230,7 @@ func (s *Server) apiConfigure(data []byte) error {
 		// now it's safe to save the configure
 		s.engineConfig.SyncViper(c)
 		s.engineConfig = &c
-		if err := s.engineConfig.WriteYaml(); err != nil {
+		if err := s.engineConfig.WriteDefault(); err != nil {
 			return err
 		}
 		log.Printf("[api] config saved")