babashka

https://github.com/babashka/babashka. Also see #sci, #nbb and #babashka-circleci-builds .
borkdude 2020-10-15T07:58:27.428800Z

@isak Status should be a BList of BString, not a JSON-ized value

isak 2020-10-15T14:54:37.431500Z

@borkdude nice, it works now with that fix, thanks!

1
isak 2020-10-15T14:55:53.432600Z

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?

borkdude 2020-10-15T14:56:39.433300Z

that's not by design. the process should be killed, since there is a shutdown hook that will destroy the process.

1
borkdude 2020-10-15T14:57:41.433800Z

Maybe for some reason this doesn't work on windows?

isak 2020-10-15T14:59:57.434100Z

Hmm, it looks like the destroy function expects a pod-id, but is sent a pod

borkdude 2020-10-15T15:00:42.434300Z

Hmm, you're right, this may be a bug...

isak 2020-10-15T15:00:47.434500Z

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)

borkdude 2020-10-15T15:01:56.434700Z

yeah, could be the case. I'll fix

1
borkdude 2020-10-15T15:26:52.435Z

I pushed a fix. You can find a new binary in the Appveyor build in a few minutes.

borkdude 2020-10-15T15:33:37.435400Z

hmm, wait a minute, I broke something

1
borkdude 2020-10-15T16:03:09.435700Z

@isak This one should work: https://ci.appveyor.com/project/borkdude/babashka/builds/35772827/artifacts

isak 2020-10-15T16:14:59.435900Z

@borkdude that worked, thanks!

borkdude 2020-10-15T16:15:41.436100Z

woohoo! thanks for spotting the mistake

borkdude 2020-10-15T16:31:47.436700Z

I added support for transit+json payloads in babashka pods on master. This can be useful for sending over bytes, dates, etc.

1
🍾 1
😎 1