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

Replace Required-Bundle o.e.osgi.services and remove unused dependencies #155

Merged
merged 2 commits into from
Jun 23, 2022

Conversation

HannesWell
Copy link
Member

No description provided.

@HannesWell HannesWell force-pushed the cleanUpManifest branch 2 times, most recently from 3dffffb to b986921 Compare June 18, 2022 20:15
@vogella
Copy link
Contributor

vogella commented Jun 21, 2022

/rebase

@HannesWell
Copy link
Member Author

Unfortunately I cannot reproduce the failure locally. They always pass, although I use a minimal set of bundles launched into the test application.
Lets see if this is the first or the second commit.

@HannesWell HannesWell force-pushed the cleanUpManifest branch 2 times, most recently from 566e964 to ca86853 Compare June 22, 2022 22:20
@HannesWell
Copy link
Member Author

Btw. it looks like that the GH build succeeds even if there are test-failures. That is very likely because of the -Dmaven.test.failure.ignore=true CLI argument. But there should be a later step that finally fails the build (I think the intention is to run all tests and only fail in the very end to get all results in one build).

@laeubi
Copy link
Contributor

laeubi commented Jun 23, 2022

The repository is missing the junit workflow to parse the junit results and publish the report.

@HannesWell HannesWell merged commit 942b842 into eclipse-platform:master Jun 23, 2022
@HannesWell HannesWell deleted the cleanUpManifest branch June 23, 2022 06:41
@pbodnar
Copy link
Contributor

pbodnar commented Jun 25, 2022

@HannesWell, maybe a dump question, but I'm still new to this Eclipse "setup" stuff... In 4f05fba "Remove unused required bundles" you remove for example the dependency on "org.eclipse.ui.workbench". Yet apparently, this bundle is still required by for example "/org.eclipse.ui.ide/extensions/org/eclipse/ui/dialogs/FilteredResourcesSelectionDialog.java". So do I get it right that the bundle usage is still there, but it is defined just transitively via "/org.eclipse.ui/META-INF/MANIFEST.MF" now?

Due to changes #12 I was about to increase the version of "org.eclipse.ui.workbench" in the modified /org.eclipse.ui.ide/META-INF/MANIFEST.MF, so should I do this in "/org.eclipse.ui/META-INF/MANIFEST.MF" instead?

@HannesWell
Copy link
Member Author

@HannesWell, maybe a dump question, but I'm still new to this Eclipse "setup" stuff... In 4f05fba "Remove unused required bundles" you remove for example the dependency on "org.eclipse.ui.workbench". Yet apparently, this bundle is still required by for example "/org.eclipse.ui.ide/extensions/org/eclipse/ui/dialogs/FilteredResourcesSelectionDialog.java". So do I get it right that the bundle usage is still there, but it is defined just transitively via "/org.eclipse.ui/META-INF/MANIFEST.MF" now?

That's not a dump question (and its arguable if there are any), don't hesitate to ask if something is unclear :)
And yes since org.eclipse.ui requires and reexports org.eclipse.ui.workbench and org.eclipse.ui.ide requires org.eclipse.ui, org.eclipse.ui.ide still requires org.eclipse.ui.workbench transitively.

Due to changes #12 I was about to increase the version of "org.eclipse.ui.workbench" in the modified /org.eclipse.ui.ide/META-INF/MANIFEST.MF, so should I do this in "/org.eclipse.ui/META-INF/MANIFEST.MF" instead?

Yes you should increase the lower bound of the requirement to org.eclipse.ui.workbench in org.eclipse.ui. And IIRC you should also increase the minor version of org.eclipse.ui (that's the bad thing about re-exporting).

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.

4 participants