Skip to content

Begin fixing osgi metadata for maven build#1020

Closed
ajuckel wants to merge 2 commits intojruby:masterfrom
ajuckel:fix_osgi
Closed

Begin fixing osgi metadata for maven build#1020
ajuckel wants to merge 2 commits intojruby:masterfrom
ajuckel:fix_osgi

Conversation

@ajuckel
Copy link
Contributor

@ajuckel ajuckel commented Sep 17, 2013

Initial work toward re-enabling OSGi metadata generation in the new maven build.

The bnd configuration wasn't brought over from the old ant build
into the new maven build process.  This is a first (perhaps clumsy)
attempt at getting working metadata added.  At least enough to
satisfy the jruby-osgi-test tests.  Further investigation is needed
to verify:

  1) Nothing is included in the bundle that shouldn't be shipped
  2) The bundle adequately expresses it's required/optional packages
@BanzaiMan
Copy link
Member

Do we need Eclipse metadata?

@ghost ghost assigned mkristian Sep 17, 2013
@BanzaiMan
Copy link
Member

@mkristian Please take a look.

@mkristian
Copy link
Member

I moved the osgi-test suite to be an integration test on maven/jruby-complete (/src/it/osgi-test) and as such it did pass. the actual osgi instrcutions are in the main pom.xml which get shared between: jruby-complete, jruby-stdlib, jruby-core and jruby main artifact (/maven/jruby)

I did not add an integration test to jruby main artifact yet since I was not sure what kind of 'box" I am opening here.

maybe you could leave out those eclipse config files since they are not part of the actual patch.

@ajuckel
Copy link
Contributor Author

ajuckel commented Sep 17, 2013

It seems @mkristian already did everything I was trying to accomplish in this patch, I just wasn't up to date with his recent work. Closing this request as it's unneeded. I'll test out his results instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants