Skip to content

Artifacts should be OSGi enabled #158

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

Closed
szeiger opened this issue Nov 27, 2014 · 6 comments
Closed

Artifacts should be OSGi enabled #158

szeiger opened this issue Nov 27, 2014 · 6 comments
Milestone

Comments

@szeiger
Copy link

szeiger commented Nov 27, 2014

This is required for use in OSGi containers and, since this is an API, for any OSGI-enabled implementer of that API. Lack of OSGi support is a blocker for use of Reactive Streams in Slick (which publishes OSGI-enabled artifacts).

I have a working version in https://github.com/szeiger/reactive-streams/compare/osgi-test but this is based on the sbt build which was unfortunately just abandoned. I am clueless when it comes to Gradle.

@viktorklang
Copy link
Contributor

@ktoso
Copy link
Contributor

ktoso commented Nov 27, 2014

Yeah it's going to be this plugin, though I'm a bit confused how exactly to use it (spent 10mins or so on it, so not too much yet..).

// cc @alkemist?

@viktorklang
Copy link
Contributor

There seems to be a bit of issues with it?
http://forums.gradle.org/gradle/topics/the_osgi_plugin_has_several_flaws

ktoso added a commit to ktoso/reactive-streams that referenced this issue Nov 27, 2014
ktoso added a commit to ktoso/reactive-streams that referenced this issue Nov 27, 2014
ktoso added a commit to ktoso/reactive-streams that referenced this issue Nov 27, 2014
ktoso added a commit to ktoso/reactive-streams that referenced this issue Nov 27, 2014
@ldaley
Copy link
Contributor

ldaley commented Nov 27, 2014

The built in OSGi plugin should be adequate for these modest needs.

ktoso added a commit to ktoso/reactive-streams that referenced this issue Nov 28, 2014
@viktorklang
Copy link
Contributor

Fixed by @alkemist with the new Gradle build

@ktoso
Copy link
Contributor

ktoso commented Dec 3, 2014

Wohoo! :-) Thanks @alkemist

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

4 participants