-
Notifications
You must be signed in to change notification settings - Fork 406
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
Avoid refreshing bundles whose dependency closure includes JDT-LS. #2951
Conversation
org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/BundleUtils.java
Outdated
Show resolved
Hide resolved
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.
Overall, this seems to fix the issue! Just wouldn't mind having a closer look.
I was reproducing this (using VS Code) by making sure vscode-pde was installed, and then applying the following patch to JDT-LS :
diff --git a/org.eclipse.jdt.ls.product/languageServer.product b/org.eclipse.jdt.ls.product/languageServer.product
index 3b97b4ff..4389cb7e 100644
--- a/org.eclipse.jdt.ls.product/languageServer.product
+++ b/org.eclipse.jdt.ls.product/languageServer.product
@@ -66,6 +66,7 @@
<plugin id="org.eclipse.osgi.services"/>
<plugin id="org.eclipse.text"/>
<plugin id="org.eclipse.xtext.xbase.lib"/>
+ <plugin id="org.eclipse.team.core"/>
</plugins>
<configurations>
The first time it would always fail. Afterwards, I assume it'd succeed because the team.core
bundle from vscode-pde got properly detected by the framework, hence the singleton warnings. I would then reproduce the behaviour by calling rm -rf server/config_linux/org.eclipse.*
on the Equinox state files.
org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/BundleUtils.java
Outdated
Show resolved
Hide resolved
@rgrunber I have updated the PR. |
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.
I would just update the commit message and error message. After that, feel free to merge. Can you make it :
Avoid refreshing bundles whose dependency closure includes JDT-LS.
Just a note, we might be able to keep track of this in the future since the logs mention the failure. It isn't harmful though as the exception gets caught and doesn't break initialization.
Fixes redhat-developer/vscode-java#3349