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

Fix master build #96

Merged
merged 1 commit into from
Feb 6, 2023
Merged

Fix master build #96

merged 1 commit into from
Feb 6, 2023

Conversation

akurtakov
Copy link
Member

As can be seen at
https://ci.eclipse.org/platform/job/eclipse.platform.ua/job/master/109/console :

[ERROR] Cannot resolve project dependencies:
[ERROR]   Software being installed: org.eclipse.jdt.feature.group
3.19.0.v20230205-1800
[ERROR]   Missing requirement: org.eclipse.jdt 3.19.0.v20230204-1800
requires 'osgi.bundle; org.eclipse.ui.cheatsheets [3.2.0,4.0.0)' but it
could not be found
[ERROR]   Missing requirement: org.eclipse.jdt 3.19.0.v20230205-1800
requires 'osgi.bundle; org.eclipse.ui.cheatsheets [3.2.0,4.0.0)' but it
could not be found
[ERROR]   Cannot satisfy dependency: org.eclipse.jdt.feature.group
3.19.0.v20230204-1955 depends on: org.eclipse.equinox.p2.iu;
org.eclipse.jdt [3.19.0.v20230204-1800,3.19.0.v20230204-1800]
[ERROR]   Cannot satisfy dependency: org.eclipse.jdt.feature.group
3.19.0.v20230205-1800 depends on: org.eclipse.equinox.p2.iu;
org.eclipse.jdt [3.19.0.v20230205-1800,3.19.0.v20230205-1800]

As can be seen at
https://ci.eclipse.org/platform/job/eclipse.platform.ua/job/master/109/console
:
```
[ERROR] Cannot resolve project dependencies:
[ERROR]   Software being installed: org.eclipse.jdt.feature.group
3.19.0.v20230205-1800
[ERROR]   Missing requirement: org.eclipse.jdt 3.19.0.v20230204-1800
requires 'osgi.bundle; org.eclipse.ui.cheatsheets [3.2.0,4.0.0)' but it
could not be found
[ERROR]   Missing requirement: org.eclipse.jdt 3.19.0.v20230205-1800
requires 'osgi.bundle; org.eclipse.ui.cheatsheets [3.2.0,4.0.0)' but it
could not be found
[ERROR]   Cannot satisfy dependency: org.eclipse.jdt.feature.group
3.19.0.v20230204-1955 depends on: org.eclipse.equinox.p2.iu;
org.eclipse.jdt [3.19.0.v20230204-1800,3.19.0.v20230204-1800]
[ERROR]   Cannot satisfy dependency: org.eclipse.jdt.feature.group
3.19.0.v20230205-1800 depends on: org.eclipse.equinox.p2.iu;
org.eclipse.jdt [3.19.0.v20230205-1800,3.19.0.v20230205-1800]

```
@akurtakov
Copy link
Member Author

@laeubi Is this caused by recent change in Tycho? Also do you have recommendation how to simplify this extra requirements list.

@akurtakov
Copy link
Member Author

The target-platform-configuration is totally insane here and I have no idea how/why these bundles are there. Merging but this would better be revised to something manageable.

@akurtakov akurtakov merged commit 32d3a4f into eclipse-platform:master Feb 6, 2023
@laeubi
Copy link
Contributor

laeubi commented Feb 6, 2023

The target-platform-configuration is totally insane here

Yep, but your change looks reasonable.

If I remember correctly this because we collect from different actors items in one big artifact, probably it would getting better if we merge more repositories and thus remove inter-dpendency-cycles.

@laeubi
Copy link
Contributor

laeubi commented Feb 6, 2023

Also do you have recommendation how to simplify this extra requirements list.

Tycho now support them to adding by the "additional.bundles" aka "Automatically managed dependencies" in PDE, that has the advantage that one can "manage" them from the UI instead of the pom as done here:
https://github.com/eclipse-platform/eclipse.platform/blob/489bf9a096c1d2ad2299c596707e8656568f16b0/update/org.eclipse.update.configurator.tests/build.properties

@laeubi
Copy link
Contributor

laeubi commented Feb 7, 2023

I have investigated a bit more and it seems that there is an issue with the new resolver regarding test-dependencies are sometimes not found I can reproduce this also with equinox (what only use a simple target-platform configuration), I'll investigate this and seems I need to disable the resolver for now see:

@laeubi
Copy link
Contributor

laeubi commented Feb 7, 2023

By the way I think initially this was intended to be handled by the following PDE feature:

but it seems it is currently broken, but I plan to revive this feature and add support for it (together with @HannesWell ) as this would solve some other cases as well.

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