[Bug 28103] New: gdi32 font.ok fails if sun-java6-fonts installed, does nothing otherwise?

http://bugs.winehq.org/show_bug.cgi?id=28103

Summary: gdi32 font.ok fails if sun-java6-fonts installed, does
                    nothing otherwise?
           Product: Wine
           Version: 1.3.26
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: gdi32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

I'm starting to think that 
http://source.winehq.org/git/wine.git/?a=commit;h=86495cea9486aa5df4ebdf0261...
was wrong.

All the results I've seen for it so far on test.winehq.org show
it saying
font.c:3748: Tests skipped: Lucida Sans is not installed

On my machine, it does that unless sun-java6-fonts is installed,
in which case it fails:

sudo apt-get install sun-java6-fonts
cd dlls/gdi32/tests
make font.ok
../../../tools/runtest -q -P wine -M gdi32.dll -T ../../.. -p gdi32_test.exe.so
font.c && touch font.ok
font.c:3774: Test failed: font full names don't match: Lucida Sans Demibold
Roman != Arial
font.c:3774: Test failed: font full names don't match: Lucida Sans Italic !=
Arial

Looking at test.winehq.org, it seems to

[Bug 28103] gdi32 font.ok fails if sun-java6-fonts installed, does nothing otherwise?

http://bugs.winehq.org/show_bug.cgi?id=28103

Grazvydas Ignotas  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from Grazvydas Ignotas  2011-08-17 06:58:26 CDT ---
You're right, the test assumes if "Lucida Sans" font family is available,
"Lucida Sans Demibold Roman" and "Lucida Sans Italic" fonts are available,
which is mostly true under Windows as those fonts come by default.

However under Linux this will only work if you copy those 2 fonts from some
Windows installation. In the case of that java package, it provides "Lucida
Sans Regular" and "Lucida Sans Demibold", which in turn provide "Lucida Sans"
family but not exact fonts the tests expects.

Let me cook some patches to correct this.

[Bug 28103] gdi32 font.ok fails if sun-java6-fonts installed, does nothing otherwise?

http://bugs.winehq.org/show_bug.cgi?id=28103

--- Comment #2 from Grazvydas Ignotas  2011-08-17 08:38:47 CDT ---
Created an attachment (id=36010)
 --> (http://bugs.winehq.org/attachment.cgi?id=36010)
fullname check

Please try this patch. It should skip the right fonts.

[Bug 28103] gdi32 font.ok fails if sun-java6-fonts installed, does nothing otherwise?

http://bugs.winehq.org/show_bug.cgi?id=28103

--- Comment #3 from Dan Kegel  2011-08-17 08:50:46 CDT ---
yes, that patch makes tests succeed even when sun-java6-fonts is installed.

[Bug 28103] gdi32 font.ok fails if sun-java6-fonts installed, does nothing otherwise?

http://bugs.winehq.org/show_bug.cgi?id=28103

--- Comment #4 from Grazvydas Ignotas  2011-09-06 05:35:11 CDT ---
I think this one is ok now, close?

[Bug 28103] gdi32 font.ok fails if sun-java6-fonts installed, does nothing otherwise?

http://bugs.winehq.org/show_bug.cgi?id=28103

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Dan Kegel  2011-09-06 09:56:36 CDT ---
I guess it should have been fixed by 4a463e06505af991f23b9c2ce47efedaf9738928
I'll reopen if I run into it again.
Thanks!

[Bug 28103] gdi32 font.ok fails if sun-java6-fonts installed, does nothing otherwise?

http://bugs.winehq.org/show_bug.cgi?id=28103

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #6 from Alexandre Julliard  2011-09-09 12:54:46 CDT ---
Closing bugs fixed in 1.3.28.