2009/2/23 Paul Vriens <paul.vriens.w...@gmail.com>: > Nicolas Le Cam wrote: >> 2009/2/23 Francois Gouget <fgou...@free.fr>: >>> On Sun, 22 Feb 2009, Nicolas Le Cam wrote: >>> >>>> This avoid a messagebox in regression tests on systems that don't have >>>> GdiPlus. >>> >>> This seems wrong because winetest.exe is already supposed to check for >>> missing dlls. If I remember correctly this was done explicitely so that >>> individual tests would not have to dynamically link everywhere. >>> >>> Also I tested on a platform that does not have gdiplus, and I did not >>> get a message box. In fact, on that platform, winetest did not extract >>> gdiplus_test.exe, which also seems very wrong to me. >>> >>> So I think that if there's any issue, it's on the winetest.exe side. >>> >>> -- >>> Francois Gouget <fgou...@free.fr> http://fgouget.free.fr/ >>> "Lotto: A tax on people who are bad at math." -- unknown >>> "Windows: Microsoft's tax on computer illiterates." -- WE7U >>> >>> >>> >> >> Thanks, >> >> On my w2k pro when I launch the winetest obtained from >> test.winehq.org, I get a MessageBox saying gdiplus is missing, and >> there's already some tests that load dlls dynamically (advpack, >> cryptui, fusion) so thought it was the right way. >> >> I will have a look on the winetest side. >> >> Nicolas Le Cam >> >> >> > The reason you see this is because next to a normal LoadLibrary we also use > the > .NET variant. On some boxes gdiplus.dll cannot be found through LoadLibrary > but > will trough LoadLibraryShim. As winetest now thinks the library is there it > will > try to launch "gdiplus_test.exe --list" and that of course fails. > > Not sure how to approach this. > > -- > Cheers, > > Paul. > > >
This shouldn't be the case on my box as gdiplus is not present by default on W2K, and should be manually installed (not available through windows update), or installed by a third party app. My test box is a clean install without any third apps, except PDFCreator to simulate a printer to test this part of the suite. I will check if it has gdiplus bundled in. Nicolas Le Cam