Skip to content

Commit a0ee177

Browse files
committed
fix: don't check for executable in path for dryRun
1 parent 63ee54c commit a0ee177

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

cmd/cli/commands/launch.go

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -207,15 +207,17 @@ func launchContainerApp(cmd *cobra.Command, ca containerApp, baseURL string, ima
207207

208208
// launchHostApp launches a native host app executable.
209209
func launchHostApp(cmd *cobra.Command, bin string, baseURL string, cli hostApp, appArgs []string, dryRun bool) error {
210-
if _, err := exec.LookPath(bin); err != nil {
211-
cmd.PrintErrf("%q executable not found in PATH.\n", bin)
212-
if cli.envFn != nil {
213-
cmd.PrintErrf("Configure your app to use:\n")
214-
for _, e := range cli.envFn(baseURL) {
215-
cmd.PrintErrf(" %s\n", e)
210+
if !dryRun {
211+
if _, err := exec.LookPath(bin); err != nil {
212+
cmd.PrintErrf("%q executable not found in PATH.\n", bin)
213+
if cli.envFn != nil {
214+
cmd.PrintErrf("Configure your app to use:\n")
215+
for _, e := range cli.envFn(baseURL) {
216+
cmd.PrintErrf(" %s\n", e)
217+
}
216218
}
219+
return fmt.Errorf("%s not found; please install it and re-run", bin)
217220
}
218-
return fmt.Errorf("%s not found; please install it and re-run", bin)
219221
}
220222

221223
if cli.envFn == nil {

0 commit comments

Comments
 (0)