-
-
Notifications
You must be signed in to change notification settings - Fork 645
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
Timing problem for jack-in? #441
Comments
Could you, please, post your cider-related config? |
I have confirmed that the problem still exists with an empty cider-config. I haven't been able to reproduce it in a project that uses the following deps: :dependencies [[org.clojure/clojure "1.5.1"]
[clj-webdriver "0.6.0"]
[org.clojure/core.incubator "0.1.2"]
[com.datomic/datomic-free "0.8.4020.24" :exclusions [org.apache.httpcomponents/httpclient]]
[fs "1.3.2"]] My intermittently failing project does only use Midje. (Other projects still fail without midje) |
I asked about your config, because unless you've enabled REPL pretty-printing I can't think of why |
Hmm... strange. But I might have found something relevant in {:user {:plugins [[lein-midje "3.1.3"] [lein-kibit "0.0.8"]]
;:dependencies [[io.aviso/pretty "0.1.8"]]
}
;:repl-options {:nrepl-middleware [io.aviso.nrepl/pretty-middleware]}
} aviso/pretty seems a likely culprit, but it is disabled.
|
|
Nothing in nREPL itself, which doesn't use pretty-printing at all anywhere. The "intermittent" thing is bizarre; perhaps it only seems intermittent because you're twiddling around with configurations? ;-) |
I did some testing just now with But as I said before, I haven't discovered any ill effects so far. |
I had this a lot on a Debian Wheezy machine, but also intermittent. My "fix" was to add a short delay in defun cider-init-repl-buffer (connection buffer &optional noprompt)
"Initialize the REPL for CONNECTION in BUFFER.
Insert a banner, unless NOPROMPT is non-nil."
(with-current-buffer buffer
(unless (eq major-mode (quote cider-repl-mode))
(cider-repl-mode))
;; use the same requires by default as clojure.main does
(sit-for 1) ; allow lein repl to come up fully?
(cider-eval-sync nrepl-repl-requires-sexp)
(cider-repl-reset-markers)
(unless noprompt
(cider-repl--insert-banner-and-prompt nrepl-buffer-ns))
(cider-remember-clojure-buffer cider-current-clojure-buffer)
(current-buffer))) Maybe there is a race going on here between emacs and the repl clojure instantiation? |
@mpwillson @bonega Can you insert the following code in place of the
I have to know whether the server is causing the problem or the connection to it. |
This is the output in Messages when the stack trace occurs (it's the same when it doesn't):
Let me know if you need anything else. -mark |
I can't reproduce the problem anymore. |
@mpwillson You get only a Java stack trace, right? An Emacs stack trace would help me narrow the issue further, but I doubt any is produced. |
Yes, only the Java stack trace is produced; emacs is silent, I'm afraid. -mark |
Have you set the |
I have the same bug occasionally, I can confirm that it happens non-uniformly. My relevant .emacs configuration: (add-hook 'clojure-mode-hook 'flymake-mode-on)
(require 'ac-nrepl)
(add-hook 'cider-repl-mode-hook 'ac-nrepl-setup)
(add-hook 'cider-mode-hook 'ac-nrepl-setup)
(eval-after-load "auto-complete"
'(add-to-list 'ac-modes 'cider-repl-mode)) |
@noisesmith , thanks for the suggestion. However, with debug-on-error set to t, there is still no evidence of an emacs error. -mark |
@mpwillson could you try to require and use pp after the error message? |
@bonega I can use pp fine in the cider repl - no errors at all. One other data point. If I change
the IllegalAccessError does not occur. -mark |
I get the following stacktrace intermittently upon doing
M-x: cider-jack-in
The frequency might be dependent on something in the project used, not sure yet.
I can't really tell if anything breaks when it happens.
PP exists in the repl directly after the stacktrace.
Thought it would be helpfull anyway, I saw some old reports posting similar stacktraces but without the timing aspect.
The text was updated successfully, but these errors were encountered: