memoization of js-dependency-index and get-upstream-deps needs knobs

Description

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

Environment

None

Activity

Show:
Mike Fikes
November 20, 2017, 2:04 AM

Patch no longer applies.

Bruce Hauman
March 28, 2015, 9:43 PM

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

Bruce Hauman
March 28, 2015, 8:44 PM

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

David Nolen
March 28, 2015, 8:26 PM

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

Bruce Hauman
March 28, 2015, 8:22 PM

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

Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Unassigned

Reporter

David Nolen

Patch

Code