groovy

Add OSGi headers to manifest

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.5.1
  • Fix Version/s: 1.6-rc-2
  • Component/s: None
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    2

Description

Add OSGi headers to the manifest so that the groovy runtime can be loaded in OSGi environments such as Eclipse. This will benefit the Groovy Eclipse plugin and other OSGi-based applications such as Eclipse RCP applications.

Issue Links

Activity

Hide
Dave Dunkin added a comment -

Updated patch that doesn't have list antlr and asm in Bundle-Requires for groovy-all and groovy-all-minimal jars. Also make the Bundle-Version OSGi compliant (0.0.0.q format).

Show
Dave Dunkin added a comment - Updated patch that doesn't have list antlr and asm in Bundle-Requires for groovy-all and groovy-all-minimal jars. Also make the Bundle-Version OSGi compliant (0.0.0.q format).
Hide
Paul King added a comment -

Do we need to have something like ';visibility:=reexport' added to the antlr and org.objectweb.asm dependencies for the vanilla jar?

Show
Paul King added a comment - Do we need to have something like ';visibility:=reexport' added to the antlr and org.objectweb.asm dependencies for the vanilla jar?
Hide
Dave Dunkin added a comment -

I wouldn't think so. That would imply that users of the groovy bundle are going to be accessing the AST directly or modifying bytecode. For my use case in an RCP application I don't need to do any of that.

Show
Dave Dunkin added a comment - I wouldn't think so. That would imply that users of the groovy bundle are going to be accessing the AST directly or modifying bytecode. For my use case in an RCP application I don't need to do any of that.
Hide
Paul King added a comment -

OK, that makes sense. My OSGi is a little rusty - did some early last year but haven't touched it for a while.

Show
Paul King added a comment - OK, that makes sense. My OSGi is a little rusty - did some early last year but haven't touched it for a while.
Hide
Paul King added a comment -

Added. Thanks for the patch.

Note to Jochen: Added to HEAD. I see no reason to not add this to 1.5.2 but I will leave that to you.

Show
Paul King added a comment - Added. Thanks for the patch. Note to Jochen: Added to HEAD. I see no reason to not add this to 1.5.2 but I will leave that to you.
Hide
Paul King added a comment -

close off release 1.5.4

Show
Paul King added a comment - close off release 1.5.4
Hide
Hendy Irawan added a comment -

Related as of Groovy 1.5.6 to GROOVY-2863

Show
Hendy Irawan added a comment - Related as of Groovy 1.5.6 to GROOVY-2863

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: