bbin icon indicating copy to clipboard operation
bbin copied to clipboard

No need to wrap git binary with process/exec

Open jeroenvandijk opened this issue 3 years ago • 3 comments

Like #46 scripts installed from a git dependency don't have to be called via process/exec. Instead, they can be called directly in combination with babashka.deps/add-deps.

The time savings depend on your local hardware, but should be something like:

time bb -e 'nil'
bb -e 'nil'  0.02s user 0.01s system 83% cpu 0.036 total

(Some background in a recent discussion around Carve)

jeroenvandijk avatar Jan 26 '23 16:01 jeroenvandijk

This may need a change in bb, allowing you to call bb --deps-root ... programmatically instead of from the CLI, for projects that are composed of multiple dependencies (like carve)

borkdude avatar Jan 26 '23 16:01 borkdude

Or perhaps adding the project as :local/root works...

borkdude avatar Jan 26 '23 16:01 borkdude

I think it works already, but need to fix some tests

jeroenvandijk avatar Jan 26 '23 16:01 jeroenvandijk