search

home-assistant-libs / aioslimproto

SLIMProto implementation in async python allows you to control squeezebox players (and compatibles)
Apache License 2.0
7 stars 5 forks source link

Package embeds non-free fonts #371

Open thomasgoirand opened 2 weeks ago

thomasgoirand commented 2 weeks ago

Hi,

While embedding the DejaVu font in the package is at least irritating, shipping a version of Arial.ttf and Gidole-Regular.ttf render the full package non-free, as per: https://bugs.debian.org/298545#29

I'm attempting to package your module, and added: Depends: fonts-dejavu, fonts-liberation to my package, then I'm doing the necessary symlinks. Note that the filenames your Python module are using are wrong, because different from the one on the Debian package "fonts-dejavu-extra" and "fonts-dejavu-core" that "fonts-dejavu" pulls, so that's annoying too, I had to write a kind of dictionary in my package :/

Please remove these non-free fonts from your Python module, and please use standard names so I can make symlinking more automated.

Thanks for considering the above, Cheers,

Thomas Goirand

marcelveldt commented 2 weeks ago

Next time, try to use a bit less "irritating" "annoying" and use polite sentences.

You think the filenames are wrong, I don't care as it was never intended to follow any standard and especially not follow any debian package names. A font is needed to render content on a screen, thats it.

The nonfree fonts we should fix. I wasn't even aware there were nonfree fonts in this package (last time I checked there werent) - so lets remove the fonts completely and find a font that is free to use and remove support for all other. I am busy atm with other stuff so this has to wait until I have time or somebody PR's it.

May I ask out of curiosity why you are packaging it ?