@isak Status should be a BList
of BString
, not a JSON-ized value
@borkdude nice, it works now with that fix, thanks!
Btw, I noticed the process used by the pod doesn't get killed when the script finishes on windows unless unload-pod
is called. Is that by design?
that's not by design. the process should be killed, since there is a shutdown hook that will destroy the process.
@isak https://github.com/babashka/babashka.pods/blob/dd46020efcc39404a105e6d9323b07ab2d0fffc8/src/babashka/pods/impl.clj#L304
Maybe for some reason this doesn't work on windows?
Hmm, it looks like the destroy
function expects a pod-id
, but is sent a pod
Hmm, you're right, this may be a bug...
Maybe other operating systems destroy subprocesses automatically, and windows doesn't. (I have noticed windows does not via .NET (unless you make some special calls, for example)
yeah, could be the case. I'll fix
I pushed a fix. You can find a new binary in the Appveyor build in a few minutes.
https://ci.appveyor.com/project/borkdude/babashka/builds/35772182
hmm, wait a minute, I broke something
@isak This one should work: https://ci.appveyor.com/project/borkdude/babashka/builds/35772827/artifacts
@borkdude that worked, thanks!
woohoo! thanks for spotting the mistake
I added support for transit+json payloads in babashka pods on master. This can be useful for sending over bytes, dates, etc.