Dev UI: open code in IDE should use IDE path from scanning processes #15721
Comments
|
This is a really good idea. |
|
Nice |
|
on first hand this does looks interesting but please be aware that wont work if using osx Applications. i.e. if you call the eclipse executable directly on osx it will launch a new instance, not reuse the existing already running eclipse. I dont know if same goes for other IDE's, but I can very well imagine that there is a shell / batch wrapper to handle reusing logic thus using the command listed in process table is not a sure thing. I would say it could be used if and only if no matching binary/script is found in the PATH. |
That's a great idea. |
|
Is someone working on this or should I take it? |
Sure, |
…ript doesn't exist Currently support for this is only added for IntelliJ. Code is opened client side (via the URL handler it installs) so this support isn't needed, Netbeans already used something similar and Eclipse seems to be sort of problematic because we don't have a command which we can test that would not launch the IDE. Fixes: quarkusio#15721
…ript doesn't exist Currently support for this is only added for IntelliJ. Code is opened client side (via the URL handler it installs) so this support isn't needed, Netbeans already used something similar and Eclipse seems to be sort of problematic because we don't have a command which we can test that would not launch the IDE. Fixes: quarkusio#15721
…ript doesn't exist Currently support for this is only added for IntelliJ. Code is opened client side (via the URL handler it installs) so this support isn't needed, Netbeans already used something similar and Eclipse seems to be sort of problematic because we don't have a command which we can test that would not launch the IDE. Fixes: quarkusio#15721
…ript doesn't exist Currently support for this is only added for IntelliJ. Code is opened client side (via the URL handler it installs) so this support isn't needed, Netbeans already used something similar and Eclipse seems to be sort of problematic because we don't have a command which we can test that would not launch the IDE. Fixes: quarkusio#15721
Add support for opening the IDE from DevUI even if the default IDE script doesn't exist
…ript doesn't exist Currently support for this is only added for IntelliJ. Code is opened client side (via the URL handler it installs) so this support isn't needed, Netbeans already used something similar and Eclipse seems to be sort of problematic because we don't have a command which we can test that would not launch the IDE. Fixes: quarkusio#15721 (cherry picked from commit f15355f)
…ript doesn't exist Currently support for this is only added for IntelliJ. Code is opened client side (via the URL handler it installs) so this support isn't needed, Netbeans already used something similar and Eclipse seems to be sort of problematic because we don't have a command which we can test that would not launch the IDE. Fixes: quarkusio#15721
ATM we scan the processes to find which IDE to use, but then we discard the running process command, and invoke
ideaoreclipsehoping it's in the user's$PATH, which turns out is not necessarily true, for people with multiple versions of each IDEs.It would be nice if the scanning of processes returned the exact IDE command path and we used that instead of relying on
$PATH.CC @geoand
The text was updated successfully, but these errors were encountered: