Skip to content

Conversation

@Vogel612
Copy link

@Vogel612 Vogel612 commented Aug 22, 2018

Removed slf4j-ext from winery, replacing usages of XLogger with usages of Logger.
Also explicitly export slf4j dependencies to OSGi consumers in the version used in winery.

Additional fixes include corrections to some incorrect MANIFEST.MF files (de.danielbechler:java-object-diff).
This fixes the remainder of issues encountered with OSGi

Reviews are highly appreciated.

@clarity-bot
Copy link

clarity-bot bot commented Aug 22, 2018

I generated a structure-diff below from pulling a7102bb on OpenTOSCA:fix/osgi-slf4j-issues into 02c11c8 on OpenTOSCA:ustutt.

Structure Diff

@Vogel612
Copy link
Author

Possibly of note: the issue with the incorrect MANIFEST for java-object-diff has a pending PR upstream to fix the root problem, cf. SQiShER/java-object-diff#206

@Vogel612 Vogel612 force-pushed the fix/osgi-slf4j-issues branch from 8d8ae9c to eb4a9fc Compare August 22, 2018 13:31
@Vogel612 Vogel612 changed the title Fix/osgi slf4j issues (WIP) Fix/osgi slf4j issues Aug 22, 2018
Export slf4j-logging deps for consumption by container.
Update OSGi specs to deal with optional dependencies and incorrect MANIFEST files.
Add variable to enable quick versioning changes to exported OSGi artifacts.

Signed-off-by: Clemens Lieb <[email protected]>
@miwurster
Copy link

Please raise this PR in eclipse/winery. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants