Skip to content

[tycho-4.0.x] Fix DependencyComputer missing bundles when they have different versions#2842

Merged
laeubi merged 1 commit intotycho-4.0.xfrom
backport/tycho-4.0.x/pr-2841
Sep 23, 2023
Merged

[tycho-4.0.x] Fix DependencyComputer missing bundles when they have different versions#2842
laeubi merged 1 commit intotycho-4.0.xfrom
backport/tycho-4.0.x/pr-2841

Conversation

@github-actions
Copy link
Copy Markdown

Backport

This will backport the following commits from master to tycho-4.0.x:

Questions ?

Please refer to the Backport tool documentation

Currently the DependencyComputer sorts only by bundle name (even though
the comment for the code mentions the version as well) this can lead to
the situation that if a bundle requires the same bundle in different
versions (e.g. because it has actually different package names like
javax -> jakarta) it only gets one of both on the classpath even though
the P2 input suggests both.

This simply enhances the used key in the mak with the version as already
suggested by the java comment.

(cherry picked from commit 1500f0c)
@akurtakov
Copy link
Copy Markdown
Member

@laeubi should this backport be merged?

@laeubi laeubi merged commit 2980870 into tycho-4.0.x Sep 23, 2023
@laeubi
Copy link
Copy Markdown
Member

laeubi commented Sep 23, 2023

@laeubi should this backport be merged?

Yep!

@akurtakov akurtakov deleted the backport/tycho-4.0.x/pr-2841 branch September 24, 2023 06:09
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