Skip to content
This repository has been archived by the owner on Oct 25, 2018. It is now read-only.

Add <license> section to grizzly-npn pom.xml #1

Merged
merged 1 commit into from
Jul 10, 2017

Conversation

dhalperi
Copy link
Contributor

@dhalperi dhalperi commented Jul 3, 2017

This is copied from https://github.com/javaee/grizzly/blob/master/pom.xml#L101

The LICENSE.txt files in each repo appear to match, although they are line-wrapped differently:

This is copied from https://github.com/javaee/grizzly/blob/master/pom.xml#L101

The LICENSE.txt files in each repo appear to match, although they are line-wrapped differently:

* https://github.com/javaee/grizzly/blob/master/LICENSE.txt for grizzly, is wrapped
* https://github.com/javaee/grizzly-npn/blob/master/LICENSE.txt for grizzly-npn is one-para-per-line.
@rlubke
Copy link
Member

rlubke commented Jul 7, 2017

@dhalperi We'll need a signed OCA before this PR can be merged. Please see [1].

[1] https://github.com/javaee/grizzly-npn/blob/master/CONTRIBUTING.md

@dhalperi
Copy link
Contributor Author

dhalperi commented Jul 7, 2017

@rlubke you need my signature for copying a bit from one of your poms to another? :)

@dhalperi
Copy link
Contributor Author

dhalperi commented Jul 7, 2017

OCA sent.

@rlubke
Copy link
Member

rlubke commented Jul 7, 2017

@dhalperi I don't make the rules and I don't want to get my virtual hand slapped again. :)

@dhalperi
Copy link
Contributor Author

dhalperi commented Jul 10, 2017

My personal OCA came through:

Hello Daniel,

my pleasure to inform you that your OCA has been approved and processed.
Now you should be able to find your name in the official list of Oracle’s
contributors at: http://www.oracle.com/technetwork/community/oca-486395.html
(once the page reloads from the server). Also your account in our bug system
has been updated to reflect the same.

<snip>

regards,
Lenka

@rlubke rlubke merged commit 1d60b60 into javaee:master Jul 10, 2017
@dhalperi dhalperi deleted the patch-1 branch July 10, 2017 16:23
@rlubke
Copy link
Member

rlubke commented Jul 10, 2017

I'll fix the license text file today.

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

2 participants