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

Use o.e.platform.ide when building the platform-source zip #1360

Merged

Conversation

HannesWell
Copy link
Member

... to replace the removed org.eclipse.platform.sdk

Additionally remove the unused
org.eclipse.platform.releng.tychoeclipsebuilder/rcp/pom.xml. From the history it looks like it was introduced as pom fro the also removed rcp.product.

Follow-up on
#1235

@HannesWell HannesWell force-pushed the replaceRemovedProducts branch from 53e915e to 4fb3852 Compare September 16, 2023 07:43
@merks
Copy link
Contributor

merks commented Sep 16, 2023

@HannesWell

I don't think this build tests what you have changed. E.g., look at this one:

https://ci.eclipse.org/platform/job/eclipse.platform.releng.aggregator/view/change-requests/job/PR-239/

So it's probably more expedient to merge this and start and I-Build which definitely will test these changes...

@HannesWell
Copy link
Member Author

I don't think this build tests what you have changed. E.g., look at this one:

Yes definitely not the change in the platformrepo.xml. But the for the removal of the rcp/pom.xml I would like confirmation that it is not referenced from some remote place. At least the direct parent does not include a rcp module:

<modules>
<module>rcp.config</module>
<module>equinox.starterkit.product</module>
<module>equinox-sdk</module>
<module>eclipse-junit-tests</module>
<module>eclipse.platform.repository</module>
</modules>

... in order to replace the removed org.eclipse.platform.sdk product
definition.

Additionally remove the unused
org.eclipse.platform.releng.tychoeclipsebuilder/rcp/pom.xml.
From the history it looks like it was introduced as pom for the now also
removed rcp.product.

Follow-up on
eclipse-platform#1235
@HannesWell HannesWell force-pushed the replaceRemovedProducts branch from 4fb3852 to c5a2815 Compare September 16, 2023 08:19
@HannesWell HannesWell merged commit c435dff into eclipse-platform:master Sep 16, 2023
2 checks passed
@HannesWell
Copy link
Member Author

Build succeeded. I'm about to kick off a new I-build.

@HannesWell HannesWell deleted the replaceRemovedProducts branch September 16, 2023 08:20
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.

2 participants