Fork me on GitHub
#protorepl
<
2016-04-06
>
phreed21:04:45

@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 ?

jasongilman21:04:12

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

phreed21:04:08

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

phreed21:04:26

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'

phreed21:04:00

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

phreed21:04:49

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

jasongilman21:04:08

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

jasongilman21:04:43

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

phreed21:04:59

The sample is actually missing a bit.

phreed21:04:32

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

jasongilman21:04:35

I'm taking a look now.

phreed22:04:31

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

jasongilman22:04:14

That's merged in now

jasongilman22:04:29

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

jasongilman22:04:50

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

jasongilman22:04:02

clojure.tools.namespace scans the classpath by default

jasongilman22:04:09

I wonder if something isn't setup the same way

jasongilman22:04:20

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

jasongilman22:04:00

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

jasongilman22:04:33

@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?

jasongilman22:04:21

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

phreed22:04:45

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)