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

Add a NOTICE file which contains the Postgres license #144

Merged
merged 1 commit into from
Aug 21, 2024
Merged

Conversation

JelteF
Copy link
Collaborator

@JelteF JelteF commented Aug 21, 2024

The PostgreSQL license is very liberal but, like MIT and BSD, it does
require attribution and requires that the original license is included.
This adds a NOTICE file containing the license, because we've vendored
in a few files/functions from the Postgres repository and we'll likely need
to vendor in more.

If/when we decide to to merge #143 we should also include PgBouncer its
license. Which is the ISC license, which is also attribution-only.

The PostgreSQL license is very liberal but, like MIT and BSD, it does
require attribution and requires that the original license is included.
This adds a NOTICE file containing the license, because we've vendored
in a few files and we'll likely need to vendor in more.
@wuputah
Copy link
Collaborator

wuputah commented Aug 21, 2024

good catch, thanks!

@JelteF JelteF merged commit f0d41ba into main Aug 21, 2024
@JelteF JelteF deleted the notice-file branch August 21, 2024 19:56
mkaruza pushed a commit that referenced this pull request Sep 11, 2024
The PostgreSQL license is very liberal but, like MIT and BSD, it does
require attribution and requires that the original license is included.
This adds a NOTICE file containing the license, because we've vendored
in a few files/functions from the Postgres repository and we'll likely
need
to vendor in more.

If/when we decide to to merge #143 we should also include PgBouncer its
license. Which is the ISC license, which is also attribution-only.
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.

2 participants