[Bug 28097] New: gdiplus fonts test fails unless Times New Roman installed?
http://bugs.winehq.org/show_bug.cgi?id=28097 Summary: gdiplus fonts test fails unless Times New Roman installed? Product: Wine Version: 1.3.26 Platform: x86 OS/Version: Linux Status: NEW Severity: normal Priority: P2 Component: gdiplus AssignedTo: wine-bugs(a)winehq.org ReportedBy: dank(a)kegel.com "make test" is failing for me on a fresh Ubuntu 11.04 install with font.c:264: Tests skipped: Times New Roman not installed font.c:322: Test failed: Expected Times New Roman, got L"Liberation Serif" It looks like the check just above line 322 is wrong... -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=28097 André H. <nerv(a)dawncrow.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nerv(a)dawncrow.de --- Comment #1 from André H. <nerv(a)dawncrow.de> 2011-08-16 14:28:23 CDT --- i already sent a patch for this problem last month, but it wasnt accepted: http://www.winehq.org/pipermail/wine-patches/2011-July/105014.html -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=28097 --- Comment #2 from Vincent Povirk <madewokherd(a)gmail.com> 2011-08-16 14:32:31 CDT --- We should probably skip this test if "Times New Roman" isn't installed (which means testing whether "Times New Roman" is installed, not whether we got it from GdipGetGenericFontFamilySerif). -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=28097 --- Comment #3 from Vincent Povirk <madewokherd(a)gmail.com> 2011-10-10 10:37:12 CDT --- May be fixed by bcd793bf3b9c0f334b012b6afe8bef3160d64f2b. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=28097 Jerome Leclanche <adys.wh(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |adys.wh(a)gmail.com --- Comment #4 from Jerome Leclanche <adys.wh(a)gmail.com> 2011-10-27 06:59:57 CDT --- adys(a)azura ~/src/wine/build32/dlls/gdiplus/tests % make test -j1 ../../../../tools/runtest -q -P wine -M gdiplus.dll -T ../../.. -p gdiplus_test.exe.so ../../../../dlls/gdiplus/tests/font.c && touch font.ok font.c:56: Tests skipped: Arial not installed font.c:206: Tests skipped: Arial not installed font.c:242: Tests skipped: Arial not installed font.c:265: Tests skipped: Times New Roman not installed err:gdiplus:GdipGetGenericFontFamilySerif Missing 'Times New Roman' font font.c:324: Test failed: Expected 00000000, got 0000000e err:gdiplus:GdipGetGenericFontFamilyMonospace Missing 'Courier New' font font.c:331: Test failed: Expected 00000000, got 0000000e font.c:369: Tests skipped: Arial not installed make: *** [font.ok] Error 2 Not sure why it's failing. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=28097 --- Comment #5 from Austin English <austinenglish(a)gmail.com> --- Still in wine-1.7.15: font.c:784: Test failed: wrong face name Liberation Sans font.c:794: Test failed: wrong face name Liberation Sans sent a patch: https://source.winehq.org/patches/data/103395 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=28097 --- Comment #6 from Austin English <austinenglish(a)gmail.com> --- (In reply to Austin English from comment #5)
Still in wine-1.7.15: font.c:784: Test failed: wrong face name Liberation Sans font.c:794: Test failed: wrong face name Liberation Sans
sent a patch: https://source.winehq.org/patches/data/103395
https://www.winehq.org/pipermail/wine-devel/2014-March/103601.html still in wine-1.7.16-117-ge3383d1 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=28097 super_man(a)post.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |super_man(a)post.com --- Comment #7 from super_man(a)post.com --- Dmitry: "That's a gdi32 breakage (it shouldn't return "Liberation Sans"), nothing to do with gdiplus tests." If thats correct this title and patch are wrong. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=28097 Dmitry Timoshkov <dmitry(a)baikal.ru> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #8 from Dmitry Timoshkov <dmitry(a)baikal.ru> --- This bug is actually fixed, the tests no longer fail because if missing Times New Roman font. Not sure which sha1 is responsible for the fix. Now the tests fail because of missing Arial, and for that one I'd suggest to open new bug report. And a possible fix is to skip the test if Arial is not installed. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=28097 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #9 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 1.7.52. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=28097 --- Comment #10 from Austin English <austinenglish(a)gmail.com> --- (In reply to Dmitry Timoshkov from comment #8)
This bug is actually fixed, the tests no longer fail because if missing Times New Roman font. Not sure which sha1 is responsible for the fix.
Now the tests fail because of missing Arial, and for that one I'd suggest to open new bug report. And a possible fix is to skip the test if Arial is not installed.
I don't think anyone filed a bug for it, but Arial was fixed as well, by: https://source.winehq.org/git/wine.git/commitdiff/9fb9cb2bb7113f664a3db12472... -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs＠winehq.org