Skip to content

LICENSE (the MIT License) and AUTHORS #18

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

Merged
merged 1 commit into from
Sep 5, 2014

Conversation

sprin
Copy link
Contributor

@sprin sprin commented Sep 3, 2014

@yosifkit
Copy link
Member

yosifkit commented Sep 3, 2014

LGTM

/cc @tianon @jberkus

@yosifkit
Copy link
Member

yosifkit commented Sep 4, 2014

According to the mailing list, @jberkus is a go on the postgresql license. I'll count that as a LGTM from him, unless @jberkus otherwise objects here.

so @tianon ok to merge?

@tianon
Copy link
Member

tianon commented Sep 4, 2014

Is that copyright even meaningful? I don't think "Docker PostgreSQL Maintainers" is a legal entity of any kind, is it?

Also, I don't have a problem with the license, but it seems kind of silly that it's essentially just a custom version of Expat/MIT/X11.

@tianon
Copy link
Member

tianon commented Sep 4, 2014

(in other words, LGTM)

@jberkus
Copy link

jberkus commented Sep 4, 2014

tianon: yeah, we could also use MIT or BSD for this. The only reason The PostgreSQL License has different text is that it's a legacy license which can't be changed for the PostgreSQL Project, because it's inherited from UC Berkeley.

@sprin sprin changed the title LICENSE (the PostgreSQL License) LICENSE (the MIT License) and MAINTAINERS Sep 4, 2014
@sprin
Copy link
Contributor Author

sprin commented Sep 4, 2014

Switched to MIT, and added tianon and yosifkit as maintainers (valid legal entities, I hope!).

@jberkus
Copy link

jberkus commented Sep 5, 2014

Please add "other contributors not specifically named here" at the bottom. That way we don't have to update the license every time we accept a new PR.

@tianon
Copy link
Member

tianon commented Sep 5, 2014

Or maybe reference a separate AUTHORS file for the copyright since maintainers implies more than just copyright. ;)

(But I'm still +1 on "other contributors", perhaps even with a link to GitHub's contributors list)

@sprin
Copy link
Contributor Author

sprin commented Sep 5, 2014

Changed MAINTAINERS to AUTHORS and added to AUTHORS "And other contributors not specifically named here."

@sprin sprin changed the title LICENSE (the MIT License) and MAINTAINERS LICENSE (the MIT License) and AUTHORS Sep 5, 2014
@tianon
Copy link
Member

tianon commented Sep 5, 2014

LGTM

yosifkit added a commit that referenced this pull request Sep 5, 2014
LICENSE (the MIT License) and AUTHORS
@yosifkit yosifkit merged commit 9cc263e into docker-library:master Sep 5, 2014
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.

4 participants