Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken OSGi bundle wiring and build #3844

Closed
pavelhoral opened this issue Feb 8, 2024 · 0 comments · Fixed by #3845
Closed

Broken OSGi bundle wiring and build #3844

pavelhoral opened this issue Feb 8, 2024 · 0 comments · Fixed by #3845

Comments

@pavelhoral
Copy link
Contributor

Describe the bug

Current build on main is broken so I am creating this issue to have a place for a discussion how to tackle it. The issue is probably caused by recent changes that broke bundle wiring and introduced unwanted Import-Package declarations.

One example of such package dependency (that gets picked up by bndtool) is org.flowable.common.engine.impl.aot.FlowableMyBatisRuntimeHints that uses classes from org.apache.ibatis.javassist.util.proxy package that is not being exported by mybatis bundle.

ERROR: Bundle flowable-osgi [77] Error starting reference:file:target/classes (org.osgi.framework.BundleException: Unable to resolve flowable-osgi [77](R 77.0): missing requirement [flowable-osgi [77](R 77.0)] osgi.wiring.package; (&(osgi.wiring.package=org.flowable.common.engine.impl.javax.el)(version>=7.1.0)(!(version>=8.0.0))) [caused by: Unable to resolve org.flowable.common.engine.impl [15](R 15.0): missing requirement [org.flowable.common.engine.impl [15](R 15.0)] osgi.wiring.package; (osgi.wiring.package=org.apache.ibatis.javassist.util.proxy)] Unresolved requirements: [[flowable-osgi [77](R 77.0)] osgi.wiring.package; (&(osgi.wiring.package=org.flowable.common.engine.impl.javax.el)(version>=7.1.0)(!(version>=8.0.0)))])
org.osgi.framework.BundleException: Unable to resolve flowable-osgi [77](R 77.0): missing requirement [flowable-osgi [77](R 77.0)] osgi.wiring.package; (&(osgi.wiring.package=org.flowable.common.engine.impl.javax.el)(version>=7.1.0)(!(version>=8.0.0))) [caused by: Unable to resolve org.flowable.common.engine.impl [15](R 15.0): missing requirement [org.flowable.common.engine.impl [15](R 15.0)] osgi.wiring.package; (osgi.wiring.package=org.apache.ibatis.javassist.util.proxy)] Unresolved requirements: [[flowable-osgi [77](R 77.0)] osgi.wiring.package; (&(osgi.wiring.package=org.flowable.common.engine.impl.javax.el)(version>=7.1.0)(!(version>=8.0.0)))]
	at org.apache.felix.framework.Felix.resolveBundleRevision(Felix.java:4398)
	at org.apache.felix.framework.Felix.startBundle(Felix.java:2308)
	at org.apache.felix.framework.Felix.setActiveStartLevel(Felix.java:1566)

Expected behavior

OSGi bundles should not depend on non-OSGi packages and tests should pass.

Additional context

I am sure you are looking into this as well as this causes pipeline on main to fail. However as external person I wanted to have a place where I can write my own observations. Originally discussed here #3843 (comment) .

pavelhoral added a commit to pavelhoral/flowable-engine that referenced this issue Feb 8, 2024
pavelhoral added a commit to pavelhoral/flowable-engine that referenced this issue Feb 8, 2024
pavelhoral added a commit to pavelhoral/flowable-engine that referenced this issue Feb 8, 2024
pavelhoral added a commit to pavelhoral/flowable-engine that referenced this issue Feb 8, 2024
filiphr pushed a commit that referenced this issue Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant