Re: [ClojureScript] Allow more google closure options to be set

2014-01-02 Thread David Nolen
Looks OK to me. JIRA Ticket + patch please, thanks! David On Thu, Jan 2, 2014 at 8:38 PM, Ivan Willig wrote: > Hi list, > > While I was messing around with ClojureScript and nodejs this week, I > noticed that the Google Closure compiler complains when JavaScript > reserved words are used. New

Re: [ClojureScript] :output-file vs :output-to (minor)

2014-01-02 Thread David Nolen
I wasn't even aware of :output-file to be honest. Ticket + patch welcome. Thanks, David On Thu, Jan 2, 2014 at 9:39 PM, Timothy Pratley wrote: > ./bin/cljsc hello.cljs '{:output-file "foo.js"}' > > fails > > The problem/fix is straightforward: > > (apply add-dependencies all-opts > - (conc

[ClojureScript] :output-file vs :output-to (minor)

2014-01-02 Thread Timothy Pratley
./bin/cljsc hello.cljs '{:output-file "foo.js"}' fails The problem/fix is straightforward: (apply add-dependencies all-opts - (concat (if (coll? compiled) compiled [compiled]) + (concat (if (satisfies? IJavaScript compiled) [compiled] compiled) However I'm not entirely sure what the d

[ClojureScript] Allow more google closure options to be set

2014-01-02 Thread Ivan Willig
Hi list, While I was messing around with ClojureScript and nodejs this week, I noticed that the Google Closure compiler complains when JavaScript reserved words are used. Newer versions of JavaScript engines allow reserved words (like "static", "class") to be used as object properties. The Closur

[ClojureScript] Re: ANN: Om, a ClojureScript binding to Facebook's React

2014-01-02 Thread David Pidcock
Very nice. Relatively easy to follow. Basically it's re-implementing the drag-list behaviour in React. I had thought to re-use some existing code, provided I could inform React that the DOM was changing. (This is the approach used by the example in the fiddler, albeit with jquery instead of

Re: [ClojureScript] Improving cljs.closure :none vs other

2014-01-02 Thread David Nolen
On Thu, Jan 2, 2014 at 4:31 PM, Thomas Heller wrote: > Now if I could only convince everyone that closure JSModules should be > used always since they are way more flexible. > > @david: I'll look into creating a patch but unfortunately I realized that > cljs.closure is missing the extra part that

Re: [ClojureScript] Improving cljs.closure :none vs other

2014-01-02 Thread Thomas Heller
Now if I could only convince everyone that closure JSModules should be used always since they are way more flexible. @david: I'll look into creating a patch but unfortunately I realized that cljs.closure is missing the extra part that you currently don't define "main" namespaces and everything

Re: [ClojureScript] Improving cljs.closure :none vs other

2014-01-02 Thread Mimmo Cosenza
Nice! At the moment I use Enlive to substitute/add the

Re: [ClojureScript] require refer do not give errors

2014-01-02 Thread David Nolen
Thanks! On Thu, Jan 2, 2014 at 3:44 AM, Timothy Pratley wrote: > Patch is up for this: > http://dev.clojure.org/jira/browse/CLJS-615 > Additional details in the ticket description. > > Regards, > Timothy > > -- > Note that posts from new members are moderated - please be patient with > your fir

Re: [ClojureScript] Improving cljs.closure :none vs other

2014-01-02 Thread David Nolen
Feel free to open an enhancement ticket in JIRA w/ patch. Thanks! David On Thu, Jan 2, 2014 at 8:24 AM, Thomas Heller wrote: > Howdy, > > I wrote a build library https://github.com/thheller/shadow-build which > replaces cljs.closure and I'd like the see some of the stuff I did flow > back into

[ClojureScript] Improving cljs.closure :none vs other

2014-01-02 Thread Thomas Heller
Howdy, I wrote a build library https://github.com/thheller/shadow-build which replaces cljs.closure and I'd like the see some of the stuff I did flow back into core cljs. I wanted to start discussing some of the stuff I did since I feel the current situation is "bad" for ClojureScript in gener

Re: [ClojureScript] require refer do not give errors

2014-01-02 Thread Timothy Pratley
Patch is up for this: http://dev.clojure.org/jira/browse/CLJS-615 Additional details in the ticket description. Regards, Timothy -- Note that posts from new members are moderated - please be patient with your first post. --- You received this message because you are subscribed to the Google Gr