https://bugzilla.redhat.com/show_bug.cgi?id=1425275 Remi Collet <fedora@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fedora@xxxxxxxxxxxxxxxxx --- Comment #1 from Remi Collet <fedora@xxxxxxxxxxxxxxxxx> --- Hmm... Sorry, I don't think I can review this one. I think we have a guidelines about fonts https://fedoraproject.org/wiki/Packaging:FontsPolicy And don't think all the bundled fonts are "free" (e.g. Verdana which is a Microsoft one), if some are not-free, you even have to drop them from the source archive. Perhaps you can package this library without any font. The only ref to the fonts directory is in loadFont function, and if not found, the 'FontName' will be considered as a system font (GD will search in the default system path) Default value may need to be fixed (+ __construct) public $FontName = "GeosansLight.ttf"; Of course, this may require some additional tweaking in the application which rely on this library (to only use system available fonts) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx