Welcome! Please see the About page for a little more info on how this works.
I think that the following line might cause the problem on windows: https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/repl.cljc#L713
For file param e.g. "src\duct3\client.cljs"
(str "file://" (.getAbsolutePath file))
evaluates on windows as: "file://C:\Projects\Playground\duct3\src\duct3\client.cljs"
which is not legal file Url (https://en.wikipedia.org/wiki/File_URI_scheme#Windows)
and final result is: java.net.UnknownHostException (java treat that URL as FTP address).
Comment made by: vojko
Permalink to mentioned line: https://github.com/clojure/clojurescript/blob/98656d305e6447c62e36849ee615532d53211fdd/src/main/clojure/cljs/repl.cljc#L736
Added different prefix for windows platform.
Attached patch.
Comment made by: admin
Please don't mark issues resolved - only project admins like dnolen or mfikes should do that.
Comment made by: mfikes
I wonder if the {{file://}} string construct is necessary.
If not, then a simpler patch would involve replacing the expression
(ana/analyze-file (str "file://" (.getAbsolutePath file)) opts)
with
(ana/analyze-file file opts)
I have tried WO "file://", as you have said, and it works perfectly. Thx!
Hey David, even though you marked this one as accepted, my take on the above was that I had suggested perhaps a cleaner way to Vojimir, he had tried it, and it worked for him. I think we'd be interested in Vojimir's cleaner patch.
Hi Mike, I have attached a new patch with your suggested approach.
CLJS-2155-2.patch passes CI (/)
CLJS-2155-2.patch added to Patch Tender (i)
