search

akhikhl / wuff

Gradle plugin for automating assembly of OSGi/Eclipse bundles and applications
MIT License
152 stars 51 forks source link

Merge required bundles with respect to modifiers. #80

Closed michalsvagerka closed 9 years ago

michalsvagerka commented 9 years ago

Previously, if MANIFEST.MF contained a Required-Bundle: with resolution:=optional, and wuff inferred that the bundle is required, the mergeRequireBundle received 'org.eclipse.package.name;resolution:=optional' in baseValue and 'org.eclipse.package.name' in mergeValue.

With the way the method worked, it yielded 'org.eclipse.package.name;resolution:=optional,org.eclipse.package.name', effectively marking the bundle as required. This patch addresses this issue by merging those modifiers in a naive way, that solves this issue.