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@winehq.org ReportedBy: dank@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...
http://bugs.winehq.org/show_bug.cgi?id=28097
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |nerv@dawncrow.de
--- Comment #1 from André H. nerv@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
http://bugs.winehq.org/show_bug.cgi?id=28097
--- Comment #2 from Vincent Povirk madewokherd@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).
http://bugs.winehq.org/show_bug.cgi?id=28097
--- Comment #3 from Vincent Povirk madewokherd@gmail.com 2011-10-10 10:37:12 CDT --- May be fixed by bcd793bf3b9c0f334b012b6afe8bef3160d64f2b.
http://bugs.winehq.org/show_bug.cgi?id=28097
Jerome Leclanche adys.wh@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |adys.wh@gmail.com
--- Comment #4 from Jerome Leclanche adys.wh@gmail.com 2011-10-27 06:59:57 CDT --- adys@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.
https://bugs.winehq.org/show_bug.cgi?id=28097
--- Comment #5 from Austin English austinenglish@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
https://bugs.winehq.org/show_bug.cgi?id=28097
--- Comment #6 from Austin English austinenglish@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
https://bugs.winehq.org/show_bug.cgi?id=28097
super_man@post.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |super_man@post.com
--- Comment #7 from super_man@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.
https://bugs.winehq.org/show_bug.cgi?id=28097
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED
--- Comment #8 from Dmitry Timoshkov dmitry@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.
https://bugs.winehq.org/show_bug.cgi?id=28097
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #9 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 1.7.52.
https://bugs.winehq.org/show_bug.cgi?id=28097
--- Comment #10 from Austin English austinenglish@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...
-
wine-bugs＠winehq.org