Skip to content
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

tag-expressions: (Java) Make the jar a bundle to support osgi. #99

Merged
merged 1 commit into from May 25, 2017

Conversation

brasmusson
Copy link
Contributor

Summary

Build the tag-expressions jar as a OSGi bundle.

Details

  • Declare the packaging type as bundle
  • Use the maven-bundle-plugin to create the manifest according to the OSGi requirement.

Motivation and Context

cucumber/cucumber-jvm#873 added support to execute Cucumber-JVM in OSGi containers. To not lose that ability when using the tag-expression library in Cucumber-JVM, the tag-expressions jar need to be built as an OSGi bundle.

How Has This Been Tested?

I have locally run the pax-exam example of Cucumber-JVM, which exemplifies testing OSGi bundles, successfully with these changes.

Types of changes

  • New feature (non-breaking change which adds functionality).

@mattwynne mattwynne changed the title [tag-expressions] (Java) Make the jar a bundle to support osgi. tag-expressions: (Java) Make the jar a bundle to support osgi. Mar 4, 2017
@mattwynne mattwynne merged commit afce978 into master May 25, 2017
@mattwynne
Copy link
Member

mattwynne commented May 26, 2017

@aslakhellesoy @brasmusson how do we get this released? Who will take responsibility for doing it?

@mattwynne mattwynne deleted the tagexpressions-osgi-bundle-java branch May 26, 2017 11:56
@aslakhellesoy
Copy link
Contributor

@mattwynne please always update the History/CHANGELOG when you merge a PR - otherwise consumers have a hard time knowing what's new in a given release.

In this case, tag-expressions doesn't even have a change log, so that needs to be added too.

@mattwynne
Copy link
Member

@aslakhellesoy noted. Are you able to make a release? Is seems a lot of other stuff depends on this.

@aslakhellesoy
Copy link
Contributor

I'll make a release when I find some spare cycles.

@lock
Copy link

lock bot commented Oct 24, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants