Linguist: Find grammars to replace those removed in #2884

Created on 19 Mar 2016  路  13Comments  路  Source: github/linguist

Most helpful comment

@copumpkin @wmertens #2914 is now live on GitHub.

All 13 comments

Why were https://github.com/nanoant/assembly.tmbundle and https://github.com/idleberg/InnoSetup-Sublime-Text removed? They seem to be under MIT license :/

Good catch. I'll add those back. Not sure why I removed those...

What's wrong with grammars under non-permissive licenses? In README you state:

The language grammars included in this gem are covered by their repositories' respective licenses.

I'm no lawyer, but it seems the worst thing that could happen when using linguist with (say) a GPL-licensed grammar is that together the license would be GPL. Why not keep that as an option, at least?

@vcunat - non-permissive licenses aren't compatible with the ways in which Linguist is distribute so we need all included grammars to have a permissive license I'm afraid.

I see.

@arfon looks like @wmertens moved the nix highlighter to MIT. Can we get it reinstated on the live site? I use GitHub to browse Nix files a lot and it makes a big difference.

Yep. I will get to this early next week. We need to cut a new gem and deploy before this can be re-instated.

Thanks!

@copumpkin @wmertens #2914 is now live on GitHub.

@arfon @wmertens awesome, thank you both for dealing with this so quickly! The Nix community thanks you :smile:

@arfon @wmertens awesome, thank you both for dealing with this so quickly! The Nix community thanks you :smile:

馃憤 my pleasure, sorry for the disruption.

We've done pretty well here. Closing this out for now.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

lucasrodes picture lucasrodes  路  6Comments

pfitzseb picture pfitzseb  路  5Comments

oldmud0 picture oldmud0  路  6Comments

siscia picture siscia  路  6Comments

Alhadis picture Alhadis  路  5Comments