boot-clj / boot

Build tooling for Clojure.
https://boot-clj.github.io/
Eclipse Public License 1.0
1.75k stars 180 forks source link

Warning about missing nREPL middleware descriptor when colorization is disabled #647

Open mudgen opened 6 years ago

mudgen commented 6 years ago

Platform details

Platform (macOS, Linux, Windows): Windows 10 Platform version: Windows 10 Home, Version 1703, OS Build 15063.674 JRE/JDK version (java -version): JRE: "1.8.0_141"

Boot details

Boot version (2.7.1): 2.7.1 build.boot present? (yes/no): yes ~/.boot/profile present? (yes/no): no Task name? (if applicable): repl

build.boot content (if applicable)

;; build.boot content here

~/.boot/profile content (if applicable)

(task-options!
  pom {:project     'ignition-project
       :version     "1.0"
       :description "Fix me"
       :license     {"EPL" "http://www.eclipse.org/legal/epl-v10.html"}})

Description

I get the following error when I run "boot repl" [WARNING] No nREPL middleware descriptor in metadata of boot.repl$disable_exception_colors@4c415a68, see clojure.tools.middleware/set-descriptor! nREPL server started on port 55825 on host 127.0.0.1 - nrepl://127.0.0.1:55825 REPL-y 0.3.7, nREPL 0.2.12 Clojure 1.8.0 Java HotSpot(TM) 64-Bit Server VM 1.8.0_60-b27

Why is this happening? Does it cause any problems? Is there a way to fix it? Thanks for your help.

seancorfield commented 6 years ago

It is not Windows-specific. I see this on Mac when starting a REPL in Atom/ProtoREPL as well as from the command line. My comment on Slack:

That warning appears for Boot REPL on all platforms I believe wherever colorization is disabled. The solution would be for boot.repl/disable-exception-colors to have metadata of ^{:clojure.tools.middleware/descriptor {}} I think.

martinklepsch commented 6 years ago

Reproducible by disabling colorization, as @seancorfield mentioned above:

boot -C repl
jarcane commented 6 years ago

Found this while googling, apparently pretty had the same problem and this is how they fixed it: https://github.com/AvisoNovate/pretty/pull/29