Process project's repl-options about cider

[technomancy]

This actually needs to be handled server-side, probably in nREPL, but possibly with some changes to Leiningen's repl task.

[jsnikeris]

A workaround:

(defun my-nrepl-connected-hook ()
  (nrepl-interactive-eval
   "(do (set! *print-length* 103) (set! *print-level* 15))"))

(add-hook 'nrepl-connected-hook 'my-nrepl-connected-hook)

[kenrestivo]

If it could grab :port from either project.clj or profiles.clj, then mayb eit wouldn't need to prompt for one.

[kingtim]

This is fixed in leiningen now.
See issue technomancy/leiningen#788.

[mtopolnik]

What exactly is fixed? In the pull request mentioned in the referenced issue, only a patch specific to honoring :init is involved. This still leaves :init-ns, which is doubly convenient: it both compiles the initial namespace and sets it as the current namespace. Since practically every repl session starts with this routine performed manually, honoring init-ns in nrepl.el would obviously be a major convenience. I was actually looking into writing some elisp code that would open the .clj file of the initial namespace since this is the third part of the regular routine. I was going to rely on nrepl-current-ns, but unfortunately this doesn't get set to the init-ns.

UPDATE: I realized by reading repl task's source code that :init-ns is baked into :init. This has other issues, though: the behavior of the nrepl server is now different for headful and headless mode regarding the initial namespace (at least it looks like it).

[weissjeffm]

I couldn't get either :init-ns or :init to do anything at all in my nrepl.el repl.

I tried running nrepl-jack-in, :init-ns is ignored. lein repl :headless and then nrepl to connect, ignored. If I run a plain lein repl then it works. I think this needs to be re-opened since the only way to get it to work is to not use nrepl.el at all.

[mtopolnik]

nrepl.el needs to be fixed such that it doesn't change the current namespace upon connect. The namespace at the nrepl server side is already set to the configured initial namespace.

[weissjeffm]

@mtopolnik to me that sounds like this issue should be re-opened, or possibly open a new issue, then?

[mtopolnik]

@weissjeffm Perhaps a new issue focused on the exact requirement. I posted to nrepl.el Google Groups some time ago with this request. It would probably be better to raise an issue here.

[weissjeffm]

Opened #316