right, but at least you can add a notice in the release.

@dnolen: Tried deraen’s patch on a small personal project, but unfortunately having some problems when requiring cljs.js. Here is a minimal project to reproduce the problem: https://github.com/mneise/cljs-1437-test

Tried it with and without the patch and only got those errors once I applied the patch.

Strange. Only change to a cljc file is adding one function. Most of changes on closure.clj.

Probably related to this line: https://github.com/clojure/clojurescript/blob/master/src/main/cljs/cljs/js.cljs#L23

If you compare the dependencies included in cljs_deps.js you can see that there are two missing: cljs.core$macros and clojure.walk.

I think this is related to changing how dependencies are added during build process. Before, files were compiled and the compiled JS was checked for goog.require and goog.provide and then those CLJS ns were compiled.

After the change, only ns form is read from cljs files.

@maria thanks for the check! I recall that cljs.js needs special deps handling same as node.js

@juhoteperi: maybe this isn't there any longer? ^

@dnolen: There's different kind of special handling for cljs.js. get-compiled-cljs and cljs-source-for-namespace do special things.

get-compiled-js was used in cljs-dependencies and then dependencies were read from the compiled JS

@juhoteperi: right but it seems something is getting skipped?

@dnolen: Well the new code doesn't compile namespaces so it doesn't see dependecies which are generated from code using js/goog.require

Hrm not sure I understand yet. But I'll probably be trying this case out myself later today.

http://dev.clojure.org/jira/browse/CLJS-1469 up for review

@dnolen: ^ you were right yesterday: "we probably need to setOptions somewhere". It was initOptions

@bensu awesome, thanks!

@dnolen: let me know if the test is what you had in mind

patch looks great, thanks

testing now

worked pushed to master

great!

Then, I only have one other in the pipeline (re: node shim) which needs some peer testing

could use some help looking into http://dev.clojure.org/jira/browse/CLJS-1472

it’s the only one other than the dependencies thing I really care about getting into the next release

ok

@dnolen: the patch mentions files that look like this: BCE03FE.js I'm familiar with them but can't find where those names are created, or pin point why.

do you have any pointers?

@bensu: Sorry to jump in, but I think those names are generated here: https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/closure.clj#L1235

@maria: thanks!

I'm actually looking at one of your patches