Issue 26 in atinject: Add OSGi manifest elements

[atin...@googlecode.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 26 by m...@thebishops.org: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

The built jar for the javax.inject API on maven's central repo is not
usable in an OSGi environment because it does not contain the necessary
elements in the META-INF/MANIFEST.MF file. This file should include the
following elements:

Export-Package: javax.inject;version="1.0.0"
Bundle-Version: 1.0.0
Bundle-Name: Dependency Injection for Java
Bundle-ManifestVersion: 2
Bundle-SymbolicName: javax.inject

[atin...@googlecode.com]

Comment #1 on issue 26 by cschneid...@gmail.com: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

I would also like inject to be a OSGi bundle. Should I provide a patch?

[atin...@googlecode.com]

Comment #2 on issue 26 by m...@thebishops.org: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

If you have a patch, that would be great.

[atin...@googlecode.com]

Comment #3 on issue 26 by Ash...@gmail.com: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

Here is a patch for maven build.

Attachments:
osgi.patch 1.8 KB

[atin...@googlecode.com]

Comment #4 on issue 26 by m...@thebishops.org: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

This patch looks good.

[atin...@googlecode.com]

Comment #5 on issue 26 by cgrube...@gmail.com: Add OSGi manifest
elements
http://code.google.com/p/atinject/issues/detail?id=26

Here's a preferred patch - similar, but doesn't require changing the maven
packaging type, and therefore, keeps the "jar" type on the project. This
is not THAT big a deal in a sense, but type is metadata about a maven
artifact and it would be good to keep it, even though "bundle" is
equivalent and you'll still get .jar files in practice.

Attachments:
osgi.patch 2.1 KB

[atin...@googlecode.com]

Comment #6 on issue 26 by m...@thebishops.org: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

"plus 1" (google groups doesn't like + chars?) for Gruber's patch. It's
better than my original one.

[atin...@googlecode.com]

Comment #7 on issue 26 by m...@thebishops.org: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

Oh wait, I didn't submit the original patch.

[atin...@googlecode.com]

Comment #8 on issue 26 by mi...@raner.ws: Add OSGi manifest elements
http://code.google.com/p/atinject/issues/detail?id=26

Any ETA when this patch will be integrated?

Thanks!

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings