-
Notifications
You must be signed in to change notification settings - Fork 144
Fix for #89 - clarify font licensing #90
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
Conversation
Didn't spot the tests... They should work now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@LyesSaadi, can I ask if you were able to review the moves of the fonts themselves for conformance with your expectations? I am happy to accept this so long as there are multiple opinions as to whether the partitioning of fonts is correct.
The other changes look good to me, aside from a minor typo.
Typo fixed. All ready to go... Just waiting for LyesSaadi. |
Please feel free to shamelessly ping this thread weekly. The best time to do so is 2100 UTC on Thursdays, which maximizes the likelihood of my chances of responding over the weekend. Thanks to all involved, and if I don't respond timely it's not that I don't consider this effort valuable but just that I'm swamped :) |
So, I looked into each individual font file again. Every single file, except the FIGlet fonts, have their license included in the header, which is great. So, the only thing missing would be to add FIGlet's BSD, as that is a requirement from the License (Unless those files are not under BSD, but I remember reading that they were indeed subject to it.):
It is common to add such licenses in the same directory (or in a "licenses" subdirectory) of the copyrighted fonts, as well as the list of fonts in question in the License text or in a separate file. Also, I notice that some fonts from FIGlet are missing. This is just something I remarked, and nothing to do with this PR, so if this is a deliberate choice, please just ignore this remark. |
So... I think you're saying I should insert the original BSD license in the standard figlet font files (and check again what flf files are missing). Right? If so, I can probably do that over the weekend... BTW, I don't think we need the flc files as these are just character mappings for non-standard code pages (which pyfiglet has been working with happily for many a year). |
Just dropping the License file somewhere is enough. Putting it in the header could be practical for consistency, but is not mandatory. It's just that it is a requirement for BSD. |
OK - I've now updated each font file with the original BSD license and tested they still work on a new build. I also took the latest files from figlet just to make sure that we weren't accidentally pulling in dodgy versions from elsewhere. This added a new character to the standard font (in case you spotted that!). I think that's it! |
@LyesSaadi - just checking... Are you happy with the license update? |
Oops, sorry, I forgot to answer. And to answer, yes, I'm happy with all that 🎉! That was awesome work from you! I will also try & re-open the discussion with Fedora Legal later this week! |
@pwaller - looks like we're good to go... Are you happy with the new changes? |
Thanks both.
Can I check what open questions remain here? I take it we're good to go? |
Yes, we're good to go! I'm only referring to the remaining fonts (those in the This was a question which was actually already opened in the past with the figlet package (~2006-2012), but I couldn't find any conclusion from Fedora Legal toward the question, and the figlet package ended up being exclusively distributed with Open-Source fonts. But it could also be that the discussion was just lost, as Fedora changed its means of internal communication since. |
Hopefully this is everything we agreed...
I didn't go for the BDF files in the end as I wasn't so sure about the legality, while the other explicit license text was much more reassuring.
Are we good to go?