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

chore(OSGi): switch to biz.aQute.bnd plugin #855

Merged
merged 1 commit into from
Oct 22, 2019

Conversation

Fiouz
Copy link
Contributor

@Fiouz Fiouz commented Oct 19, 2019

On a side note, the Bundle-SymbolicName conflict reported at #772 still
exists.

- Native Gradle OSGi integration is deprecated in Gradle 5.x
  https://docs.gradle.org/5.0/release-notes.html#deprecated-osgi-plugin
- Use OSGi annotations to lower manifest synchronization cost
  https://bnd.bndtools.org/chapters/230-manifest-annotations.html
- Adjusted Export-Package for asciidoctorj-api:
  - 'org.asciidoctor.spi' removed as it seems an overlook of bc75314
  - added org.asciidoctor.syntaxhighlighter as generation was
    reporting the following warning:
    warning: Export org.asciidoctor,  has 1,  private references [org.asciidoctor.syntaxhighlighter]
    (may have been an overlook of 3e9d47e)

On a side note, the Bundle-SymbolicName conflict reported at
asciidoctor#772 still exists.
@robertpanzer
Copy link
Member

I am not really that much into OSGi, so I have a hard time testing this.
But the manifest of asciidoctorj.jar looks good.
Merging.

@robertpanzer robertpanzer merged commit 6b7369f into asciidoctor:master Oct 22, 2019
@robertpanzer
Copy link
Member

Thanks! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants