Skip to content
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 target platform to 4.25-I-builds #2128

Merged
merged 1 commit into from
Jun 15, 2022

Conversation

snjeza
Copy link
Contributor

@snjeza snjeza commented Jun 15, 2022

Fixes #2127

Signed-off-by: Snjezana Peco [email protected]

@snjeza snjeza merged commit 18ea9fb into eclipse-jdtls:master Jun 15, 2022
@jdneo
Copy link
Contributor

jdneo commented Jun 16, 2022

Not sure if it is the issue with my local environment. After updating the target platform, I cannot launch the JDT.LS using PDE. With a lot of Unresolved requirement... printed in the terminal.

// cc @testforstephen

CsCherrYY added a commit that referenced this pull request Jun 16, 2022
@snjeza
Copy link
Contributor Author

snjeza commented Jun 16, 2022

@jdneo I can't reproduce the issue. Could you try to clean your workspace?

@jdneo
Copy link
Contributor

jdneo commented Jun 16, 2022

Still not working after cleaning workspace cache. BTW, I'm using PDE in VS Code and launch from the file jdt.ls.socket-stream.launch, I haven't tried Eclipse yet.

@snjeza
Copy link
Contributor Author

snjeza commented Jun 16, 2022

I have created a vsix.

@jdneo
Copy link
Contributor

jdneo commented Jun 16, 2022

Have no idea why, but so far both @CsCherrYY and me encountered the same problem. 🤔 Will take a look and update here if we find anything.

@testforstephen
Copy link
Contributor

I'm also facing issues when using vscode-pde to launch JLS from jdt.ls.socket-stream.launch. Let's debug to see what happens.

@testforstephen
Copy link
Contributor

Adding the following bundles to the dependencies list of jdt.ls.socket-stream.launch, the launching works again. Will create a PR to fix this.

<setEntry value="org.osgi.service.prefs@default:default"/>
<setEntry value="org.osgi.util.function@default:default"/>
<setEntry value="org.osgi.util.measurement@default:default"/>
<setEntry value="org.osgi.util.position@default:default"/>
<setEntry value="org.osgi.util.promise@default:default"/>
<setEntry value="org.osgi.util.xml@default:default"/>

@snjeza
Copy link
Contributor Author

snjeza commented Jun 16, 2022

@testforstephen You could create a separate launch configuration.

@rgrunber
Copy link
Contributor

I was able to reproduce in Eclipse once I cleaned my target platform. I only needed :

<setEntry value="org.osgi.service.prefs@default:default"/>
<setEntry value="org.osgi.util.function@default:default"/>
<setEntry value="org.osgi.util.promise@default:default"/>

which is also confirmed by doing a build of the language product.

Also, looks like org.eclipse.osgi.util is no longer part of the product.

testforstephen pushed a commit to testforstephen/eclipse.jdt.ls that referenced this pull request Jun 17, 2022
- Fixes eclipse-jdtls#2128
- Add it as a quick-fix for UnclosedCloseable &
  PotentiallyUnclosedCloseable as well as UnhandledException &
  UnhandledExceptionOnAutoClose as a quick-assist
- Add testcase

Signed-off-by: Roland Grunberg <[email protected]>
@testforstephen
Copy link
Contributor

I was able to reproduce in Eclipse once I cleaned my target platform. I only needed :

<setEntry value="org.osgi.service.prefs@default:default"/>
<setEntry value="org.osgi.util.function@default:default"/>
<setEntry value="org.osgi.util.promise@default:default"/>

which is also confirmed by doing a build of the language product.

Also, looks like org.eclipse.osgi.util is no longer part of the product.

Yes, only these three bundles are necessary.

@jdneo
Copy link
Contributor

jdneo commented Jun 20, 2022

Just FYI, after adopting new target platform, we can now have more concrete type candidate when completion for an interface:

completion.mp4

@rgrunber
Copy link
Contributor

Nice :) ( eclipse-jdt/eclipse.jdt.core#15 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update target platform to 4.25-I-builds
4 participants