Fails to load when eclipse app has space in file name (Mac OS X) #12
Comments
Why do you need to use spaces in eclipse installation directory name? Supporting directories names with spaces and another exotic characters is rather tedious and I would like to avoid this unless absolutely necessary. |
For the same reason you would use any other valid character in a path name? Are you claiming space is an 'exotic' character? AFAIK, a space is a valid Just out of curiosity, why do you find supporting strings that represent On Mon, Jul 6, 2015 at 6:28 PM, ifedorenko notifications@github.com wrote:
|
Properly quoting command line invocation parameters on all platforms is tricky. Should I use single or double quotes? What happens if the path name happens to include the quote type I chose? Need to escape. What should I do with really exotic characters, non-breaking space anyone? And then we have Windows. Quite frankly, I don't have time/interest to deal with this unless there is a really good reason somebody absolutely must use directory names with spaces. |
If you don't have the time/interest to deal with correct parsing, may I I deal with many file names on many platforms many times a day, so I pretty I first noticed this problem on Mac OS X, and duplicated on Linux, haven't On Tue, Jul 7, 2015 at 10:33 AM, ifedorenko notifications@github.com
|
Not sure what you mean by "delegate". This is opensource project under EPL, you are more than welcome to fork this codebase and implement support for command line argument quoting. Keep in mind that Maven Dev Tools does not launch Maven directly. I think the real permanent fix would be to implement parameter quoting in eclipse debug launch support, but it will likely be very hard to convince eclipse developers to accept this change. m2e launch support is another place where it may make sense to implement this. As for Windows, I know that people are using this successfully on Windows, but I do not know if there are any specific limitations on that platform. (I mostly work/test on OSX with occasional Linux use) |
Default commons-exec behaviour does not correctly quote process parameters. Running the following https://gist.github.com/ifedorenko/39948b6693745ab87e90 results in unnecessary double-quotes passed to TestMain arguments on all three major platforms. Need to explicitly tell commons-exec NOT to quote. Ironic. |
I'm also having this problem on windows (and I can reproduce it on Linux): at work, for security reasons, it not possible to run executables in a directory other than C:/Program Files/... Anyway, the following fix seems to work for me on windows and linux: |
The problem exists on Windows (8.1), too.
How do I install the fix? Update eclipse does not help. |
Also on Windows 10 (unfortunately my user directory contains a space: "C:\Users\Jurrie Overgoor"). Sadly, Windows made this decision for me :( -> Windows -> Preferences -> Maven -> Source Lookup: The only other option I see is to force Windows to use another directory as my user dir... Which means jumping through burning hoops, hacking in the registry and hoping it won't affect AD in any way... |
I expect this will be solved when the code is accepted by jdt.debug (see https://bugs.eclipse.org/bugs/show_bug.cgi?id=506149). |
Here's how you recreate it:
VM arguments:
Illegal character in path at index 39: /Users/.../Applications/Eclipse 4.5 (Mars).app/Contents/Eclipse/plugins/com.ifedorenko.m2e.sourcelookup_1.1.0.201506181114/com.ifedorenko.m2e.sourcelookup.javaagent.jar
The text was updated successfully, but these errors were encountered: