[Bug 157] New: slf4j-* uses invalid OSGi version in bundle headers

http://bugzilla.slf4j.org/show_bug.cgi?id=157

           Summary: slf4j-* uses invalid OSGi version in bundle headers
           Product: SLF4J
           Version: 1.5.x
          Platform: PC
        OS/Version: Mac OS X 10.3
            Status: NEW
          Severity: blocker
          Priority: P1
         Component: Unspecified
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


For example, in 1.5.9-RC0, the bundle was given a version of 1.5.9-RC0 which
isn't a valid OSGi version (AFAICT OSGi doesn't support qualifiers).


--
Configure bugmail: http://bugzilla.slf4j.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dev mailing list
[hidden email]
http://www.slf4j.org/mailman/listinfo/dev

http://bugzilla.slf4j.org/show_bug.cgi?id=157


Ralph Goers <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #1 from Ralph Goers <[hidden email]>  2009-10-28 17:22:09 ---
My 2 cents. If it was me I'd mark this bug invalid.1.5.9-RC0 is not an official
release. That is a release candidate. That actual release would be 1.5.9. If
that is an invalid release number in OSGi then this bug would be valid.

However, I imagine it would be helpful for testing if release candidates or
other test releases did use a release number that is compatible with OSGi,
although I'm not sure that that is always possible.


--
Configure bugmail: http://bugzilla.slf4j.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dev mailing list
[hidden email]
http://www.slf4j.org/mailman/listinfo/dev

http://bugzilla.slf4j.org/show_bug.cgi?id=157





--- Comment #2 from Pete Muir <[hidden email]>  2009-10-28 17:33:13 ---
In the interest of user-friendliness, and promoting community testing of
release candidates, I would argue that they should be usable in all places the
release is. But this is a philosophical issue I guess.

My earlier statement was wrong. The issue is that the slf4j bundles uses dash
to separate the version number and qualifier, whilst OSGi expects a period.
Looking at
http://felix.apache.org/site/apache-felix-maven-bundle-plugin-bnd.html the
convention seems to be to do a conversion here.


--
Configure bugmail: http://bugzilla.slf4j.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dev mailing list
[hidden email]
http://www.slf4j.org/mailman/listinfo/dev

http://bugzilla.slf4j.org/show_bug.cgi?id=157


Ceki Gulcu <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #3 from Ceki Gulcu <[hidden email]>  2009-10-28 18:52:52 ---
Fixed in
http://github.com/ceki/slf4j/commit/e967f5135e1bd2eb0d3165ec08cdb7a35bd1b200


--
Configure bugmail: http://bugzilla.slf4j.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dev mailing list
[hidden email]
http://www.slf4j.org/mailman/listinfo/dev

http://bugzilla.slf4j.org/show_bug.cgi?id=157





--- Comment #4 from Pete Muir <[hidden email]>  2009-10-29 11:21:48 ---
Roger Kitain confirmed this fix works.


--
Configure bugmail: http://bugzilla.slf4j.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dev mailing list
[hidden email]
http://www.slf4j.org/mailman/listinfo/dev