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

Pr 2978 #2989

Merged
merged 2 commits into from
Nov 3, 2023
Merged

Pr 2978 #2989

merged 2 commits into from
Nov 3, 2023

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Nov 3, 2023

This includes the test and a fix for

Virtual configuration IU in p2.inf does not work in 4.0.3 (works in 3.0.5)
@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Nov 3, 2023
Copy link

github-actions bot commented Nov 3, 2023

Test Results

   570 files  +3     570 suites  +3   4h 24m 32s ⏱️ + 25m 31s
   370 tests +1     364 ✔️ +2    6 💤 ±0  0 ±0 
1 110 runs  +3  1 091 ✔️ +4  19 💤 ±0  0 ±0 

Results for commit 1f5e135. ± Comparison against base commit 811d389.

♻️ This comment has been updated with latest results.

Currently units from p2.inf are not published to the target-platform
what can fail resolving when these are referenced in the project.

This publishes these units into the target-platform state of the
project.

Fix eclipse-tycho#2977
@laeubi
Copy link
Member Author

laeubi commented Nov 3, 2023

Jenkins testfailure is an instability from

@laeubi laeubi merged commit 32f6249 into eclipse-tycho:master Nov 3, 2023
9 of 10 checks passed
@mdaloia
Copy link
Contributor

mdaloia commented Nov 3, 2023

@laeubi that was a really fast resolution!!! Thank you very much.

Is there any plan to release a 4.0.4 soon? Just asking to plan ahead on my side.

}
AdviceFileAdvice advice;
if (PackagingType.TYPE_ECLIPSE_PLUGIN.equals(reactorProject.getPackaging())) {
advice = new AdviceFileAdvice(reactorProject.getArtifactId(), Version.parseVersion("1.0.0"),
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this hardcoded version 1.0.0 irrelevant? what if the declared unit sets its version to anything else?
(same below on the else branch of the if)

Copy link
Member Author

Choose a reason for hiding this comment

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

The version is only used to check if the advice should be applied to a unit, as we are not interested in the advice itself but only the additional units it defines the version has actually no influence here.

@laeubi
Copy link
Member Author

laeubi commented Nov 3, 2023

The plan was to release it last week :-D

But I considered this important enough to delay it by some days, if nothing crucial came up I'll probably make a release at the weekend or on Monday.

mdaloia added a commit to mdaloia/tycho that referenced this pull request Nov 10, 2023
These changes show that the fix for issue eclipse-tycho#2977 made in PR eclipse-tycho#2989 was
partial.

When there is another bundle/plugin that requires the bundle which
contains the p2.inf file that provides and requires the virtual IU the
original error is triggered again even with Tycho 4.0.4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants