Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

replace Require-Bundle with DynamicImport-Package #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

chrisrueger
Copy link

This fixes an class-loading issue in OSGI environments reported in #28 and #22

* replaced Require-Bundle with DynamicImport-Package
* to allow janino-fragment see classes in ch.qos.logback.classic  (instead of just ch.qos.logback.core): e.g. ch.qos.logback.classic.Level 
* also added org.slf4j which is also required to be visible and is currently missing
* needed in OSGI environments
* see qos-ch#28 for details
* this also fixes the change request in qos-ch#22
@chrisrueger chrisrueger changed the title #28 replaced Require-Bundle with DynamicImport-Package replaced Require-Bundle with DynamicImport-Package Feb 2, 2020
@chrisrueger chrisrueger changed the title replaced Require-Bundle with DynamicImport-Package replace Require-Bundle with DynamicImport-Package Feb 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant