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

Update LICENSE file: #3923

Closed
wants to merge 1 commit into from
Closed

Update LICENSE file: #3923

wants to merge 1 commit into from

Conversation

nbougalis
Copy link
Contributor

The existing license file contains copyright statements and license snippets referencing code that has long since been removed from the codebase and which are not necessary any longer.

The general copyright statement for the ISC License is also changed from "Ripple Labs" which is one contributor to the codebase, to the more general "XRP Ledger Developers".

Remaining code which was originally taken from Bitcoin includes the relevant copyright statement(s) inline.

To aid in the automated detection of the license type by GitHub and tools like licensee, the following statement which referenced the ISC license, and which was listed at the bottom of the file, is now incorporated at the top:

The accompanying files incorporate work covered by the following copyright
and previous license notice:

Copyright (c) 2011 Arthur Britto, David Schwartz, Jed McCaleb,
Vinnie Falco, Bob Way, Eric Lombrozo, Nikolaos D. Bougalis, Howard Hinnant

Tagging the people from the list to ensure they see this change and have the opportunity to object, if they want to:

The existing license file contains copyright statements and license
snippets referencing code that has long since been removed from the
codebase and which are not necessary any longer.

The general copyright statement for the ISC License is also changed
from "Ripple Labs" which is one contributor to the codebase, to the
more general "XRP Ledger Developers".

Remaining code which was originally taken from Bitcoin includes the
relevant copyright statement(s) inline.

To aid in the automated detection of the license type by GitHub and
tools like `licensee`, the following statement which referenced the
ISC license, and which was listed at the bottom of the file, is now
incorporated at the top:

>The accompanying files incorporate work covered by the following copyright
>and previous license notice:
>
>Copyright (c) 2011 Arthur Britto, David Schwartz, Jed McCaleb,
>Vinnie Falco, Bob Way, Eric Lombrozo, Nikolaos D. Bougalis, Howard Hinnant
@vinniefalco
Copy link
Contributor

I do not object.

Note that the Boost Software License is the most permissive.

@manojsdoshi manojsdoshi added the Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required. label Sep 8, 2021
@manojsdoshi manojsdoshi mentioned this pull request Sep 9, 2021
@nbougalis nbougalis mentioned this pull request Sep 9, 2021
@nbougalis nbougalis deleted the license branch October 16, 2023 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Passed Passed code review & PR owner thinks it's ready to merge. Perf sign-off may still be required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants