-
Notifications
You must be signed in to change notification settings - Fork 12
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
Check for clangd in MinGW installation #440
Conversation
@jonahgraham and @ghentschke, this patch attempts to find
Please feel free to use or adapt this as you see fit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jonahgraham and @ghentschke, this patch attempts to find
clangd
in the MinGW installation known to CDT (eg MSYS2 UCRT64) if it was not found on the systemPATH
.
Thanks @jld01 for this - I was hoping something like this would be possible with MinGW class and your implementation is simpler and more straightforward than I expected :-)
Note that:
MinGW.getMinGWHome()
returnsnull
efficiently if not running on a Windows host
👍
- The
MinGW
class is not API at present
In my view CDT LSP is part of CDT so using CDT internal code is not violating any API contract. Ideally an x-friends should be added on org.eclipse.cdt.internal.core
at some point to remove warning.
Please feel free to use or adapt this as you see fit.
Thanks.
This LGTM, but I haven't run it on Windows yet. @ghentschke I am hoping you can review to verify that this is the best place to inject the MinGW's location.
Note that the MinGW class performs various checks to find an installation containing We can improve on this, but time is short for 12.0.0 and the code provides a good improvement where a new user is following the Before you begin help page to install tools. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Does it also work for MSYS2? |
Yes, the CDT MinGW class will detect MinGW tools installed via the MSYS2 package manager. This is the use case of primary interest. |
Relates to #424