-
Notifications
You must be signed in to change notification settings - Fork 1.6k
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Empty :repl profile causes FNFE for missing nREPL #893
Comments
This is not fixed. I'm using current master e01a198 , and I get the NFE error. I tried to hack around it by adding this to my profile: :repl {
:dependencies [[org.clojure/tools.nrepl "0.2.0-RC1"
:exclusions [org.clojure/clojure]]]
} But, then I get this incomprehensible, undebuggable error instead:
This is when trying to run "lein trampoline repl" task. Does trampoline actually work in leiningen or should I not be using it? |
Just to follow up, that error isn't so undebuggable, but it is very bizarre, because I'm not on windoze at all (linux wheezy). Everything works fine without trampoline. But trampoline breaks repl badly. |
Ah, nevermind, I found the cause of the problem, and hacked around it rather brutally: ``diff (defn- trampoline-file []
(defn- win-batch? [](.endsWith %28trampoline-file%29))
|
sigh --- a/src/leiningen/trampoline.clj
+++ b/src/leiningen/trampoline.clj
@@ -10,7 +10,7 @@
(def ^:dynamic *trampoline?* false)
(defn- trampoline-file []
- (System/getenv "TRAMPOLINE_FILE"))
+ (or (System/getenv "TRAMPOLINE_FILE") ".trampoline"))
(defn- win-batch? []
(.endsWith (trampoline-file) ".bat"))
|
And, FYI, with the above hack, AND with adding nrepl manually to :repl profile, I can get trampoline repl to work. |
Strange, lein-master new test
cd test
# ensure you have an empty `repl`-profile either
# in `profiles.clj` or `project.clj`, or both
lein-master repl
# works fine, C-d out
lein-master trampoline repl
# works fine too Sounds like there is an issue with the lein script here somehow, and it doesn't hook in the |
It's for the latest build from the sources.
Just discovered that the
repl
task can be configured with a:repl
profile. With an empty one -:repl {}
it causes the following exception.The text was updated successfully, but these errors were encountered: