Skip to content

Proposal for LICENSE file #765

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 2 commits into from
Nov 7, 2018
Merged

Proposal for LICENSE file #765

merged 2 commits into from
Nov 7, 2018

Conversation

phillmv
Copy link
Contributor

@phillmv phillmv commented Oct 28, 2018

The lack of an explicit license is holding back a potential contributor from submitting patches: #751 (comment)

Looking thru the macvim repo, and from what appears to be common practice within the vim community, seems to indicate that the vim license in uganda.txt is considered a widely adopted default.

Consequently, I took the legalese from uganda.txt and |manual-copyright| and created a new LICENSE file.

Hope this helps,

@ychin
Copy link
Member

ychin commented Nov 5, 2018

Thanks for the PR. I left some review comments mostly to clean it up a little bit.

@phillmv
Copy link
Contributor Author

phillmv commented Nov 5, 2018

@ychin Hrm, I don't see any comments - is it possible you didn't click "Add review" 😄?

Thanks!

@ychin
Copy link
Member

ychin commented Nov 6, 2018

Yes I did forget to. I added the comments.

@phillmv
Copy link
Contributor Author

phillmv commented Nov 6, 2018

Since the purpose of this file is to indicate the license that applies to the code,

I removed both the Summary section as proposed by @ychin and also the manual copyright declaration since it's still present in |manual-copyright|.

@ychin ychin merged commit eda162c into macvim-dev:master Nov 7, 2018
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