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

Remove DiscoveredPathContainerPage #701

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

jonahgraham
Copy link
Member

The class DiscoveredPathContainerPage used a number of APIs in the Eclipse Platform removed in Eclipse 4.31 release. This class, while API, has not be used, nor usable since the CDT 4 release as it was only for CDT 3.x style projects. The class had been deprecated since 2010.

To mitigate against the possibility that someone may have a dependency on this old class the minor version has been bumped so that version range can have ,8.3) as their upper version.

Fixes #700

The class DiscoveredPathContainerPage used a number of APIs in
the Eclipse Platform
[removed](eclipse-platform/eclipse.platform.ui#1563) in
[Eclipse 4.31 release](https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/81e406456ffae2f82fe0bb244adfdc4121c9e463/eclipse.platform.common/bundles/org.eclipse.platform.doc.isv/porting/removals.html#L595).
This class, while API, has not be used, nor usable since the CDT 4
release as it was only for CDT 3.x style projects.
The class had been deprecated since 2010.

To mitigate against the possibility that someone may have a dependency
on this old class the minor version has been bumped so that version
range can have `,8.3)` as their upper version.

Fixes eclipse-cdt#700
@jonahgraham
Copy link
Member Author

@eclipse-cdt/tools-cdt-committers can a committer review this and provide a second approval to removing this class.

@15knots
Copy link
Contributor

15knots commented Feb 8, 2024

+2

@jonahgraham jonahgraham merged commit 64433f4 into eclipse-cdt:main Feb 8, 2024
5 checks passed
@jonahgraham jonahgraham deleted the deleted-code branch February 8, 2024 20:06
Copy link
Member

@ruspl-afed ruspl-afed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are very fast :)

And what should happen with org.eclipse.cdt.make.internal.core.scannerconfig.DiscoveredPathInitializer?

build/org.eclipse.cdt.make.ui/plugin.xml Show resolved Hide resolved
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.

Platform APIs have been removed breaking CDT build on latest platform dependencies
3 participants