protorepl

phreed 2016-04-06T21:32:45.000052Z

@jasongilman: I am still having some difficulties getting the refresh to behave the same for boot as it does for leiningen. I believe the refresh is running https://github.com/jasongilman/proto-repl/blob/master/lib/proto-repl.coffee#L450 ?

jasongilman 2016-04-06T21:48:12.000054Z

That's right. What kind of behavior are you seeing?

phreed 2016-04-06T21:49:08.000055Z

It does not seem to reload the files associated with the loaded namespaces.

phreed 2016-04-06T21:50:26.000056Z

If I change a file, for example the user.clj as described in the demo.md and refersh the "Starting" message does not appear until I reload the file 'ctrl-, f'

phreed 2016-04-06T21:51:00.000057Z

This is related to the issue#5 that you posted a while back.

phreed 2016-04-06T21:51:49.000058Z

btw, While looking into this I found a couple of other small issues for which there is a pending pull request.

jasongilman 2016-04-06T21:52:08.000059Z

Is there an easy way to reproduce this so I can give it a try?

jasongilman 2016-04-06T21:52:43.000060Z

Thanks for the pull request. I'll take a look at that later.

phreed 2016-04-06T21:54:59.000063Z

The sample is actually missing a bit.

phreed 2016-04-06T21:55:32.000064Z

(defn start [] #_(println "Starting"))

jasongilman 2016-04-06T21:59:35.000065Z

I'm taking a look now.

phreed 2016-04-06T22:08:31.000066Z

There is a pull request on proto-repl-demo as well with some minor edits.

jasongilman 2016-04-06T22:20:14.000067Z

That's merged in now

jasongilman 2016-04-06T22:20:29.000068Z

It looks like there's some kind of known issue related to this https://github.com/boot-clj/boot/wiki/Repl-reloading

jasongilman 2016-04-06T22:20:50.000070Z

That's weird. I don't have to do that kind of thing with leiningen

jasongilman 2016-04-06T22:21:02.000071Z

clojure.tools.namespace scans the classpath by default

jasongilman 2016-04-06T22:21:09.000072Z

I wonder if something isn't setup the same way

jasongilman 2016-04-06T22:23:20.000073Z

(System/getProperty "java.class.path") => "/Users/jason/work/bin/boot" for me

jasongilman 2016-04-06T22:24:00.000074Z

With leiningen it returns much more including the src and dev dirs and the libraries that are available

jasongilman 2016-04-06T22:32:33.000075Z

@phreed: I've pushed changes to both Proto REPL and proto-repl-demo now. Can you give it a try and see if that resolves your issues?

jasongilman 2016-04-06T22:33:21.000076Z

If everything looks good I'll do a new release of Proto REPL.

phreed 2016-04-06T22:36:45.000077Z

I think the problem is that the classpath in the boot repl is not fully populated. (println (seq (.getURLs (java.lang.ClassLoader/getSystemClassLoader)))) or (use 'clojure.pprint) (-> (java.lang.ClassLoader/getSystemClassLoader) .getURLs seq pprint)