forwardemail / font-awesome-assets

:lipstick: Convert any of @FortAwesome's Font-Awesome icons to an asset, such as an <svg> tag or a Base64-encoded PNG/SVG <img> tag! It supports Retina devices and custom tag attributes too!
https://lad.js.org
MIT License
130 stars 18 forks source link

There may be a licensing problem #1

Closed pmario closed 7 years ago

pmario commented 7 years ago

Hi folks,

Nice work, but I do think there is a licensing problem with the assets: see: http://fontawesome.io/license/

While the CSS stuff is MIT, the /fonts/ directory is not. It's http://scripts.sil.org/OFL . Having a very short look a your code, it seems there is a problem.

Your license is MIT, but I don't see a different one for the extracted svg's. Is it possible to convert SIL to MIT? ... please check!

@davegandy what do you think?


I'm not a lawyer, so the above may be wrong.

niftylettuce commented 7 years ago

I merely have Font Awesome as a dependency.

pmario commented 7 years ago

Hmmm, IMO https://github.com/crocodilejs/font-awesome-assets/tree/master/media/rainbow is a complete republishing, with MIT and not SIL? .

niftylettuce commented 7 years ago

@pmario do I just change the license on this project, or make a note about the fonts being licensed under SIL OFL 1.1? I'll make a note in the Readme right now though.

niftylettuce commented 7 years ago

@pmario see this https://github.com/crocodilejs/font-awesome-assets/commit/3dc66595e5714567cf48a32872c35de91432076f

pmario commented 7 years ago

For me, it looks much better now. ... I'd like to see what @davegandy says.