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

Please add a LICENSE file #45

Closed
edunham opened this issue Aug 17, 2015 · 7 comments
Closed

Please add a LICENSE file #45

edunham opened this issue Aug 17, 2015 · 7 comments

Comments

@edunham
Copy link
Member

edunham commented Aug 17, 2015

Probably dual Apache + MIT like the other Rust stuff?

re: rust-lang/rust#25664

@IanConnolly
Copy link
Contributor

For what it's worth, most Vim plugins declare themselves to be distributed under the same terms as Vim itself is. (:help license)

eg. https://github.com/tpope/vim-fugitive/blame/master/README.markdown#L117-L118

@jordwalke
Copy link

Any update here?

This was referenced Nov 13, 2015
@steveklabnik
Copy link
Member

So, we'd have to make it under apache + MIT for now, because that was what it was written under.

If we can get all of these https://github.com/rust-lang/rust.vim/graphs/contributors people to sign off on changing to the vim license, then we can change it. If we think it's worth it, we can do that work.

@jordwalke
Copy link

Is there anything wrong with Apache+MIT in the mean time?

@edunham
Copy link
Member Author

edunham commented Nov 15, 2015

@steveklabnik Thanks for clarifying that it was written under the same terms as Rust (do they apply by default to everything in the Rust org, or something?). Opened new PR to add the same licenses.

I personally think it's more important to get an open source license into the repo than to bikeshed over which one to use.

@steveklabnik
Copy link
Member

(do they apply by default to everything in the Rust org, or something?)

All the editor stuff was literally in-tree, before I split them out into separate repos, so most of the development was done in rust-lang/rust, hence the licesnse

@edunham
Copy link
Member Author

edunham commented Nov 15, 2015

Excellent. #55 landed, so this is now resolved.

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

No branches or pull requests

4 participants