search

neovim / tree-sitter-vimdoc

Tree-sitter parser for Vim help files
Apache License 2.0
106 stars 13 forks source link

License inconsistency between crate and LICENSE file #112

Closed jamessan closed 1 year ago

jamessan commented 1 year ago

Cargo.toml states the crate is using the MIT license while LICENSE states Apache-2.0 is the project's license. Is Cargo.toml just an oversight from the default tree-sitter generate output?

clason commented 1 year ago

Probably. @vigoux?

vigoux commented 1 year ago

I have absolutely no idea, I'd guess that the tree-sitter generate, extracts the license frol package.json, so it'd probably be good to look into that

clason commented 1 year ago

I mean, what is the intended license?

vigoux commented 1 year ago

I think I had it to Apache2 for compatibility with neovim's license.

clason commented 1 year ago

Ok, I'll fix the package files when I regenerate the parser with the upcoming tree-sitter version (and make a release).

clason commented 1 year ago

(The vimscript parser has an MIT license, btw.)

clason commented 1 year ago

@jamessan is this blocking you?

jamessan commented 1 year ago

No, just something I noticed while packaging for Debian. For now I've documented both licenses. If a future version unifies to one license, I'll update the Debian metadata.

clason commented 1 year ago

I just pushed a commit fixing the cargo toml. If it helps, I can make a new patch release.