Merge pull request #18 from dbalan/args_editor

Bug fix: Editor spec can have arguments.

cmd/init.go

@@ -33,6 +33,7 @@ import (
 	"fmt"
 	"io/ioutil"
 	"os"
+	"strings"
 
 	"github.com/spf13/cobra"
 	"gopkg.in/yaml.v2"
@@ -81,14 +82,21 @@ var initCmd = &cobra.Command{
 		}
 
 		if eBin != "" {
-			path, err := which(eBin)
+			// eBin might be editor + args
+			// for e.g emacsclient -t
+			actualBinPath := strings.Split(eBin, " ")[0]
+			path, err := which(actualBinPath)
 			errorGuard(err, "No such editor found!")
 
 			// expanded path
 			if path != eBin {
 				fmt.Printf("Using %s as the absoulte path to editor\n", Green(path))
 			}
-			config.EBin = path
+			if actualBinPath != eBin {
+				config.EBin = path + " " + strings.Join(strings.Split(eBin, " ")[1:], " ")
+			} else {
+				config.EBin = path
+			}
 		} else {
 			errorGuard(fmt.Errorf("empty input"), "no editor sepcified")
 		}

cmd/utils.go

@@ -108,7 +108,12 @@ func getDataStore() *pipetdata.DataStore {
 // call external editor to edit the snippet.
 func editSnippet(fn string) error {
 	editorBin := viper.Get("editor_binary").(string)
-	cmd := exec.Command(editorBin, fn)
+
+	// editor_binary is of form bin args
+	parsed := strings.Split(editorBin, " ")
+	args := parsed[1:]
+	args = append(args, fn)
+	cmd := exec.Command(parsed[0], args...)
 	cmd.Stdin = os.Stdin
 	cmd.Stdout = os.Stdout
 	cmd.Stderr = os.Stderr