chlorine-clover

About Chlorine for Atom and Clover for VS Code: https://atom.io/packages/chlorine and https://marketplace.visualstudio.com/items?itemName=mauricioszabo.clover
mauricio.szabo 2020-04-10T14:58:01.156900Z

@fabrao I think I may have a lead on what's happening with goto var definition. Just a question, are you using Atom on Windows, with WSL or without? And Clojure on Windows also with or without WSL?

fabrao 2020-04-10T15:00:01.157700Z

@mauricio.szabo I´m using Atom on Windows and I have WSL installed but not active

mauricio.szabo 2020-04-10T15:01:20.158600Z

Okay... I think I may have a lead on your problem. Will try to reproduce here 🙂

fabrao 2020-04-10T15:02:30.159100Z

What do you think WSL is "infecting" Atom Windows?

fabrao 2020-04-10T15:02:38.159400Z

I´m using WSL 2.0

mauricio.szabo 2020-04-10T16:14:45.161100Z

Not really, it can be a problem with backslashes / forward slashes on Node + Clojure + Atom. There's a lot of things that can go wrong when you add interops between Java URIs, absolute paths, windows, and Node.JS 😞

seancorfield 2020-04-10T16:19:07.162500Z

There is some custom logic in Chlorine (at least, I assume it's still present?) that tries to detect whether you are on the same O/S as your REPL, deliberately to support Atom on Windows talking to a REPL on WSL, and it maps Windows drives to WSL mount points. It should not come into play for Windows talking to Windows tho'...

mauricio.szabo 2020-04-10T16:20:22.163500Z

Yes, it is still there on load-file command. I'll try to check if I can reproduce the error on Windows, and if so, if that logic solves the problem

mauricio.szabo 2020-04-10T18:56:16.164Z

Ok, I'm able to reproduce. Will work on a fix

mauricio.szabo 2020-04-10T19:54:32.164800Z

@fabrao just one last question: what's the version of Java you're using? Even better: what java -version prints to you?

mauricio.szabo 2020-04-10T20:36:04.166100Z

I am amazed: Java returns /C:/.... as the file path... :(

seancorfield 2020-04-10T20:57:41.166900Z

I'm surprised about the leading / but otherwise not surprised that it returns / since forward slashes work on both Windows and macOS/Linux in Java.

seancorfield 2020-04-10T20:58:13.167300Z

(I suspect that breaks the load-file logic for cross-platform stuff too?)

fabrao 2020-04-10T21:10:07.168100Z

@mauricio.szabo Sorry about late response

openjdk version "1.8.0_242"
OpenJDK Runtime Environment (build 1.8.0_242-b08)
Eclipse OpenJ9 VM (build openj9-0.18.1, JRE 1.8.0 Windows 10 amd64-64-Bit Compressed References 20200122_564 (JIT enabled, AOT enabled)
OpenJ9   - 51a5857d2
OMR      - 7a1b0239a
JCL      - 8cf8a30581 based on jdk8u242-b08)

fabrao 2020-04-10T21:10:59.169100Z

I observe too that clicking into error line when Exception happens the same problem runs out

mauricio.szabo 2020-04-10T21:29:06.170600Z

Yes, it's the same code. Considering that you're the only person who have this problem, I'm thinking it may be a problem with OpenJ9 (by luck, I also installed OpenJ9). I'll change java version and see if changes anything

fabrao 2020-04-10T21:30:26.171400Z

I had to switch to OpenJ9 because I use Webstart for using with Cisco stuffs

fabrao 2020-04-10T21:30:48.171900Z

let me change to the other java

mauricio.szabo 2020-04-10T21:32:38.172400Z

Don't worry, I also use OpenJ9. I'll issue a fix, but probably will open a bug report for OpenJ9 too

mauricio.szabo 2020-04-10T21:43:19.173200Z

Ok, its not an issue of OpenJ9 - hotspot does the same thing. I'll fix it...

fabrao 2020-04-10T22:31:20.173700Z

@mauricio.szabo I tried with other version of Java and the same problem happen