-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add Manifest data to API jar #1697
Conversation
Clone the maven-bundle-plugin config from impl and modify for the API artifact
<!-- Generate the OSGi jar Manifest. --> | ||
<plugin> | ||
<groupId>org.apache.felix</groupId> | ||
<artifactId>maven-bundle-plugin</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm getting
[WARNING] 'build.plugins.plugin.version' for org.apache.felix:maven-bundle-plugin is missing. @ line 290, column 21
[WARNING]
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If version 5.1.6 is to be used (the latest) note that it needs 2 extra instructions to prevent it from creating a corrupted jar:
<instructions>
<_noimportjava>true</_noimportjava>
<_runee>JavaSE-11</_runee>
</instructions>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry. It ran fine on my machine without that, so I was unaware of the need here. :)
api/pom.xml
Outdated
<Specification-Version>4.0</Specification-Version> | ||
<!-- Set the package version to match the spec version --> | ||
<Export-Package> | ||
javax.faces*;version=4.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this the correct package?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be jakarta.* of course.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Derp. Yes. :P
Clone the maven-bundle-plugin config from impl and modify for the API artifact