ClojureScript support?

[arichiardi]

Hello guys,
I am trying to include the new puget 0.9.2 in a ClojureScript project with no luck.
I tried to downgrade to 0.8.1 but still I am not able to see puget in my classpath.
It always throws the classic no namespace found printer.puger...bla bla .cjlc ...bla

My project.clj:

:dependencies [[org.clojure/clojure "1.7.0"]
                 [org.clojure/clojurescript "1.7.145"]
                 [reagent "0.5.1"]]

  :plugins [[lein-cljsbuild "1.1.0"]
            [lein-codox "0.9.0"]
            [lein-simpleton "1.4.0-SNAPSHOT"]]

The profile.clj is empty.

Very weird, what can it be? Does puget modify the classpath at all?

[greglook]

Unfortunately Puget is not yet cross-compiled to ClojureScript. As of the 0.9.x series its main dependency (fipp) is cross-compiled though, so in theory this should be doable. I haven't worked a lot in cljs though, so if you have any suggestions I'm all ears.

Mainly I think this would involve making most of the files .cljc and then adding reader conditionals around the Java-specific parts.

[arichiardi]

Yep, are there a lot of Java specific calls? I guess I can have a look at the code and try to contribute 😉

EDIT: I changed the name of the issue.

I see a lot of Java fiddling here and here.

[piecyk]

👍 for this, also will have a look, regarding the java-handlers i think we can drop it for cljs will try to have something working.

[arichiardi]

yep agree, putting here as reference http://dev.clojure.org/display/design/Reader+Conditionals

[greglook]

Yeah, some thoughts:

• We can drop dispatch/symbolic-reader since it's not very useful standalone.
• dispatch/inheritance-lookup is going to need to be reworked, but would be nice if the contract held across both clj and cljs. Now that I think about it, I don't even know how inheritance works in cljs. 😞
• The mvxcvi/arrangement library will need to be ported as well, but shouldn't be too hard.
• Calculating the sys-id and class name for unknown rendering is currently Java-specific.
• As pointed out, the java-handlers need to be dropped. We should support the #inst tag for js Dates though.
• Similarly, the clojure-handlers and clojure-interface-handlers will probably need to be reworked a bit.

[arichiardi]

Yes I notice arrangement, but as you said, there are a few lines of code to change in there. I am more curious about sys-id, what is the purpose? Skimming the code it looks like you're using it as id string in the span creation. It looks that in that when you check it against the class name...is it used for uniquely identify an instance?

[greglook]

The sys-id used to differentiate otherwise-identical instances of the same class. For Clojure's persistent collections and other EDN primitives, this is not necessary since they support value-based equality. However, if you want to tell two instances of ByteArrayInputStream (for example) from each other, you need to use the system-level hash-code. Hence the use in the unknown formatter.

The package.foo.Bar@123abc format is used by Java as the default toString implementation, so the formatter checks that it's not going to print the same thing twice in case the object doesn't implement a custom string format.

[arichiardi]

Yes I suspected that.

[venantius]

I am going to begin work on this.

[arichiardi]

@venantius great! Feel free to ping me here or on Slack for testing!

[greglook]

Cool, I've just started picking up Clojurescript myself! I've experimented a bit with cross-compiling in alphabase and it's pretty nice so far.

[venantius]

Reference WIP PR here: #30

[worrel]

hey, any movement on the CLJS support? looks like the upstream PR in clj-arrangement was merged, anything blocking this?