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

Nebula 3.2.0 releng #616

Open
wimjongman opened this issue Nov 26, 2024 · 6 comments · Fixed by #623
Open

Nebula 3.2.0 releng #616

wimjongman opened this issue Nov 26, 2024 · 6 comments · Fixed by #623
Milestone

Comments

@wimjongman
Copy link
Contributor

Releng for release 3.2.0

@wimjongman wimjongman added this to the 3.2.0 milestone Nov 26, 2024
wimjongman added a commit that referenced this issue Nov 26, 2024
Updated feature version
merks pushed a commit that referenced this issue Nov 27, 2024
Updated feature version
@wimjongman
Copy link
Contributor Author

Anyone knows where this comes from?

image

merks added a commit that referenced this issue Nov 27, 2024
- Update the p2 task to use
http://download.eclipse.org/nebula/updates/nightly/latest
- Update the build to use
https://download.eclipse.org/releases/milestone to build always by
default against the most recent SimRel content.
- Update org.eclipse.nebula.widgets.grid.test to use bundle
org.eclipse.rap.rwt.testfixture optional to avoid forcing RAP into the
target platform.
  - I don't believe support for RAP/rwt actually works.
  - Also, this fragment isn't actually used as part of the build.

#616
@merks
Copy link
Contributor

merks commented Nov 27, 2024

@merks
Copy link
Contributor

merks commented Nov 29, 2024

@wimjongman

I asked about this and it was suggested that the scanning is code because of this configuration

https://github.com/EclipseNebula/.eclipsefdn/blob/9ed684fe553bcc57f1d57d685a6f7ff9f804f067/otterdog/eclipsenebula.jsonnet#L29

The problem appears to be that the scanner is running with an older version of Java than is required by the scanner libraries. But I'm not sure we can influence that anywhere so I think the IT staff needs to help:

https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/5363

  • Issue Fixed

@merks
Copy link
Contributor

merks commented Nov 29, 2024

@HannesWell @wimjongman

I had strange problem with my Nebula installation where Oomph was disabled because of a Nebula dependency wiring problem. Unfortunately I didn't record the error and created a new installation that worked fine. In any case, it make me start to wonder about these package requirements with bundle-version constraints. In particularly it struck me that I couldn't imagine how they are represented in the p2 metadata. It turns out that they are not expressed in the p2 metadata:

image

So the bundle will happily install but then presumably fail to wire properly. That's certainly not ideal, but I suppose more informative than class loader errors...

But now it also seems of concern that the bundle versions are very poorly maintained. I.e., here the version last change in was in 2017, but we've made what are effectively breaking changes for some consumers:

image

That's definitely not a good thing but not so easy to fix at this point in time...

@wimjongman wimjongman linked a pull request Nov 29, 2024 that will close this issue
@HannesWell
Copy link
Contributor

In any case, it make me start to wonder about these package requirements with bundle-version constraints.

They are legal according to the OSGi spec, although only used seldomly:
https://docs.osgi.org/specification/osgi.core/8.0.0/framework.module.html#framework.module.importpackage

bundle-version - A version-range to select the bundle version of the exporting bundle.
The default value is [0.0.0, ∞). See Semantic Versioning.
In the case of a fragment bundle, the version is from the host bundle.

But personally I usually use the strategy to only use Import-Package if the package is exported with a version. If packages of a bundle are exported without version I usually use Require-Bundle (although in general using Import-Package is preferred for various reasons).

So in this case, since SWT exports it's packages without version I would just use Require-Bundle: org.eclipse.swt. Or is there a specific reason to only import it's packages?

@merks
Copy link
Contributor

merks commented Nov 30, 2024

As I alluded to the other day, some of these widgets are intended to work with RAP/RWT which also exports the SWT packages:

image

But here the bundle version is 4.x, so the bundle-version constraint will exclude that the RAP bundle. Furthermore, the changes that were made to the real SWT have not been realized as corresponding changes in RAP. So I think the RAP support is now "completely broken". Maybe no consumers remain who actually care about that...

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 a pull request may close this issue.

3 participants