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

Not all plugins provided by CDT is included in a CDT feature #222

Open
Torbjorn-Svensson opened this issue Dec 14, 2022 · 6 comments
Open

Comments

@Torbjorn-Svensson
Copy link
Member

I noticed that installing an unrelated feature will query the https://download.eclipse.org/tools/cdt/releases/latest update site and download updates for the following plugins:

com.sun.xml.bind_2.3.3.v20221112-0806.jar
jakarta.xml.bind_2.3.3.v20221112-0806.jar
javax.activation_1.2.2.v20221112-0806.jar
javax.xml_1.4.1.v20220503-2331.jar
org.eclipse.tools.templates.core_1.3.0.202211062329.jar
org.eclipse.tools.templates.freemarker_1.3.0.202211062329.jar
org.eclipse.tools.templates.ui_1.4.0.202211062329.jar

It did also download updates for these 2 plugins, but they are covered by 2 features, so not really the same problem.
com.google.gson_2.9.1.v20220915-1632.jar
org.eclipse.launchbar.core_2.5.0.202211062329.jar

For the launchbar, the containing feature can't be used as the core functionality is required by a lot of plugins in CDT, but we do not want the launchbar UI.

@ruspl-afed
Copy link
Member

I would say that bundles like javax.xml should not be included to any CDT feature at all, but rather to the p2 site. This is required to let the "end-product" do decide which 3rd party version to include. For example, if 3rd party needs to be undated due to a fixed CVE.

@Torbjorn-Svensson
Copy link
Member Author

I somewhat agree on the javax.xml, as they included in the CDT repository, but not built by us.
For org.eclipse.tools.templates.*, it's a different story as we (CDT) are building and providing these plugins.

@ruspl-afed
Copy link
Member

I agree for org.eclipse.tools.templates.* and all others that CDT is building from its own sources, they definitely should belong to a CDT feature.

@jonahgraham
Copy link
Member

Not sure I understand what the bug is here.

Is it that we have bundles we depend on that aren't listed in feature.xmls? Do we need to open each feature.xml and in the Dependencies tab press Compute? Can this be automated as part of the build? Can tycho take this on as a feature request?

@Torbjorn-Svensson
Copy link
Member Author

Not sure I understand what the bug is here.

Is it that we have bundles we depend on that aren't listed in feature.xmls? Do we need to open each feature.xml and in the Dependencies tab press Compute? Can this be automated as part of the build? Can tycho take this on as a feature request?

Yes, there is no feature.xml file that mentions the org.eclipse.tools.templates.* plugins, so there is no way to lock down these based on a given CDT release. The downstream consumer would need to know the internal plugin dependencies in order to lock down these plugins.

@ruspl-afed
Copy link
Member

Not sure I understand what the bug is here.

We need to choose a concrete CDT feature to bring our org.eclipse.tools.templates.* plugins to the world

Is it that we have bundles we depend on that aren't listed in feature.xmls?

We do, and a lot of them, but I don't see an issue here.

Do we need to open each feature.xml and in the Dependencies tab press Compute?

Better to avoid this action, not sure that result will make us happy

Can this be automated as part of the build?

feature.xml is already a manifest for the automated build

Can tycho take this on as a feature request?

This is a kind of msiconfiguration on CDT side, nothing to do for Tycho here.

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

No branches or pull requests

3 participants