Fork me on GitHub
#nrepl
<
2018-11-19
>
bozhidar08:11:23

@pez Hmm, that’s odd. We didn’t really change anything on our end in this regard, so that’s likely a change that happened in lein.

bozhidar08:11:14

Actually the plugin is what generates the proper :repl-options, so you don’t have to do this manually.

bozhidar08:11:08

But the warning is actually about the lein middleware, not nREPL’s middleware.

bozhidar08:11:28

(lein middleware is simply a plugin that only transforms the project map)

bozhidar08:11:53

Not sure what we’re expected to do this here, but despite the working everything should work just fine.

pez11:11:43

OK. Checking that I understand which plugin you are referring to. If I remove [cider/cider-nrepl "0.18.0"] from the above profile, then leiningen stops complaining. Now I'll go read the links you provided. 😃

pez11:11:40

Alright, I can't make sense out of that patch. Anyway. I can ignore the warning, right?

pez11:11:14

I want to track down where from 0.2.13 -> 0.4.5 I stopped being able to decode the messages. Which version is it that is ”the same” as 0.2.13 now again?

bozhidar11:11:38

> Alright, I can’t make sense out of that patch. Anyway. I can ignore the warning, right?

bozhidar11:11:45

> I want to track down where from 0.2.13 -> 0.4.5 I stopped being able to decode the messages. Which version is it that is “the same” as 0.2.13 now again?

bozhidar11:11:16

Nothing with respect to message encoding/decoding has changed between 0.2.13 and 0.4.5.

bozhidar11:11:31

What exactly is the problem you’re encountering?

pez13:11:35

I am trying to figure out what exactly is the problem, but what happens is that lots of the decoding starts to fail and also my nrepl client goes into an infinite loop endlessly failing decoding. It doesn't necessarily have to do with the encoding of the new nrepl. The client in Calva is a bit too naive and probably very sensitive to the order in which responses come back. It is time to fix that, and maybe I should before investigating the problems it has with 0.4.5.

bozhidar17:11:20

No idea. Seems you can really use that EDN transport (or maybe the transit ones).

bozhidar17:11:59

At any rate - I’m 100% certain that the encoding logic hasn’t changed at all, so the problem is likely unrelated.

pez19:11:01

It seems that what my nrepl client is failing to decode is a java stack trace.

pez19:11:25

Found it. It was the figwheel-sidecar dependency that needed to be updated to 0.5.17.

pez19:11:06

Now most things work, but the test ops fail with a stack trace starting with Exception in thread "nREPL-worker-2" java.lang.IllegalArgumentException: No implementation of method: :send of protocol: #'clojure.tools.nrepl.transport/Transport found for class: nrepl.transport.FnTransport