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

[Build] Remove duplicated configuration of tycho-p2-extras-plugin #2994

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HannesWell
Copy link
Contributor

@HannesWell HannesWell commented Sep 21, 2024

What it does

Remove duplicated configuration of tycho-p2-extras-plugin. Most parts of the configuration are already inherited from the eclipse-platform-parent:
https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/73b7934ea89cbf0a8b0235572c8e80e84aed0104/eclipse-platform-parent/pom.xml#L372-L388

This would also help in the future in case JDT wants to use the workflow for automated version increments in pull-requests:

Also leverage the new overwritable property to permanently disable the baseline check, which is introduced in

@akurtakov, @iloveeclipse or @jukzi could you please have a look at this?

Author checklist

@jukzi jukzi force-pushed the remove-baseline-comparison-config branch from 6837e7f to d87fb8e Compare September 23, 2024 15:07
@jukzi
Copy link
Contributor

jukzi commented Sep 23, 2024

Remove duplicated configuration of tycho-p2-extras-plugin. Most parts of the configuration are already inherited from the eclipse-platform-parent:

I am not an expert on this, so i have a basic question: Where is it configured that JDT uses eclipse-platform-parent?

@akurtakov
Copy link
Contributor

Remove duplicated configuration of tycho-p2-extras-plugin. Most parts of the configuration are already inherited from the eclipse-platform-parent:

I am not an expert on this, so i have a basic question: Where is it configured that JDT uses eclipse-platform-parent?

https://github.com/eclipse-jdt/eclipse.jdt.core/blob/master/pom.xml#L15
https://github.com/eclipse-jdt/eclipse.jdt.debug/blob/master/pom.xml#L15
https://github.com/eclipse-jdt/eclipse.jdt.ui/blob/master/pom.xml#L15

@HannesWell
Copy link
Contributor Author

Can we hold this off? I'm about to work on an enhancement for this.

@HannesWell
Copy link
Contributor Author

Now it is ready, but requires eclipse-platform/eclipse.platform.releng.aggregator#2381 to succeed.
The advantage is that we can now go full pomless, even with the annotations-v1.

@HannesWell HannesWell marked this pull request as ready for review September 23, 2024 20:43
Most parts of the configuration are already inherited from the
'eclipse-platform-parent'.
And leverage the new overwritable property to permanently disable the
baseline check, which is introduced in
eclipse-platform/eclipse.platform.releng.aggregator#2381
@akurtakov akurtakov force-pushed the remove-baseline-comparison-config branch from 2ab7a15 to a580694 Compare September 24, 2024 07:06
@HannesWell
Copy link
Contributor Author

@akurtakov thanks for the update. The test-failure looks unrelated to me.

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