clojure.main/repl unconditionally refers REPL utilities into *ns*


  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: Release 1.4, Release 1.5
  • Fix Version/s: Release 1.5
  • Component/s: None
  • Labels:
  • Patch:
  • Approval:


A number of vars from clojure.repl,, and clojure.pprint are unconditionally referred into *ns* by clojure.main/repl. This is fine when it is being used e.g. as the primary driver of a terminal-bound Clojure REPL, but other usages can end up bringing those utility vars into namespaces other than 'user. This can cause problems if clojure.main/repl is used to drive a REPL within namespaces that already have referred or interned vars with the same names as those utility vars, e.g.:

$ java -jar ~/.m2/repository/org/clojure/clojure/1.5.0-alpha6/clojure-1.5.0-alpha6.jar
Clojure 1.5.0-alpha6
user=> (ns foo)
foo=> (defn pp [] "hi!")
foo=> (pp)
foo=> (clojure.main/repl)
foo=> (pp)
foo=> (defn pp [] "whoops")
CompilerException java.lang.IllegalStateException: pp already refers to: #'clojure.pprint/pp in namespace: foo, compiling:(NO_SOURCE_PATH:7:1)

Worse, nREPL uses clojure.main/repl (in large part to maximize the consistency of REPL behaviour across different Clojure versions), where each user expression is evaluated through a separate clojure.main/repl invocation. This leads to the same problems as above, but for every nREPL user, session, and expression (reported @ NREPL-31).

A simple fix for this is to perform these refers only if *ns* is 'user (which, AFAICT, was the only intended effect of CLJ-310, CLJ-454, and, the changes that added these automatic implicit refers to clojure.main/repl).

  1. CLJ-1085.diff
    10/Oct/12 2:19 PM
    1 kB
    Chas Emerick
  2. CLJ-1085-refactor.diff
    15/Oct/12 4:47 AM
    2 kB
    Chas Emerick


Chas Emerick made changes -
Field Original Value New Value
Attachment CLJ-1085.diff [ 11548 ]
Patch Code [ 10001 ]
Chas Emerick made changes -
Attachment CLJ-1085.diff [ 11548 ]
Chas Emerick made changes -
Attachment CLJ-1085.diff [ 11549 ]
Chas Emerick made changes -
Attachment CLJ-1085-refactor.diff [ 11560 ]
Stuart Halloway made changes -
Fix Version/s Release 1.5 [ 10150 ]
Approval Screened [ 10004 ]
Rich Hickey made changes -
Approval Screened [ 10004 ] Ok [ 10007 ]
Stuart Halloway made changes -
Status Open [ 1 ] Closed [ 6 ]
Resolution Completed [ 1 ]


Vote (3)
Watch (1)


  • Created: