-
Notifications
You must be signed in to change notification settings - Fork 210
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
Update CDT.setup's modular targlet to include a site for mylyn.wikitext #783
Conversation
I've locally made additional changes but I wanted to ask before creating a PR. We should change to configure Java 21 for the JRE. I added a task to clone cdt-lsp, added a source locator for the clone location to the targlet task, and added a working set for those plug-ins. Of course one could create a separate project setup for cdt-lsp, but that has more overhead. What do you think? |
Perhaps we should also ask @ghentschke here, since she is leading CDT-LSP development area |
I'm happy to help do what makes people happiest. 😀 |
From my side it's okay. WDYT @jonahgraham ? |
I added a task to clone cdt-lsp, added a source locator for the clone location to the targlet task, and added a working set for those plug-in ...But I cannot overview the consequences right now. How does it affect the cdt-lsp repo on GitHub? |
I'm waiting for this PR to be accepted before I create a new one. There is no change to the cdt-lsp repo; all the changes are in the same files as changed by this PR. This is the current state:
|
Using the setup from scratch was failing like this:
Adding https://download.eclipse.org/mylyn/updates/nightly/latest fixes that problem.
The schemaLocation changes are simply the proper newer github locations of the setup models.