PenaDevops/delve
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

pkg/terminal: support more editors with edit command (#3506)

Browse Source 
The current implementation assumes a specific command line interface
for opening an editor at a specific location within the source file.
This patch updates for different formats, namely the one used by the
editor 'hx'.
This commit is contained in:
Derek Parker 2023-09-20 00:25:48 -07:00 committed by GitHub
parent db7e60aef3
commit 4d30cd461b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 21 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
pkg/terminal/command.go
View File

@ -1914,14 +1914,21 @@ func tracepoint(t *Term, ctx callContext, args string) error {
return err
}
func runEditor(args ...string) error {
func getEditorName() (string, error) {
var editor string
if editor = os.Getenv("DELVE_EDITOR"); editor == "" {
if editor = os.Getenv("EDITOR"); editor == "" {
return fmt.Errorf("Neither DELVE_EDITOR or EDITOR is set")
return "", fmt.Errorf("Neither DELVE_EDITOR or EDITOR is set")
}
}
return editor, nil
}
func runEditor(args ...string) error {
editor, err := getEditorName()
if err != nil {
return err
}
cmd := exec.Command(editor, args...)
cmd.Stdin = os.Stdin
cmd.Stdout = os.Stdout
@ -1934,7 +1941,18 @@ func edit(t *Term, ctx callContext, args string) error {
if err != nil {
return err
}
editor, err := getEditorName()
if err != nil {
return err
}
switch editor {
case "hx":
return runEditor(fmt.Sprintf("%s:%d", file, lineno))
case "vi", "vim", "nvim":
return runEditor(fmt.Sprintf("+%d", lineno), file)
default:
return runEditor(fmt.Sprintf("+%d", lineno), file)
}
}
func watchpoint(t *Term, ctx callContext, args string) error {