xiaodududu / google-guice

Automatically exported from code.google.com/p/google-guice
Apache License 2.0
0 stars 0 forks source link

Document the missing OSGI metadata in javax.inject and aopalliance #615

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
The default Guice distro has full OSGI metadata (great!). However the required 
dependencies javax.inject and aopalliance do not. This still requires the user 
to manually generate a manifest for these dependencies, or go to a third-party 
OSGI repo (like Eclipse Orbit) to get the correct jars to install in an OSGI 
run-time.

Please add the metadata in the default distro so a user does not have to do 
this by hand. There are preconfigured jars available at: 
http://download.eclipse.org/tools/orbit/downloads/drops/S20110304120314/

Original issue reported on code.google.com by peerkehe...@gmail.com on 11 Mar 2011 at 9:27

GoogleCodeExporter commented 9 years ago
Is there a particular reason why you can't use the bundles from Eclipse Orbit 
or Spring EBR?

If we did add metadata to the aopalliance and javax.inject jars in the 
distribution then technically the Maven poms should then depend on those 
modified jars, which would then cause duplicate jars for people who don't use 
OSGi and depend on the official aopalliance and javax.inject artifacts 
elsewhere in their application.

Original comment by mccu...@gmail.com on 22 Mar 2011 at 5:56

GoogleCodeExporter commented 9 years ago
No reason, it would just be more convenient. Potential duplicate jars was a 
problem I did not consider. I guess the only way to resolve this would be by 
modifying the official jars.

Maybe a note in the documentation that while Guice is OSGi compliant, supplied 
dependencies are not, and Orbit/Spring EBR are places to go look for them? I 
would be happy to assist in writing some documentation on this.

Original comment by peerkehe...@gmail.com on 23 Mar 2011 at 7:43

GoogleCodeExporter commented 9 years ago
I agree, we should document this, and provide some recommendations for tycho 
usage.

Original comment by cgruber@google.com on 5 Jun 2012 at 2:55

GoogleCodeExporter commented 9 years ago
Issue 704 has been merged into this issue.

Original comment by cgruber@google.com on 6 Jun 2012 at 3:32