-
Notifications
You must be signed in to change notification settings - Fork 918
-
Notifications
You must be signed in to change notification settings - Fork 918
Show docs breaks instarepl on windows #1732
Comments
@joshuafcole I can't repro this myself based on your steps. My InstaREPL is connected to LightTable ("LightTable-REPL"). Is yours a project? If so, can you provide a minimal project in which you can still repro this? I'm running version 0.7.2 – maybe this was fixed since 0.7.1? [Incidentally, I just upgraded from 0.6.7 and I miss the auto-closing parentheses.] |
[And, I already see that I'm not the only one that didn't read the change-log!] |
I'm getting this issue too. I'm sorry I can't say when it started because I pressed on for a while. Search language docs does work though. My stack: LT 0.7.2 on Yosemite
|
@alilee Can you create a tempfile from a clojure repl? To find out do:
Also, what Java version are you running? |
Seems to work fine. Java version from repl signature: Java HotSpot(TM) 64-Bit Server VM 1.6.0_65-b14-466.1-11M4716
|
Interesting. Any other thoughts on what might help unravel this mystery? I had expected it to be platform specific. I suppose it's plausible that there's a race condition, but unless Show Docs is infrequently used, I'd have expected more reports about it by now. |
@joshuafcole @alilee I'm guessing the reason it passes for the above repl session and fails in the editor is a difference in
I'm happy to fix if I can get those debugging values |
Thanks very much for the strategy.
Capturing that output is tricky btw:
|
FWIW, I just had this problem too, using LT 0.7.2 on Yosemite (10.10.1). I've now fixed it by installing the latest JDK (8u25) and forcing the /usr/bin/java stubs to point to this version, specifically by adding this to my dotfiles: export JAVA_HOME= Restarted LT and now it works properly. Not sure whether updating JAVA_HOME was strictly necessary for LT; I did the JDK install and edited dotfiles at the same time so can't prove it one way or t'other. Note for ref that I tried updating just the JRE first of all to no effect. It needed the whole JDK. Also for completeness, java versions before... $ java -version ...and after: $ java -version Hope this helps a bit. |
@jdwaterson thank you 👍 |
Can confirm that I was having the same issue and I had tried (before reading this issue) the same approach of upgrading from JDK 7 to JDK 8 and it seemed to have fixed my issue, |
@joshuafcole @cldwalker could either or both of you confirm whether this is resolved by upgrading the JDK to version 8? |
I had the same issue on a mac. Switched to Java 8 and that resolved it. A nice way to manage multiple versions of java is by using http://www.jenv.be . |
Apologies for getting back to this thread late. Too many issues and too small of a core team. As you all have discovered jump to source targeted Java 8. The Clojure plugin will no longer support Java 7 as Oracle hasn't supported it since April of this year. I've updated the Clojure plugin to reflect this - LightTable/Clojure@ae40383 |
I was recently walking a friend through generating music algorithmically in overtone. On introducing him to LT's instarepl for him to start writing some code, we found that the show docs function was generating 40 lines of error stack trace. This seems related to a previous issue involving temp file creation that I can't seem to find right now. The instarepl works fine normally before show docs is used. We used it on a regular function (in this case
+
). I can continue working with him to provide more diagnostic information.Repro:
Stack Trace:
The text was updated successfully, but these errors were encountered: