@rahul080327 @cap10morgan @kipz I want to release bb but the docker step is failing. https://app.circleci.com/pipelines/github/babashka/babashka/5241/workflows/3fbeaa62-99d7-4fcb-bb83-887d62091fc5/jobs/25741
I'll take a look
we fixed it 😄
> error: failed to solve: failed to compute cache key: "/bb": not found
@kipz has joined the channel
@borkdude i think its https://github.com/babashka/babashka/blob/master/.circleci/script/docker#L52
That isn't new though, right?
this is first time we are doing a non snapshot release
after the changes
would you remove it @borkdude?
oh right.
should I remove rm -f bb?
yeah i think so
ok, will try
it was in a different place before, got messed up in the complexity
ah yes
what happens if we just don't rm? nothing I guess?
yeah should be fine, build context is more by sizeof the binary
in snapshot builds
what's this then?
for p in "${platforms[@]}"; do
rm -rf $p
donewhy even remove that?
yeah none of the removes are making much of a difference
oh haha, I didn't know Github would render emoji: https://github.com/babashka/babashka/commit/530177eefde6c021d2a8759a31adc90cac9090cb
that above remove is to remove the linux-amd64 and linux-arm64 dirs
just has the binary
ok, whatevs :)
yeah rm in a ephemeral CI is pretty much redundant
it worked!
bananadance
thanks all
weekly reminder why these scripts needs to be in clojure 😕
:)
feel free to take a stab at it, whoever wants to do it