Explodes when used with CLJS 2173

[kenrestivo]

I tried to upgrade to CLJS 2173 because that is required by the latest OM 0.5.1 which is required because it has stuff I wanted to try.

Weasel breaks with CLJS 2173.

Specifically, a form like this:

(weasel.repl.websocket/repl-env :ip "0.0.0.0" :port 9001)

breaks like this:

java.lang.NullPointerException: null
 at clojure.core$complement$fn__4048.invoke (core.clj:1357)
    clojure.core$filter$fn__4226.invoke (core.clj:2529)
    clojure.lang.LazySeq.sval (LazySeq.java:42)
    clojure.lang.LazySeq.seq (LazySeq.java:60)
    clojure.lang.RT.seq (RT.java:484)
    clojure.core$seq.invoke (core.clj:133)
    clojure.core$map$fn__4207.invoke (core.clj:2479)
    clojure.lang.LazySeq.sval (LazySeq.java:42)
    clojure.lang.LazySeq.seq (LazySeq.java:60)
    clojure.lang.RT.seq (RT.java:484)
    clojure.core$seq.invoke (core.clj:133)
    clojure.core$filter$fn__4226.invoke (core.clj:2523)
    clojure.lang.LazySeq.sval (LazySeq.java:42)
    clojure.lang.LazySeq.seq (LazySeq.java:60)
    clojure.lang.RT.seq (RT.java:484)
    clojure.core$seq.invoke (core.clj:133)
    clojure.core$set.invoke (core.clj:3712)
    cljs.closure$cljs_dependencies.invoke (closure.clj:666)

It seems cljs-dependencies itself is broken in CLJS 2173. Not only does this fail as above with an NPE:

(cljsc/cljs-dependencies {} ["weasel.repl"])

But this also fails with the same NPE:

  (cljsc/cljs-dependencies {} ["goog.string" "cljs.core"])

Backing off to CLJS 2156 makes weasel work again.

This is about where I had to pull myself out of the rabbit hole.

[kenrestivo]

I had my src path set to something different, figured that might have something to do with it, but changing it back to src does not make the problem go away.

[kenrestivo]

Minimal breaking test case is this:

diff --git a/weasel-example/project.clj b/weasel-example/project.clj
index 71cdc70..8b55fdc 100644
--- a/weasel-example/project.clj
+++ b/weasel-example/project.clj
@@ -6,7 +6,7 @@
             :distribution :repo}

   :dependencies [[org.clojure/clojure "1.5.1"]
-                 [org.clojure/clojurescript "0.0-2138"]
+                 [org.clojure/clojurescript "0.0-2173"]
                  [weasel "0.1.0"]]

Then fire up a repl in the example project, and execute:

  (require 'weasel.repl.websocket)
  (require 'cemerick.piggieback)
  (cemerick.piggieback/cljs-repl :repl-env
                                 (weasel.repl.websocket/repl-env :ip "0.0.0.0" :port 9001))

and enjoy the fireworks.

[tomjakubowski]

Hey, thanks for the thorough bug report! This issue looks similar to this one in Austin: cemerick/austin#55. There's a workaround I can patch in (and that seems to work okay), but I'd like to give it a little more thought first. BTW, weasel 0.1.0 should work as-is with Clojurescript master because of http://dev.clojure.org/jira/browse/CLJS-770.

[cemerick]

@tomjakubowski Note that if you're using a ClojureScript compiler environment, you will need to change your usage of cljs.env/default-compiler-env so that users' library-related configuration is passed through (e.g. :libs, :foreign-libs, etc).