https://bugs.winehq.org/show_bug.cgi?id=43716
Bug ID: 43716 Summary: freetype 2.8.1 breaks Wine build during font conversion with sfnt2fon Product: Wine Version: 2.17 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: fonts Assignee: wine-bugs@winehq.org Reporter: kinodont@gmail.com Distribution: ---
Distribution: Arch Linux freetype2 2.8.1-1 wine 2.17-1
The command `../tools/sfnt2fon/sfnt2fon -o coue1255.fon ../../wine/fonts/courier.ttf -d 128 13,1255,8` fails with: Error: Cannot open face ../../wine/fonts/courier.ttf
The file courier.ttf exists. The error returned by FT_New_Face is 0x8 (Invalid_Table).
Reverting this freetype commit https://git.savannah.gnu.org/cgit/freetype/freetype2.git/commit/?id=ce367774... fixes the issue. Only the changes in src/sfnt/sfobjs.c are relevant. face->header.Units_Per_EM had the value 13 when displayed in gdb.
https://bugs.winehq.org/show_bug.cgi?id=43716
sl1pkn07 sl1pkn07@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sl1pkn07@gmail.com
--- Comment #1 from sl1pkn07 sl1pkn07@gmail.com --- i can confirm this
../tools/sfnt2fon/sfnt2fon -o coue1255.fon ../../wine/fonts/courier.ttf -d 128 13,1255,8 Error: Cannot open face ../../wine/fonts/courier.ttf make: *** [Makefile:182: coue1255.fon] Error 1
https://bugs.winehq.org/show_bug.cgi?id=43716
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |austinenglish@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=43716
Rafał Mużyło galtgendo@o2.pl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |galtgendo@o2.pl
https://bugs.winehq.org/show_bug.cgi?id=43716
tt_1 herrtimson@yahoo.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |herrtimson@yahoo.de
https://bugs.winehq.org/show_bug.cgi?id=43716
Sebastian Lackner sebastian@fds-team.de changed:
What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.winehq.org/sho | |w_bug.cgi?id=43715
https://bugs.winehq.org/show_bug.cgi?id=43716
Sebastian Lackner sebastian@fds-team.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sebastian@fds-team.de
https://bugs.winehq.org/show_bug.cgi?id=43716
Sebastian Lackner sebastian@fds-team.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |blocker
--- Comment #2 from Sebastian Lackner sebastian@fds-team.de --- Setting importance to blocker. Was this issue already reported to the freetype people? If I don't miss anything, there isn't any good way to add a workaround on the Wine side in this case. To stay compatible with Windows fonts the font size most likely cannot be changed. We could maybe add pre-generated files to the repo, and ask users to patch freetype if they want to do a full / maintainer build...
https://bugs.winehq.org/show_bug.cgi?id=43716
--- Comment #3 from kinodont@gmail.com --- (In reply to Sebastian Lackner from comment #2)
Was this issue already reported to the freetype people?
I've just opened a bug report: https://savannah.nongnu.org/bugs/index.php?52098
https://bugs.winehq.org/show_bug.cgi?id=43716
--- Comment #4 from Nikolay Sivov bunglehead@gmail.com --- UnitsPerEm field has nothing to do with font size. It was likely set to match target size for simplicity, to have global font metrics to have exact predefined values. I don't think there's a reason we can't use common 2048 or 1000 for this field, maybe with some adjustments to sfnt2fon.
https://bugs.winehq.org/show_bug.cgi?id=43716
Fabian Maurer dark.shadow4@web.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4@web.de
https://bugs.winehq.org/show_bug.cgi?id=43716
Hugh McMaster hugh.mcmaster@outlook.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |hugh.mcmaster@outlook.com
--- Comment #5 from Hugh McMaster hugh.mcmaster@outlook.com --- Is this bug severe enough that Debian should *not* release freetype 2.8.1 to its repositories?
The new freetype package is in the 'deferred' queue and can be pushed back indefinitely.
https://bugs.winehq.org/show_bug.cgi?id=43716
Werner Lemberg wl@gnu.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |wl@gnu.org
--- Comment #6 from Werner Lemberg wl@gnu.org --- Created attachment 59267 --> https://bugs.winehq.org/attachment.cgi?id=59267 fonts-courier.ttf (sanitized)
Massaged manually using ttx (from the fonttools bundle).
https://bugs.winehq.org/show_bug.cgi?id=43716
--- Comment #7 from Werner Lemberg wl@gnu.org --- The attached font is a quick try to sanitize `fonts-courier.ttf' – it simply scales everything from 13ppem to 2048ppem, also removing unnecessary data.
Note, however, that 2048 is not a multiple of 13; if you need an *exact* scale you have to use a different (non-standard) ppem value. On the other hand, since this is a bitmap-only font, it probably doesn't matter.
https://bugs.winehq.org/show_bug.cgi?id=43716
--- Comment #8 from sl1pkn07 sl1pkn07@gmail.com --- Error: Cannot open face ../../wine/fonts/fixedsys.ttf
seems this font have the same problem
also:
warning: Courier 13: missing glyph for char 05b0 warning: Courier 13: missing glyph for char 05b1 warning: Courier 13: missing glyph for char 05b2 warning: Courier 13: missing glyph for char 05b3 warning: Courier 13: missing glyph for char 05b4 warning: Courier 13: missing glyph for char 05b5 warning: Courier 13: missing glyph for char 05b6 warning: Courier 13: missing glyph for char 067e warning: Courier 13: missing glyph for char 05b7 warning: Courier 13: missing glyph for char 05b8 warning: Courier 13: missing glyph for char 05b9 warning: Courier 13: missing glyph for char 0679 warning: Courier 13: missing glyph for char 05ba warning: Courier 13: missing glyph for char 05bb warning: Courier 13: missing glyph for char 0686 warning: Courier 13: missing glyph for char 05bc warning: Courier 13: missing glyph for char 0698 warning: Courier 13: missing glyph for char 05bd warning: Courier 13: missing glyph for char 0688 warning: Courier 13: missing glyph for char 05be warning: Courier 13: missing glyph for char 06af warning: Courier 13: missing glyph for char 05bf warning: Courier 13: missing glyph for char 05c0 warning: Courier 13: missing glyph for char 06a9 warning: Courier 13: missing glyph for char 05c1 warning: Courier 13: missing glyph for char 0691 warning: Courier 13: missing glyph for char 05c2 warning: Courier 13: missing glyph for char 05c3 warning: Courier 13: missing glyph for char 06ba warning: Courier 13: missing glyph for char 05f0 warning: Courier 13: missing glyph for char 060c warning: Courier 13: missing glyph for char 05f1 warning: Courier 13: missing glyph for char 06be warning: Courier 13: missing glyph for char 05f2 warning: Courier 13: missing glyph for char 05f3 warning: Courier 13: missing glyph for char 05f4 warning: Courier 13: missing glyph for char 061b warning: Courier 13: missing glyph for char f88d warning: Courier 13: missing glyph for char 061f warning: Courier 13: missing glyph for char f88e warning: Courier 13: missing glyph for char 06c1 warning: Courier 13: missing glyph for char f88f warning: Courier 13: missing glyph for char 0621 warning: Courier 13: missing glyph for char f890 warning: Courier 13: missing glyph for char 0622 warning: Courier 13: missing glyph for char f891 warning: Courier 13: missing glyph for char 0623 warning: Courier 13: missing glyph for char f892 warning: Courier 13: missing glyph for char 0624 warning: Courier 13: missing glyph for char f893 warning: Courier 13: missing glyph for char 0625 warning: Courier 13: missing glyph for char 05d0 warning: Courier 13: missing glyph for char 0626 warning: Courier 13: missing glyph for char 05d1 warning: Courier 13: missing glyph for char 0627 warning: Courier 13: missing glyph for char 05d2 warning: Courier 13: missing glyph for char 0628 warning: Courier 13: missing glyph for char 05d3 warning: Courier 13: missing glyph for char 0629 warning: Courier 13: missing glyph for char 05d4 warning: Courier 13: missing glyph for char 062a warning: Courier 13: missing glyph for char 05d5 warning: Courier 13: missing glyph for char 062b warning: Courier 13: missing glyph for char 05d6 warning: Courier 13: missing glyph for char 062c warning: Courier 13: missing glyph for char 05d7 warning: Courier 13: missing glyph for char 062d warning: Courier 13: missing glyph for char 05d8 warning: Courier 13: missing glyph for char 062e warning: Courier 13: missing glyph for char 05d9 warning: Courier 13: missing glyph for char 062f warning: Courier 13: missing glyph for char 05da warning: Courier 13: missing glyph for char 0630 warning: Courier 13: missing glyph for char 05db warning: Courier 13: missing glyph for char 0631 warning: Courier 13: missing glyph for char 05dc warning: Courier 13: missing glyph for char 0632 warning: Courier 13: missing glyph for char 05dd warning: Courier 13: missing glyph for char 0633 warning: Courier 13: missing glyph for char 05de warning: Courier 13: missing glyph for char 0634 warning: Courier 13: missing glyph for char 05df warning: Courier 13: missing glyph for char 05e0 warning: Courier 13: missing glyph for char 0635 warning: Courier 13: missing glyph for char 05e1 warning: Courier 13: missing glyph for char 0636 warning: Courier 13: missing glyph for char 05e2 warning: Courier 13: missing glyph for char 0637 warning: Courier 13: missing glyph for char 05e3 warning: Courier 13: missing glyph for char 0638 warning: Courier 13: missing glyph for char 05e4 warning: Courier 13: missing glyph for char 0639 warning: Courier 13: missing glyph for char 05e5 warning: Courier 13: missing glyph for char 063a warning: Courier 13: missing glyph for char 05e6 warning: Courier 13: missing glyph for char 0640 warning: Courier 13: missing glyph for char 05e7 warning: Courier 13: missing glyph for char 05e8 warning: Courier 13: missing glyph for char 0641 warning: Courier 13: missing glyph for char 05e9 warning: Courier 13: missing glyph for char 0642 warning: Courier 13: missing glyph for char 05ea warning: Courier 13: missing glyph for char 0643 warning: Courier 13: missing glyph for char f894 warning: Courier 13: missing glyph for char 0644 warning: Courier 13: missing glyph for char f895 warning: Courier 13: missing glyph for char 0645 warning: Courier 13: missing glyph for char f896 warning: Courier 13: missing glyph for char 0646 warning: Courier 13: missing glyph for char 0647 warning: Courier 13: missing glyph for char 0648 warning: Courier 13: missing glyph for char 0649 warning: Courier 13: missing glyph for char 064a warning: Courier 13: missing glyph for char 064b warning: Courier 13: missing glyph for char 064c warning: Courier 13: missing glyph for char 064d warning: Courier 13: missing glyph for char 064e warning: Courier 13: missing glyph for char 064f warning: Courier 13: missing glyph for char 0650 warning: Courier 13: missing glyph for char 0651 warning: Courier 13: missing glyph for char 0652 warning: Courier 13: missing glyph for char 06d2
https://bugs.winehq.org/show_bug.cgi?id=43716
--- Comment #9 from Werner Lemberg wl@gnu.org --- I don't have time to fix all these things, sorry.
Note that I touched neither the cmaps nor the glyphs in `fonts-courier.ttf' (except the `.notdef' outline glyph which has now a sensible shape) – warnings about missing characters must have been there earlier also.
https://bugs.winehq.org/show_bug.cgi?id=43716
--- Comment #10 from sl1pkn07 sl1pkn07@gmail.com --- for the note, the fixedsys.ttf error appears after sustitute the font attached on #6 with the font/courier.ttf and try build again
https://bugs.winehq.org/show_bug.cgi?id=43716
--- Comment #11 from Nikolay Sivov bunglehead@gmail.com --- (In reply to sl1pkn07 from comment #10)
for the note, the fixedsys.ttf error appears after sustitute the font attached on #6 with the font/courier.ttf and try build again
That's because fixedsys.ttf is next in makefile list.
(In reply to sl1pkn07 from comment #8)
Error: Cannot open face ../../wine/fonts/fixedsys.ttf
seems this font have the same problem
also:
warning: Courier 13: missing glyph for char 05b0
I think warnings were always there.
But anyway, we'll need to patch source sfd files, not ttfs.
https://bugs.winehq.org/show_bug.cgi?id=43716
Laurent Carlier lordheavym@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |lordheavym@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=43716
urop6 urop6.u@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |urop6.u@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=43716
blog@pozimski.eu changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |blog@pozimski.eu
https://bugs.winehq.org/show_bug.cgi?id=43716
Hanno Meyer-Thurow h.mth@web.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |h.mth@web.de
https://bugs.winehq.org/show_bug.cgi?id=43716
Robert Walker bob.mt.wya@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |bob.mt.wya@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=43716
Daniel Bermond danielbermond@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |danielbermond@yahoo.com
https://bugs.winehq.org/show_bug.cgi?id=43716
Jonas Jelten jj@stusta.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jj@stusta.net
https://bugs.winehq.org/show_bug.cgi?id=43716
Mario Motohashi ankouslash@yahoo.co.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ankouslash@yahoo.co.jp
https://bugs.winehq.org/show_bug.cgi?id=43716
George Gibbs vash63@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |vash63@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=43716
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|0 |1 CC| |mstefani@winehq.org Status|UNCONFIRMED |NEW
--- Comment #12 from Michael Stefaniuc mstefani@winehq.org --- I have submitted two patches that fix this problem: https://source.winehq.org/patches/data/137429 https://source.winehq.org/patches/data/137430
https://bugs.winehq.org/show_bug.cgi?id=43716
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |2.0.x Fixed by SHA1| |40166848a7944383a4cfdaac9b1 | |8bd03fbb2b4f9 Resolution|--- |FIXED Status|NEW |RESOLVED
--- Comment #13 from Michael Stefaniuc mstefani@winehq.org --- Patches were commited
https://bugs.winehq.org/show_bug.cgi?id=43716
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #14 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 2.18.
https://bugs.winehq.org/show_bug.cgi?id=43716
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|2.0.x |---
--- Comment #15 from Michael Stefaniuc mstefani@winehq.org --- Removing the 2.0.x milestone from bugs included in 2.0.3.