Issue 5047 reproducer

[jan-pesta]

No description provided.

[akurtakov]

This PR looks totally wrong. Please submit only what you have changed/added.

[jan-pesta]

I just reworked it

[jan-pesta]

those test failures indikates the issue with versioning. What next is needed here ?

[laeubi]

those test failures indikates the issue with versioning

Thanks for the testcase!

What next is needed here ?

Someone needs time to pick this up and provide a fix for it.

[github-actions]

Test Results

1 035 files  1 035 suites   5h 37m 37s ⏱️
1 352 tests 1 332 ✅ 20 💤 0 ❌
4 056 runs  3 993 ✅ 63 💤 0 ❌

Results for commit 8e6cd06.

♻️ This comment has been updated with latest results.

[jan-pesta]

I can create fix for it.
I would like to open discussion because this code has been added with #611.
I am not aware why this part of code has been added, because I did not find a reason for such behavior.

I can imagine to keep it backward compatible to restrict fix only if new configuration is specified. e.q.

    @Parameter(name = "stripQualifierIfDuplicates", defaultValue = "true", property = "tycho.buildqualifier.stripduplicatedqualifier")
    protected Boolean stripQualifierIfDuplicates;

[laeubi]

My recommendation would be to add a proposed fix to this PR and then we will see if anything fails and then one might understand it better or even notice that it was unnecessary or ... as we currently preparing a new major release also no immediate need for staying backward compatible.

[akurtakov]

This slipped through the cracks. Please sign your ECA and rebase on master.

[akurtakov]

@laeubi As you have worked on this one, would you please review it?