Share your thoughts in the 2024 State of Clojure Survey!

Welcome! Please see the About page for a little more info on how this works.

0 votes
in ClojureScript by

knobs should be exposed for more dynamic compilation environments like Figwheel which may desire to add dependencies to the classpath on the fly.

9 Answers

0 votes
by

Comment made by: bhauman

A patch that caches upstream dependencies in the compiler env.

0 votes
by

Comment made by: bhauman

Actually I'm going to submit another patch that includes the memoize calls in js-deps.

0 votes
by

Comment made by: bhauman

New patch that moves {{cljs.js-deps}} memoization to current * } as well as {{get-upstream-deps}}.

Unfortunately there is a circular dep between {{cljs.env}} and {{cljs.js-deps}}, if we want to cache in * }. I overcame this with a resolve.

Compile performance is either completely unchanged or slightly improved based on several test runs.

0 votes
by

Comment made by: bhauman

Hold off on this. Its not behaving as expected. Doesn't seem to be caching in certain situations.

0 votes
by

Comment made by: dnolen

Thanks for the update. This will definitely not land until after the pending REPL/piggieback release anyhow.

0 votes
by

Comment made by: bhauman

Yeah there is an obvious bug and a subtle one. Hopefully will finish it up soonish.

0 votes
by

Comment made by: bhauman

Alright, this latest patch works. There was a subtle memoizing nil value bug.

0 votes
by

Comment made by: mfikes

Patch no longer applies.

0 votes
by
Reference: https://clojure.atlassian.net/browse/CLJS-1141 (reported by dnolen)
...