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

PAYARA-2341 repackage JCIP annotations with OSGi headers #2214

Merged
merged 2 commits into from
Dec 21, 2017
Merged

PAYARA-2341 repackage JCIP annotations with OSGi headers #2214

merged 2 commits into from
Dec 21, 2017

Conversation

arjantijms
Copy link
Contributor

PAYARA-2341 creates a JCIP jar with the OSGi headers in MANIFEST.MF, but does not yet add this as a dependency to Payara.

@arjantijms
Copy link
Contributor Author

jenkins test please

@payara-ci
Copy link
Contributor

Quick build and test passed!

@smillidge
Copy link
Contributor

Can you add an entry in https://github.com/payara/Payara/blob/master/appserver/packager/legal/src/main/resources/glassfish/legal/PAYARA-3RD-PARTY-LICENSE.txt if this is a new 3rd party dependency being packaged into Payara

<artifactItem>
<groupId>com.github.stephenc.jcip</groupId>
<artifactId>jcip-annotations</artifactId>
<version>1.0-1</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please define and use a version property from appserver/pom.xml

@arjantijms
Copy link
Contributor Author

jenkins test please

@payara-ci
Copy link
Contributor

Quick build and test passed!

@arjantijms arjantijms dismissed Pandrex247’s stale review December 21, 2017 23:03

feedback addressed

@arjantijms arjantijms merged commit 149e656 into payara:master Dec 21, 2017
@arjantijms arjantijms added this to the Payara 4.181 milestone Dec 21, 2017
@arjantijms arjantijms deleted the PAYARA-2341-jcip-osgi branch December 21, 2017 23:13
michaelranaldo pushed a commit to michaelranaldo/Payara that referenced this pull request Jan 8, 2018
* PAYARA-2341 repackage JCIP annotations with OSGi headers

* PAYARA-2341 Addressing feedback
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.

6 participants