Re: clojurescript browser repl possible regression

2012-12-05 Thread Herwig Hochleitner
One more strange fact: When you remove third-party, but run with the released closure-library, you don't see the warning, but repl still doesn't work. Thinking of it, I have run into that before. So my theory is, that the build with the plain goog.jar fails because its deps.js refers to

Re: clojurescript browser repl possible regression

2012-12-05 Thread Brent Millare
Ah finally, some progress. This is very relieving, I'm not that crazy it seems. It's enough for me to fix my cljs tools for now. Thank you. On Wednesday, December 5, 2012 8:35:26 AM UTC-5, Herwig Hochleitner wrote: I can reproduce this. When I replace lib/goog.jar with

Re: clojurescript browser repl possible regression

2012-12-04 Thread David Nolen
I believe this error is because you're not including the google closure third party jar as a dependency. David On Tue, Dec 4, 2012 at 2:07 AM, Brent Millare brent.mill...@gmail.comwrote: https://github.com/downloads/bmillare/dj.web.browser/minimal.tar.gz @Mimmo, I don't use cljsbuild as

Re: clojurescript browser repl possible regression

2012-12-04 Thread Herwig Hochleitner
Am 04.12.2012 09:41 schrieb David Nolen dnolen.li...@gmail.com: I believe this error is because you're not including the google closure third party jar as a dependency. Yes, this is http://dev.clojure.org/jira/browse/CLJS-418 -- You received this message because you are subscribed to the

Re: clojurescript browser repl possible regression

2012-12-04 Thread Brent Millare
After adding third_party jar as a dependency to classpath (by copying google-closure-library-third-party-0.0-2029.jar to clojurescript/lib, as it should automatically be picked up by the ./script/repl script) I still get the clojure is not defined error. The other error doesn't appear. I also

Re: clojurescript browser repl possible regression

2012-12-04 Thread Herwig Hochleitner
I managed to get your example to work by copying the third-party jar into lib/ One thing I ran into: Do a `rm .repl/ out/ -r` between runs, especially when compiles have failed. -- You received this message because you are subscribed to the Google Groups Clojure group. To post to this group,

Re: clojurescript browser repl possible regression

2012-12-04 Thread Brent Millare
Strange, so 'rm .repl /out/ -r' is important but only so I see the goog.async.Deferred error consistently now. Starting from scratch: git clone git://github.com/clojure/clojurescript.git cd clojurescript ./script/bootstrap cp

clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
Using clojurescript 3842d3f9e0d68853077117a919e93e169079 Trying to do the simplest case of a clojurescript browser repl: running clojurescript/script/repl, then Taken straight from the documentation: (require '[cljs.repl :as repl])(require '[cljs.repl.browser :as browser]) ;; require the

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
I've already tried both ways. Creating the html file with the script tag with the simple code you showed, and just simply trying to connect directly with http://localhost:9000/repl. In both cases, the same error message shows. Note that the error message isn't from my source code, but rather

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
Don't call your file repl.* this has special meaning - it's the file that's meant to be loaded into the cross page iframe. On Monday, December 3, 2012, Brent Millare wrote: I've already tried both ways. Creating the html file with the script tag with the simple code you showed, and just simply

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
The file with the clojurescript and call to connect isn't named repl, its named view.html. repl is the name of the response from the call (repl/connect http://localhost:9000/repl;) On Monday, December 3, 2012 3:29:13 PM UTC-5, David Nolen wrote: Don't call your file repl.* this has special

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
It sounds like you are trying to navigate to http://localhost:9000/replthough right? You should be navigating to http://localhost:9000/ On Mon, Dec 3, 2012 at 8:51 PM, Brent Millare brent.mill...@gmail.comwrote: The file with the clojurescript and call to connect isn't named repl, its named

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
At the moment, all I am doing is dragging and dropping the html file into the browser, which contains the compiled clojurescript (js) which looks like this view.html ... compiled cljs stuff ... body scriptfoo.main();/script /body ... Where foo.cljs is: (ns foo (:require

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
file:// urls don't work anymore due to changes in the Google Closure Library. You need to point your browser to http://localhost:9000/, by default it looks for index.html. David On Mon, Dec 3, 2012 at 10:05 PM, Brent Millare brent.mill...@gmail.comwrote: At the moment, all I am doing is

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
Still getting the same error regardless. 1. creating index.html (with compiled cljs code and call to connect) and putting in clojurescript directory 2. starting up clojurescript clojure repl with ./script/repl 3. Running the clojurescript repl with: (require '[cljs.repl :as repl]) (require

Re: clojurescript browser repl possible regression

2012-12-03 Thread Mimmo Cosenza
Have you tried using lein-cljsbuild 0.2.9 plugin without specifying any CLJS version in the :dependencies? mimmo On Dec 4, 2012, at 1:02 AM, Brent Millare brent.mill...@gmail.com wrote: Still getting the same error regardless. 1. creating index.html (with compiled cljs code and call to

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
Please create a minimal project that demonstrates the issue for you, then we can try to run that. On Tue, Dec 4, 2012 at 12:02 AM, Brent Millare brent.mill...@gmail.comwrote: Still getting the same error regardless. 1. creating index.html (with compiled cljs code and call to connect) and

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
https://github.com/downloads/bmillare/dj.web.browser/minimal.tar.gz @Mimmo, I don't use cljsbuild as thats mainly for automation that doesn't fit my use cases. Instead I am using the github clojurescript docs for this reference project. Normally, I use my own build tools. I created a minimal