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

UI build fails with "The re-exported type org.eclipse.swt.ole.win32.OLE has been removed from org.eclipse.jface_3.32.0" #1654

Closed
iloveeclipse opened this issue Feb 7, 2024 · 8 comments · Fixed by eclipse-tycho/tycho#3484
Labels
bug Something isn't working help wanted Extra attention is needed regression

Comments

@iloveeclipse
Copy link
Member

See #1653 and https://ci.eclipse.org/platform/job/eclipse.platform.ui/job/PR-1653/1/console

10:25:14.707 [ERROR] Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.6-SNAPSHOT:verify (verify) on project org.eclipse.jface: There are API errors:
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OLE has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleAutomation has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleClientSite has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleControlSite has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleEvent has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleFrame has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleFunctionDescription has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleListener has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OleParameterDescription has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.OlePropertyDescription has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:0 The re-exported type org.eclipse.swt.ole.win32.Variant has been removed from org.eclipse.jface_3.32.0
10:25:14.707 [ERROR] META-INF/MANIFEST.MF:5 The major version should be incremented in version 3.33.0, since API breakage occurred since version 3.32.0
@iloveeclipse
Copy link
Member Author

The re-exported type org.eclipse.swt.ole.win32 - I don't see why PDE API complains about a package that was never exported by SWT main bundle? It was and is only part of native fragment on Windows.

@laeubi
Copy link
Contributor

laeubi commented Feb 7, 2024

The re-exported type org.eclipse.swt.ole.win32 - I don't see why PDE API complains about a package that was never exported by SWT main bundle? It was and is only part of native fragment on Windows.

Seems like API tools does not consider Platfom filters correctly.

@iloveeclipse
Copy link
Member Author

@HannesWell : I wonder if eclipse-pde/eclipse.pde#1094 helps in any way here as it seem to remove SWT special handling from PDE API tooling? Or rather we should add something additional to avoid SWT API from different platform be checked?

@iloveeclipse
Copy link
Member Author

Seems like API tools does not consider Platfom filters correctly.

This must be recently introduced or caused by recent SWT project or build restructuring.

@laeubi
Copy link
Contributor

laeubi commented Feb 7, 2024

I'll take a look if it could be fixed on Tycho side assuming we dont see this in the IDE?

@iloveeclipse
Copy link
Member Author

Thanks. Is this Tycho change visible immediately, or do we need to wait some days to see the fixed Tycho build?

@HannesWell
Copy link
Member

HannesWell commented Feb 7, 2024

Thanks. Is this Tycho change visible immediately, or do we need to wait some days to see the fixed Tycho build?

I just submitted the backport and its build should be completed in about an hour or two or less (don't know the exact runtimes).

Then it depends how fast the verification builds update the snapshot dependencies/plugins. The latest it should be every 24h, but if -U is passed it should be immediately.

Regarding the cause, Christoph can probably tell best, but to me it looks like the enhancement from eclipse-tycho/tycho#3457 in combination with the re-export of SWT by jface let the APItools find more packages in the baseline than actually exist.

@laeubi
Copy link
Contributor

laeubi commented Feb 8, 2024

Thanks. Is this Tycho change visible immediately, or do we need to wait some days to see the fixed Tycho build?

Here is the build job of the tycho-4 line: https://ci.eclipse.org/tycho/job/tycho-github/job/tycho-4.0.x/

As you can see this specific fix is/was available from 07.02.2024, 19:02 CEST + roughly 2.5hrs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed regression
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants