-
Notifications
You must be signed in to change notification settings - Fork 254
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
New test #117
Comments
I'll need to look at the fonts to see if there's an applicable license. Otherwise, I'll look for something that's compatible with the library license. |
Thank you so much, it helps with packaging |
The AFM files included with Dompdf are for the PDF core fonts. The license for those is in the mustRead.html. I will look at what we need to do for the DejaVu fonts. The UFM files were generated by the Dompdf team and not provided by the DejaVu team. |
I probably missed them, a license file would be great and more explicit
Can you provide a way to re-build them? |
The UFM files are human readable. They're just a delimited text file with a format similar to the AFM format. The files were built using php-font-lib so re-building them is straightforward:
|
Thank you, you can close this issue now or when you cherry-picked my test if you want to |
I'll keep this open so I don't forget to review that test. |
Suggestions on how to handle this in php-font-lib? Intel Clear is licensed under Apache 2.0 Noto fonts are licensed under OFL |
A folder for fonts in tests with a license file in each subfolder? |
- move fonts to tests resources directory - remove Intel Clear-Light - add W3C unit test font Ahem - add new font info test fixes #117
PR #130 adds that new test and modifies the font structure. I also switched from Intel Clear-Light to the W3C font Ahem, which was developed more for layout unit testing but which I thought it may prove useful in future test development since it uses glyph metrics designed for ease of testing. Let me know if you have any feedback. |
- move fonts to tests resources directory - remove Intel Clear-Light - add W3C unit test font Ahem - add new font info test fixes #117
I wrote a new test at Debian, feel free to copy it https://salsa.debian.org/php-team/pear/php-font-lib/-/commit/8eaf263e4e46e058c10092bb2fc8f57d166353d7#c7b64e342fbabe70eb0b971e27dbfec490fccf92_0_54
Or to close this issue :)
Yeah I had to remove the font files because of the missing licences for them in
sample-fonts
The text was updated successfully, but these errors were encountered: