search

denoland / deno_bindgen

Write high-level Deno FFI libraries in Rust.
MIT License
287 stars 28 forks source link

add `--nocache` option #59

Closed mwcz closed 2 years ago

mwcz commented 2 years ago

related to #57

This adds a --nocache option which can be used alongside --release to disable caching. As far as I can tell, caching is already disabled for development builds, which is why this fix doesn't apply to #57.

I'm using deno_bindgen to build a couple of crates that are painfully slow in debug mode, so I always build them with deno_bindgen --release. The problem was that caching kept my changes from showing up, so I'd have to manually clear the cache before every build.

This solved my problem, but I'm also curious about what the benefit is of caching local artifacts at all. Network, sure, but if disabling local artifact caching entirely is a good option, this PR is unnecessary.

Thanks for deno_bindgen, it's been awesome, and thanks for humoring me with this PR. :upside_down_face:

littledivy commented 2 years ago

Thanks for the PR!

This solved my problem, but I'm also curious about what the benefit is of caching local artifacts at all. Network, sure, but if disabling local artifact caching entirely is a good option, this PR is unnecessary.

+1 Let's do it this way instead of adding a flag. Only network artificts should be cached.