Fork me on GitHub
#cljs-dev
<
2019-12-03
>
p-himik11:12:52

It seems that it's impossible to run a CLJS REPL on a host other than localhost without repeating the host twice: clj --main cljs.main -ro '{:host "192.168.1.220"}' -co '{:closure-defines {"clojure.browser.repl.HOST" "192.168.1.220"}}' --repl That's because of this line: https://github.com/clojure/clojurescript/blob/master/src/main/cljs/clojure/browser/repl.cljs#L31 Shouldn't the value of clojure.browser.repl.HOST be the same as :host in -ro at all times?

p-himik11:12:38

Also, it looks like the --host value should replace the :host value in -ro, but that doesn't happen. The option seems to be completely ignored.

dnolen14:12:47

they should be the same - that google define should be set by the browser REPL - if not - just a bug

dnolen14:12:18

note it won't work if you already compiled and then change the host

dnolen14:12:36

at least I don't think it will - you need a clean build if you change that value

p-himik14:12:42

Yep, did a clean build. Still works only with :closure-defines .

dnolen14:12:21

ok open a ticket - and a patch for the problem is welcome

dnolen14:12:10

I don't think we've done much testing of other domains as it's less common thing to do - other REPLs like Figwheel probably handle this better

p-himik14:12:35

I thought the issue tracker is off limits now, no? The page at https://clojurescript.org/community/reporting-issues says about describing an issue at http://ask.clojure.org to let the developers know. I can basically duplicate what I've written here if you want though.

Alex Miller (Clojure team)14:12:23

if you don't have jira access, and you're not going to work on the patch, then either someone with access should log or you can file a question on ask.clojure

👍 4
Alex Miller (Clojure team)14:12:52

if you want to work on the patch, then sign the CA, and submit a request to get a jira acct, and make a jira