`leiningen.cljsbuild.config/compiler-global-dirs` leaks invalid `:externs` and `:libs`

Open swannodette opened this issue 11 years ago • 3 comments

Trying to solve http://dev.clojure.org/jira/browse/CLJS-994 revealed this issue.

Mar 15 '15 15:03 swannodette

Yeah, that's a vestige of #95, which coped with the lack of a library distribution story back in the day. It'll take a bit of work to excise the cruft.

May 13 '15 12:05 cemerick

I hoped a quick fix would work (just removing the offending default configuration), but doing so breaks a ton of tests (written using midje, which I'm loathe to touch).

May 14 '15 15:05 cemerick

Looked at this again, but just couldn't bring myself to nuke the relevant (useless) tests from orbit that would allow the elimination of the closure-js hack. Not a good idea to do so on the way out, IMO. :-(

Aug 24 '15 14:08 cemerick