aboutsummaryrefslogtreecommitdiffstats
path: root/commands
diff options
context:
space:
mode:
Diffstat (limited to 'commands')
-rw-r--r--commands/commands.go2
-rw-r--r--commands/input/input.go104
-rw-r--r--commands/new.go75
-rw-r--r--commands/pull.go12
-rw-r--r--commands/push.go12
5 files changed, 191 insertions, 14 deletions
diff --git a/commands/commands.go b/commands/commands.go
index 4f892fdd..64b2cbc1 100644
--- a/commands/commands.go
+++ b/commands/commands.go
@@ -6,6 +6,7 @@ import (
)
const bugsRefPattern = "refs/bugs/*"
+const messageFilename = "BUG_MESSAGE_EDITMSG"
// Command represents the definition of a single command.
type Command struct {
@@ -23,6 +24,7 @@ func (cmd *Command) Run(repo repository.Repo, args []string) error {
// CommandMap defines all of the available (sub)commands.
var CommandMap = map[string]*Command{
+ "new": newCmd,
"pull": pullCmd,
"push": pushCmd,
diff --git a/commands/input/input.go b/commands/input/input.go
new file mode 100644
index 00000000..531a4386
--- /dev/null
+++ b/commands/input/input.go
@@ -0,0 +1,104 @@
+// Taken from the git-appraise project
+
+package input
+
+import (
+ "bufio"
+ "bytes"
+ "fmt"
+ "github.com/MichaelMure/git-bug/repository"
+ "io/ioutil"
+ "os"
+ "os/exec"
+)
+
+// LaunchEditor launches the default editor configured for the given repo. This
+// method blocks until the editor command has returned.
+//
+// The specified filename should be a temporary file and provided as a relative path
+// from the repo (e.g. "FILENAME" will be converted to ".git/FILENAME"). This file
+// will be deleted after the editor is closed and its contents have been read.
+//
+// This method returns the text that was read from the temporary file, or
+// an error if any step in the process failed.
+func LaunchEditor(repo repository.Repo, fileName string) (string, error) {
+ editor, err := repo.GetCoreEditor()
+ if err != nil {
+ return "", fmt.Errorf("Unable to detect default git editor: %v\n", err)
+ }
+
+ path := fmt.Sprintf("%s/.git/%s", repo.GetPath(), fileName)
+
+ cmd, err := startInlineCommand(editor, path)
+ if err != nil {
+ // Running the editor directly did not work. This might mean that
+ // the editor string is not a path to an executable, but rather
+ // a shell command (e.g. "emacsclient --tty"). As such, we'll try
+ // to run the command through bash, and if that fails, try with sh
+ args := []string{"-c", fmt.Sprintf("%s %q", editor, path)}
+ cmd, err = startInlineCommand("bash", args...)
+ if err != nil {
+ cmd, err = startInlineCommand("sh", args...)
+ }
+ }
+ if err != nil {
+ return "", fmt.Errorf("Unable to start editor: %v\n", err)
+ }
+
+ if err := cmd.Wait(); err != nil {
+ return "", fmt.Errorf("Editing finished with error: %v\n", err)
+ }
+
+ output, err := ioutil.ReadFile(path)
+ if err != nil {
+ os.Remove(path)
+ return "", fmt.Errorf("Error reading edited file: %v\n", err)
+ }
+ os.Remove(path)
+ return string(output), err
+}
+
+// FromFile loads and returns the contents of a given file. If - is passed
+// through, much like git, it will read from stdin. This can be piped data,
+// unless there is a tty in which case the user will be prompted to enter a
+// message.
+func FromFile(fileName string) (string, error) {
+ if fileName == "-" {
+ stat, err := os.Stdin.Stat()
+ if err != nil {
+ return "", fmt.Errorf("Error reading from stdin: %v\n", err)
+ }
+ if (stat.Mode() & os.ModeCharDevice) == 0 {
+ // There is no tty. This will allow us to read piped data instead.
+ output, err := ioutil.ReadAll(os.Stdin)
+ if err != nil {
+ return "", fmt.Errorf("Error reading from stdin: %v\n", err)
+ }
+ return string(output), err
+ }
+
+ fmt.Printf("(reading comment from standard input)\n")
+ var output bytes.Buffer
+ s := bufio.NewScanner(os.Stdin)
+ for s.Scan() {
+ output.Write(s.Bytes())
+ output.WriteRune('\n')
+ }
+ return output.String(), nil
+ }
+
+ output, err := ioutil.ReadFile(fileName)
+ if err != nil {
+ return "", fmt.Errorf("Error reading file: %v\n", err)
+ }
+ return string(output), err
+}
+
+func startInlineCommand(command string, args ...string) (*exec.Cmd, error) {
+ cmd := exec.Command(command, args...)
+ cmd.Stdin = os.Stdin
+ cmd.Stdout = os.Stdout
+ cmd.Stderr = os.Stderr
+ err := cmd.Start()
+ return cmd, err
+}
diff --git a/commands/new.go b/commands/new.go
new file mode 100644
index 00000000..0156477e
--- /dev/null
+++ b/commands/new.go
@@ -0,0 +1,75 @@
+package commands
+
+import (
+ "flag"
+ "fmt"
+ "github.com/MichaelMure/git-bug/bug"
+ "github.com/MichaelMure/git-bug/commands/input"
+ "github.com/MichaelMure/git-bug/repository"
+ "github.com/pkg/errors"
+)
+
+var newFlagSet = flag.NewFlagSet("new", flag.ExitOnError)
+
+var (
+ newMessageFile = newFlagSet.String("F", "", "Take the message from the given file. Use - to read the message from the standard input")
+ newMessage = newFlagSet.String("m", "", "Provide a message to describe the issue")
+)
+
+func newBug(repo repository.Repo, args []string) error {
+ newFlagSet.Parse(args)
+ args = newFlagSet.Args()
+
+ var err error
+
+ if len(args) == 0 {
+ return errors.New("No title provided")
+ }
+ if len(args) > 1 {
+ return errors.New("Only accepting one title is supported")
+ }
+
+ title := args[0]
+
+ if *newMessageFile != "" && *newMessage == "" {
+ *newMessage, err = input.FromFile(*newMessageFile)
+ if err != nil {
+ return err
+ }
+ }
+ if *newMessageFile == "" && *newMessage == "" {
+ *newMessage, err = input.LaunchEditor(repo, messageFilename)
+ if err != nil {
+ return err
+ }
+ }
+
+ // Note: this is very primitive for now
+ author, err := bug.GetUser(repo)
+ if err != nil {
+ return err
+ }
+
+ comment := bug.Comment{
+ Author: author,
+ Message: *newMessage,
+ }
+
+ bug := bug.Bug{
+ Title: title,
+ Comments: []bug.Comment{comment},
+ }
+
+ fmt.Println(bug)
+
+ return nil
+
+}
+
+var newCmd = &Command{
+ Usage: func(arg0 string) {
+ fmt.Printf("Usage: %s new <title> [<option>...]\n\nOptions:\n", arg0)
+ newFlagSet.PrintDefaults()
+ },
+ RunMethod: newBug,
+}
diff --git a/commands/pull.go b/commands/pull.go
index 0f6080d4..20009a00 100644
--- a/commands/pull.go
+++ b/commands/pull.go
@@ -1,14 +1,14 @@
-package commands
+package commands
import (
+ "errors"
"fmt"
"github.com/MichaelMure/git-bug/repository"
- "errors"
)
func pull(repo repository.Repo, args []string) error {
if len(args) > 1 {
- return errors.New("only pulling from one remote at a time is supported")
+ return errors.New("Only pulling from one remote at a time is supported")
}
remote := "origin"
@@ -27,7 +27,5 @@ var pullCmd = &Command{
Usage: func(arg0 string) {
fmt.Printf("Usage: %s pull [<remote>]\n", arg0)
},
- RunMethod: func(repo repository.Repo, args []string) error {
- return pull(repo, args)
- },
-} \ No newline at end of file
+ RunMethod: pull,
+}
diff --git a/commands/push.go b/commands/push.go
index 4465f561..9b808511 100644
--- a/commands/push.go
+++ b/commands/push.go
@@ -1,14 +1,14 @@
-package commands
+package commands
import (
+ "errors"
"fmt"
"github.com/MichaelMure/git-bug/repository"
- "errors"
)
func push(repo repository.Repo, args []string) error {
if len(args) > 1 {
- return errors.New("only pushing to one remote at a time is supported")
+ return errors.New("Only pushing to one remote at a time is supported")
}
remote := "origin"
@@ -27,7 +27,5 @@ var pushCmd = &Command{
Usage: func(arg0 string) {
fmt.Printf("Usage: %s push [<remote>]\n", arg0)
},
- RunMethod: func(repo repository.Repo, args []string) error {
- return push(repo, args)
- },
-} \ No newline at end of file
+ RunMethod: push,
+}