Change standalone debugger to no longer limit number of bundles #782
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lsp4e depends on mylyn.wikitext but it could not be found. For example: https://github.com/eclipse-cdt/cdt/actions/runs/9026888723/job/24804874235?pr=778
jonahgraham
force-pushed
the
fix-standalone
branch
from
1765f77 to
6d2a508
Compare
The maintenance of having a streamlined standalone debugger that starts as fast as possible is no longer possible. See for example eclipse-cdt#591 - therefore when using standalone debugger, use the same sets of plug-ins/features as the product it is installed in uses. The side effect is that the standalone debugger in this use case will start slower and extra "stuff" will be present in this UI. For people just building the standalone debugger, provide a minimum feature set. This will be many more bundles than before, but should still provide a reasonably small set that starts well. This simplification also includes removing the the duplicates set of CDT docs (debug/org.eclipse.cdt.debug.application.doc). These provided a simplified version of CDT's documentation targetted at just standalone debugger. However there are a few problems related to this duplication: - The two sets of docs were not kept in sync - The standalone docs appear in the online help, leading to duplicated entries - With the config.ini changes above, there is no way to exclude the main docs in the standalone case, so remove the duplicate A number of directly related clean-ups are included too: - Remove the `ConfigGenerator.java` that stopped being referenced in PR eclipse-cdt#761 - Complete the removal of `build-standalone-debugger-rcp` profile that was started in eclipse-cdt#761. There is a small drawback to not having this profile, the standalone debugger is very slow to build compared to the rest of CDT. If this becomes a problem, restoring this profile along with the changes made in eclipse-cdt#761 is reasonable. - bring debug.product's licenses up to date - modernize command line args to eclipse when using debug.product Fixes eclipse-cdt#781
The simrel and cdt update sites can be used to run partial builds and are not used by default when doing builds. This change brings those URLs up to date.
jonahgraham
force-pushed
the
fix-standalone
branch
from
6d2a508 to
6f8746e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The maintenance of having a streamlined standalone debugger that starts as fast as possible is no longer possible. See for example #591 - therefore when using standalone debugger, use the same sets of plug-ins/features as the product it is installed in uses. The side effect is that the standalone debugger in this use case will start slower and extra "stuff" will be present in this UI.
For people just building the standalone debugger, provide a minimum feature set. This will be many more bundles than before, but should still provide a reasonably small set that starts well.
This simplification also includes removing the the duplicates set of CDT docs (debug/org.eclipse.cdt.debug.application.doc). These provided a simplified version of CDT's documentation targetted at just standalone debugger. However there are a few problems related to this duplication:
A number of directly related clean-ups are included too:
ConfigGenerator.javathat stopped being referenced in PR [#657] CDT cannot be built on Java 21 due to security manager changes #761build-standalone-debugger-rcpprofile that was started in [#657] CDT cannot be built on Java 21 due to security manager changes #761. There is a small drawback to not having this profile, the standalone debugger is very slow to build compared to the rest of CDT. If this becomes a problem, restoring this profile along with the changes made in [#657] CDT cannot be built on Java 21 due to security manager changes #761 is reasonable.Fixes #781