[CLJS-770] Include :js-dependency-index in result of `(env/default-compiler-env)` Created: 18/Feb/14 Updated: 23/Feb/14 Resolved: 23/Feb/14
It looks like the change from
I'm mucking with this in conjunction w/
|Comment by David Nolen [ 18/Feb/14 8:11 AM ]|
I'm ok with pulling this stuff into it's own namespace.
|Comment by Chas Emerick [ 23/Feb/14 9:35 AM ]|
Attached patch refactors goog-style JS dependency stuff into cljs.js-deps, and changes cljs.env/default-compiler-env to populate :js-dependency-index based on the provided set of options.
I didn't move the externs stuff, seemed like it should stay in cljs.closure.
Note that this doesn't change the fact that CLJS compiler API consumers will need to change: cljs.env/default-compiler-env now takes a map argument of options, which needs to be provided if such consumers need to have their e.g. :libs, :foreign-libs options picked up. The advantage is default-compiler-env again does the job it's intended to do. The refactoring / slimming of cljs.closure is a nice side effect, tho.
|Comment by David Nolen [ 23/Feb/14 4:24 PM ]|