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

Messed up MANIFEST since 3.0.6 #772

Closed
karolkpl opened this issue Mar 8, 2018 · 2 comments
Closed

Messed up MANIFEST since 3.0.6 #772

karolkpl opened this issue Mar 8, 2018 · 2 comments

Comments

@karolkpl
Copy link

karolkpl commented Mar 8, 2018

Since 3.0.6 cometd dosen't work in OSGI environment. Export/Import sections are messed up.
e.g. same import/export packages in bayeux-api

Export-Package = 
	org.cometd.bayeux;version=3.0.10,
	org.cometd.bayeux.server;uses:="org.cometd.bayeux,org.cometd.bayeux.client";version=3.0.10,
	org.cometd.bayeux.client;uses:=org.cometd.bayeux;version=3.0.10
Import-Package = 
	org.cometd.bayeux,
	org.cometd.bayeux.client

I noticed that there were Bnd tool version change between 3.0.5 and 3.0.6
Tool = Bnd-2.4.1.201501161923 for 3.0.5
Tool = Bnd-3.2.0.201605172007 for 3.0.6 and later

@sbordet
Copy link
Member

sbordet commented Apr 25, 2018

When you say that it does not work, have you actually tried, or you are assuming based on the MANIFEST entries ?

I ask because the configuration of the maven-bundle-plugin used to generate the OSGi MANIFEST entries is similar to that of the Jetty project, and we know that Jetty works in OSGi.

The maven-bundle-plugin documentation for the default behavior of Import-Package says:

Any exported packages are also imported by default, to ensure a consistent class space

I have doubts that the super-simple configuration we have for the maven-bundle-plugin generates by default broken MANIFEST entries, so I need a strong confirmation that this issue is real.

@karolkpl
Copy link
Author

Sorry for confusion, my problems were caused by something else

@sbordet sbordet modified the milestone: 3.1.5 Sep 4, 2018
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

No branches or pull requests

2 participants