Fork me on GitHub
#cider
<
2018-01-21
>
bozhidar06:01:11

@donmullen I don’t get how this can be happening to begin with. None of the deps of cider-nrepl has changed recently and the only thing I see depending on 1.2 is nREPL itself.

dominicm08:01:49

This is actually a boot thing, because clojure hadn't been specified in boot.properties and build.boot

donmullen12:01:17

@dominicm - the strange thing was that I had specified clojure version in build.boot and boot.properties and was still getting that error. At any rate - is working with 16 stable.

dominicm09:01:12

@bozhidar we might not want to Mr Anderson in orchard, and instead have users do that themselves? So that they can load multiple versions of orchard.

bozhidar12:01:50

@dominicm This makes sense for cider-nrepl, but doesn’t make sense for people who’d want to use it directly.

bozhidar12:01:04

I’ll think about it, it’s not a big deal right now.

bozhidar12:01:45

I also think there was some issue with dynapath and MrAnderson. I think that’s the reason we had used MrAnderson on it in cider-nrepl.

bozhidar12:01:11

At any rate - at least now a lot of cider-nrepl is in orchard. I had a surprisingly productive day yesterday. Unfortunately now we’re left with the really hard work - solidifying a client API and documenting it. 😄

bozhidar12:01:11

There’s mode code to be extracted from cider-nrepl as well, it’d be nice to also extract some things from refactor-nrepl (//cc @benedek), but the foundation is here.

benedek18:01:09

@bozhidar i think ,classpath command is broken under java9 because of https://dev.clojure.org/jira/browse/CLASSPATH-8

benedek18:01:17

want me to file an issue against orhcard i guess, or?

bozhidar18:01:39

Yeah, sure. I’m guessing a lot of things are broken on Java 9, because of this. We use the classpath library at several places.