Not yet I think but it’s pretty easy to add this. Start adding doc keys to your vars and make an issue at the pods repo. Your also welcome to add this support but I’d be happy to do it

So here:

"vars" [{"name" "execute!"}

we could add "doc" "foobar"

and then we need to update this bit of code: https://github.com/babashka/pods/blob/de4c3610c9ef3879370d01b7202a9f3a9d056f6e/src/babashka/pods/impl.clj#L116 and several other places where this is used

Cool. Sounds good.

processes are derefable, so you’d just @(p/process ["aws" "some-command"])

or you can use (p/check) which will throw if the process returns a bad exit-code

correct! and you can also use (babashka.tasks/shell "aws some-command") which will do all of the above - it accepts similar options to bb process but as the first argument

Thanks. Although I find bb /shell a bit cumbersome and hard to use in repl ..(

what's the reason?

The main being that if a shell command fails (eg error or not known) I only get the exception and the error is printed in the Calva output which I never use. The second reason is that parsing the output is rather cumbersome compared to java sh

@U96LS78UV there is also babashka.process/sh which is similar to clojure.java.shell/sh but tokenizes the arguments for you

(process/sh "aws some-command") ;;=> {:out ... :err ... :exit ...}

and it doesn't throw non non-zero

you can also use :continue true in shell to prevent throwing

process/sh is also blocking

so it seems that's what you need in this context

Thanks for the tip with continue! This works ok

((juxt :out :err) (shell {:continue true :out :string :err :string} "aws some-command" ))

but I guess I prefer the process now : )

:)

But isn't clojure.java.shell/sh blocking as well??

yes. babashka.process/sh is like clojure.java.shell/sh (but it also tokenizes the string for you)

So why the hell clojure.java.shell/sh is not blocking the first aws command for me). This what I expected

huh

it doesn't?

perhaps the aws command itself spawns another process and returns early?

I guess this is the case.