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

Error validating SDK launch due missing JRE 1.1 support #1231

Closed
iloveeclipse opened this issue Apr 15, 2024 · 8 comments
Closed

Error validating SDK launch due missing JRE 1.1 support #1231

iloveeclipse opened this issue Apr 15, 2024 · 8 comments
Labels
bug Something isn't working regression Regression defect

Comments

@iloveeclipse
Copy link
Member

After removing support for JRE 1.1 (eclipse-equinox/equinox#572 & eclipse-jdt/eclipse.jdt.debug#424) there is a validation error shown on startup of SDK from SDK.

PDE should update it's logic to validate launches.

image

@akurtakov
Copy link
Member

@merks As this one comes from Orbit I would say that the fix should be there, no?

@iloveeclipse
Copy link
Member Author

PDE allows to use any libraries in the platform, and PDE can be used to debug older platforms.
So the fix must be in PDE / SDK.

@merks
Copy link
Contributor

merks commented Apr 15, 2024

There's nothing wrong with such a requirement. Even the latest Java 21 provides those system capabilities:

image

So if PDE says those capabilities are missing, it's not looking at the right thing or not looking in the right way.

@akurtakov
Copy link
Member

akurtakov commented Apr 15, 2024

Acked about capability being provided and thus PDE fix is due.
eclipse-equinox/equinox#571 (comment) makes me believe that we should try to not ship bundles with "non-standard" EEs too.

@merks
Copy link
Contributor

merks commented Apr 15, 2024

FYI, for history, you can see this issue:

eclipse-orbit/orbit#52

This legacy bundle is a drop-in replacement for this:

https://repo1.maven.org/maven2/xml-apis/xml-apis-ext/1.3.04/

That thing provides no source. If it did, I would have just wrapped in with m2e/tycho BND support which would produce this manifest:

image

So standard or not, BND produces that and org.eclipse.osgi.storage.Storage.calculateEECapabilities(Version) computes that, so I tried to reproduce something that includes actual sources and that follows that same expect result.

@HannesWell
Copy link
Member

PDE allows to use any libraries in the platform, and PDE can be used to debug older platforms. So the fix must be in PDE / SDK.

Yes. After all I think it is simpler to just revert the removal in Equinox instead of cleaning up everthing from JRE-1.1. There are some left-overs of it in Equinox (e.g. what Ed mentioned) and also in PDE etc.
Created eclipse-equinox/equinox#596 to revert it and add the system-packages currently hard-coded in PDE.
To fully fix this issue eclipse-jdt/eclipse.jdt.debug#425 has to be reverted because PDE relies on the EE JDT provides. So at least the JRE-1.1 EE must be restored in JDT.

Once eclipse-equinox/equinox#596 is submitted the following hard-coded list can be removed from PDE:

"JRE-1.1", List.of("java.applet", //
"java.awt", //
"java.awt.datatransfer", //
"java.awt.event", //
"java.awt.image", //
"java.awt.peer", //
"java.beans", //
"java.io", //
"java.lang", //
"java.lang.reflect", //
"java.math", //
"java.net", //
"java.rmi", //
"java.rmi.dgc", //
"java.rmi.registry", //
"java.rmi.server", //
"java.security", //
"java.security.acl", //
"java.security.interfaces", //
"java.sql", //
"java.text", //
"java.text.resources", //
"java.util", //
"java.util.zip"),

HannesWell added a commit to eclipse-equinox/equinox that referenced this issue Apr 15, 2024
The packages are read from ancient Java-1.1 JREs.

Fixes #571
Part of eclipse-pde/eclipse.pde#1231
@iloveeclipse
Copy link
Member Author

Once eclipse-equinox/equinox#596 is submitted the following hard-coded list can be removed from PDE

@HannesWell : do you plan to provide a PR for that?

iloveeclipse added a commit to iloveeclipse/eclipse.jdt.debug that referenced this issue Apr 16, 2024
This reverts commit 0cd092a because
corresponding equinox change was reverted.

See discussion on eclipse-equinox/equinox#571
See eclipse-pde/eclipse.pde#1231
See eclipse-equinox/equinox#596
iloveeclipse added a commit to eclipse-jdt/eclipse.jdt.debug that referenced this issue Apr 16, 2024
This reverts commit 0cd092a because
corresponding equinox change was reverted.

See discussion on eclipse-equinox/equinox#571
See eclipse-pde/eclipse.pde#1231
See eclipse-equinox/equinox#596
@HannesWell
Copy link
Member

Once eclipse-equinox/equinox#596 is submitted the following hard-coded list can be removed from PDE

@HannesWell : do you plan to provide a PR for that?

See #1239.

With eclipse-jdt/eclipse.jdt.debug#431 being merged I think this is resolved.
Thank you @iloveeclipse for providing the PR for JDT.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression Regression defect
Projects
None yet
Development

No branches or pull requests

4 participants