Skip to content

Add org.eclipse.core.tools to org.eclipse.releng.testsIU#3656

Merged
HannesWell merged 1 commit intoeclipse-platform:masterfrom
merks:pr-add-core-tools-feature-to-category-xml
Feb 19, 2026
Merged

Add org.eclipse.core.tools to org.eclipse.releng.testsIU#3656
HannesWell merged 1 commit intoeclipse-platform:masterfrom
merks:pr-add-core-tools-feature-to-category-xml

Conversation

@merks
Copy link
Copy Markdown
Contributor

@merks merks commented Feb 18, 2026

  • This is the category labeled Eclipse Tests, Tools, Examples, and Extras with description Collection of Misc. Features, such as unit tests, SWT and e4 tools, examples, and compatibility features not shipped as part of main SDK, but which some people may desire in creating products based on previous versions of Eclipse..

eclipse-platform/eclipse.platform#2457

@merks
Copy link
Copy Markdown
Contributor Author

merks commented Feb 18, 2026

@akurtakov @iloveeclipse

For your review and approval...

@merks
Copy link
Copy Markdown
Contributor Author

merks commented Feb 18, 2026

It looks like we really haven't been building the feature, but that doesn't make sense:

image

@HannesWell @laeubi

Is there some chicken-end-egg problem here where we really need to do an I-build for this problem to go away?

@iloveeclipse
Copy link
Copy Markdown
Member

For your review and approval...

The category is OK, exact same like for SWT tools, which is what we want.

Is there some chicken-end-egg problem here where we really need to do an I-build for this problem to go away?

For the rest: no idea.

@iloveeclipse
Copy link
Copy Markdown
Member

If you want, just trigger IBuild right now, we will see. If it fails, we can revert today.

@merks
Copy link
Copy Markdown
Contributor Author

merks commented Feb 18, 2026

If you want, just trigger IBuild right now, we will see. If it fails, we can revert today.

I'll give @HannesWell or @laeubi some time to comment. With the pomless structure I'm not really sure what drives things being built, but given it's parallel to the core.runtime feature, I expect it to just be built...

@HannesWell
Copy link
Copy Markdown
Member

With the pomless structure I'm not really sure what drives things being built, but given it's parallel to the core.runtime feature, I expect it to just be built...

It can either be a an aggregator module, which indeed automatically discovers all PDE type of projects (Bundles, Features, p2-repos, products) in a directory and only that aggregator-directory has to be listed in a pom.xml.
Or, and this seems to be the case here, a project (of whatever type) is listed explicitly in a pom.xml:
https://github.com/eclipse-platform/eclipse.platform/blob/acd8d2e28426241909c776d63a8b2d7391c94428/runtime/pom.xml#L26

Is there some chicken-end-egg problem here where we really need to do an I-build for this problem to go away?

No, it looks like the feature is just not build.
We can either update the linked pom, to list the other feature too or, and that's my suggestion, change the listing and just include the features directory. Then all included projects are build, which in this case, just adds the desired feature.
Once that's on the master of the eclipse.platform repository this build should succeed.

It looks like we really haven't been building the feature, but that doesn't make sense:

Sometimes the unbelievable happens :)

@HannesWell
Copy link
Copy Markdown
Member

I can provide the described fix tomorrow morning.

@HannesWell
Copy link
Copy Markdown
Member

We can either update the linked pom, to list the other feature too or, and that's my suggestion, change the listing and just include the features directory. Then all included projects are build, which in this case, just adds the desired feature.

Done in

- This is the category labeled `Eclipse Tests, Tools, Examples, and
Extras`  with description `Collection of Misc. Features, such as unit
tests, SWT and e4 tools, examples, and compatibility features not
shipped as part of main SDK, but which some people may desire in
creating products based on previous versions of Eclipse.`.

eclipse-platform/eclipse.platform#2457
@HannesWell HannesWell force-pushed the pr-add-core-tools-feature-to-category-xml branch from b8e7c91 to b080141 Compare February 19, 2026 07:00
Copy link
Copy Markdown
Member

@HannesWell HannesWell left a comment

Choose a reason for hiding this comment

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

Build is fine now.

@HannesWell HannesWell merged commit cf93dfc into eclipse-platform:master Feb 19, 2026
4 of 7 checks passed
@iloveeclipse
Copy link
Copy Markdown
Member

Thanks Hannes!

@merks merks deleted the pr-add-core-tools-feature-to-category-xml branch February 19, 2026 07:46
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.

3 participants