Skip to content

Commit

Permalink
fix: dont handle LookPath in Command wrapper
Browse files Browse the repository at this point in the history
  • Loading branch information
xoxys committed May 17, 2024
1 parent 119641f commit e5e67d4
Showing 1 changed file with 6 additions and 12 deletions.
18 changes: 6 additions & 12 deletions exec/command.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,23 +34,17 @@ func (c *Cmd) Run() error {
return c.Wait()
}

// Command creates a new Cmd struct with the given name and arguments. It looks up the
// absolute path of the executable using execabs.LookPath, and sets up the Cmd with
// the necessary environment and output streams. The Cmd is configured to trace
// the command execution by setting Trace to true and TraceWriter to os.Stdout.
func Command(name string, arg ...string) (*Cmd, error) {
abs, err := execabs.LookPath(name)
if err != nil {
return nil, fmt.Errorf("could not find executable %q: %w", name, err)
}

// Command creates a new Cmd with the given name and arguments. The Cmd is configured
// with Trace set to true and TraceWriter set to os.Stdout. The Cmd's Env is set
// to the current environment.
func Command(name string, arg ...string) *Cmd {
cmd := &Cmd{
Cmd: execabs.Command(abs, arg...),
Cmd: execabs.Command(name, arg...),
Trace: true,
TraceWriter: os.Stdout,
}

cmd.Env = os.Environ()

return cmd, nil
return cmd
}

0 comments on commit e5e67d4

Please sign in to comment.