[SLING-9860] Merging artifacts with the ALL strategy should keep the origins for clashing bundles and their origins only

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Feature Model 1.2.10
• Fix Version/s: Feature Model 1.2.12
• Component/s: Feature Model
• Assignee: A. J. David Bosschaert
• Reporter: A. J. David Bosschaert
• Created: 28/Oct/20 17:09
• Updated: 24/Nov/20 09:08
• Resolved: 29/Oct/20 11:17

Description

When merging 2 features that both contain the same artifact, but in different versions, both artifacts are preserved, however, the feature-origins of one of the artifact contains both features instead of just its own feature.

For example Feature A contains:

• bundle org.foo:bar:1.2

And Feature B contains

• bundle org.foo:bar:1.3

Then once merged with the ALL strategy the resulting feature contains the artifact with (origins A,B).

While this allows for package substitution to work between the bundles it can be problematic in cases where that is not desired for isolation purposes. As the interpretation of the origins is up to extensions, it seems better to not make the union decision in this place but leave it up to merge extensions and handlers later.

In other words, we should change this so that the origins for version 1.2 should be just A, not both A and B.

Issue Links

• links to: GitHub Pull Request #22