January 2011 - Wine-devel - list.winehq.org

new winetricks 20110123: work around bug in wine-1.3.12. new winetricks-alpha 20110123: ditto, plus new verbs cnc3_demo, guildwars, hphbp_demo, nfsshift_demo, sims3, spotify, vc2005trial
by Dan Kegel 24 Jan '11

24 Jan '11

Another week, another winetricks. This release is mainly to add a workaround for http://bugs.winehq.org/show_bug.cgi?id=25838 in wine-1.3.12, which broke winetricks. winetricks-alpha version 20110123 adds the following new verbs: cnc3_demo: Command and Conquer 3 demo guildwars: Guild Wars hphbp_demo: Harry Potter Half Blood Prince demo nfsshift_demo: Need For Speed: Shift demo sims3: The Sims 3 spotify: the popular music service vc2005trial: Visual C++ 2005 Trial winetricks version 20110123 adds no new verbs. Both scripts can be downloaded from winetricks.org at the obvious urls: http://winetricks.org/winetricks http://winetricks.org/winetricks-alpha (The old download url at kegel.com/wine/winetricks still works, too.) Bug tracker and svn repository are at http://winetricks.googlecode.com ----------------------------------- Changelog for winetricks-alpha: Dan Kegel: Work around wine bug 25838 :-( Ported verbs hphbp_demo, sims3 from wisotool Fix mingw PATH (again) Add example to help for 'list' command w_download{,_manual}: get rid of need to specify package name. w_download_manual: open destination folder in desktop for user python26_comtypes: metadata should be "year", not "date". Shortened title for psdkwin7 to fit window. Shannon Vanwagner: New verb spotify Phil Blankenship: Ported verbs cnc3_demo, guildwars, nfsshift_demo, vc2005trial from wisotool. ----------------------------------- Changelog for winetricks: Dan Kegel: Work around wine bug 25838 :-( Austin English: update opensymbol url/sha1sum ----------------------------------- Enjoy!

1 0

wine-1.3.12 broke winetricks :-(
by Dan Kegel 24 Jan '11

24 Jan '11

Looks like wine-1.3.12 went out without the fix for http://bugs.winehq.org/show_bug.cgi?id=25838 I've checked a workaround into winetricks, and will do a release, but I bet a few users will be confused...

1 0

3Dmark2001se & heap corruption
by wylda＠volny.cz 23 Jan '11

23 Jan '11

Hi, as there is a strong Valgrind wave these days ;) and i failed to find the problem myself here is a little exercise for anyone interested: 1. WINEDEBUG=warn+heap wine 3DMark2001SE.exe 2. Under selected test -> (click) Clear -> (tab) Custom -> (check) "Game2 Dragothic Low Detail" -> OK 3. (click) Benchmark At the end, console is flooded by many: Heap 0x110000: block 0x2c41070 tail overwritten at 0x2c410a1 (byte 0/15 == 0x00) Heap 0x110000: block 0x1c46b08 tail overwritten at 0x1c46b39 (byte 0/39 == 0x00) W.

4 6

re: Correction to crash inside RtlCaptureStackBackTrace() + test case
by Dan Kegel 23 Jan '11

23 Jan '11

Hi Janne, I haven't reviewed your code, but I can suggest sending each patch as a separate message to wine-patches, and using titles more like ntdll: test corrupt stack frame in RtlCaptureStackBackTrace() ntdll: handle corrupt stack frame in RtlCaptureStackBackTrace() and mention clearly that this fixes the online store in Anarchy Online (if it does). Also, please send the test patch by itself through winebot before posting and verify that it passes on all flavors of windows. See http://wiki.winehq.org/SubmittingPatches Thanks!

2 1

Re: dinput: Combine ASCII and Unicode device create callbacks. Add tests. (try 2)
by Marvin 23 Jan '11

23 Jan '11

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=8488 Your paranoid android. === WNT4WSSP6 (32 bit device) === device.c:168: Test failed: DirectInputCreate() failed: 80040154

3 5

Winetricks game verbs - good or bad? (was: Re: New winetricks 20110117-alpha: new verbs dxdiag, firefox4, ut3, hegemonygold_demo, ...)
by Dan Kegel 23 Jan '11

23 Jan '11

On Mon, Jan 17, 2011 at 7:33 PM, Vitaliy Margolen <wine-devel(a)kievinfo.com> wrote: > On 01/17/2011 11:30 AM, Dan Kegel wrote: >> New version 20110117-alpha adds the following new verbs: >> >> firefox4 - beta9 of Firefox's new web browser >> ut3 - the first-person shooter Unreal Tournament 3 (requires dvd) > > Isn't that exactly why we marked all other scripts like this a "third party > unsupported tools"? I suspect PlayOnLinux got the cold shoulder because they use old versions of wine, they use their own patches, they don't have a 'real' version control system for game scripts, and they didn't communicate with the wine community. So problem reports from PlayOnLinux users weren't very helpful for Wine developers. Winetricks doesn't encourage users to run old or patched wine, and I'm not a stranger on wine-devel. In fact, if we play our cards right, winetricks might actually reduce the number of clueless support requests. People have trouble following the HOWTOs in the appdb. A winetricks verb for a game should be like a perfectly automated little implementation of the HOWTO, with bug workarounds clearly marked, and skipped automatically when using new enough wine. Better still, since it's automated, we can run it nightly to make sure it doesn't break. What concrete problems do you see with winetricks having verbs for popular games? - Dan

2 3

FYI: GCC-4.6 rebuild tests
by Susan Cragin 23 Jan '11

23 Jan '11

GCC-4.6 has more build failures than mine. Here's the info. They don't test wine, apparently. It's not on the list. Maybe I can add it, or get it added. -----Forwarded Message----- >From: Matthias Klose <doko(a)ubuntu.com> >Sent: Jan 19, 2011 10:28 AM >To: ubuntu-devel <ubuntu-devel-announce(a)lists.ubuntu.com>, ubuntu-devel <ubuntu-devel(a)lists.ubuntu.com> >Subject: GCC-4.6 rebuild tests > >[ Note that GCC-4.6 will not be used for natty / 11.04, but > we consider using this version for the O-series / 11.10 ] > >A test rebuild of natty/main using the current GCC version from the trunk was >finished last week. While only one internal compiler error was found, some more >build failures were seen due to new warnings and more strict frontends. A list >of build failure can be seen at > >http://people.ubuntuwire.org/~wgrant/rebuild-ftbfs-test/test-rebuild-201101… > >If a package appears in the `superseded' section, it doesn't mean that the build >failure with GCC-4.6 is fixed. > >Some comments and hints for fixes can be found at > > https://wiki.ubuntu.com/GCC4.6 > >Please extend the wiki page if appropriate. > >To work on the build failures, please use either the gcc-snapshot package from >natty or the gcc-4.6 package from the ubuntu-toolchain-r/test PPA. > >A large number of build failures is seen for packages building with -Werror. The >compiler packages above were changed to add a -Wno-error=unused-but-set-variable >when -Werror is found. Despite this local change the these packages need to >build without this local change. > > Matthias > >PS: Fixing build failures with GCC-4.5 in natty should still have priority ;) > >-- >ubuntu-devel-announce mailing list >ubuntu-devel-announce(a)lists.ubuntu.com >https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel-announce

2 1

Re: Correction to crash inside RtlCaptureStackBackTrace() + test case
by André Hentschel 23 Jan '11

23 Jan '11

Am 23.01.2011 18:20, schrieb Janne Hakonen: > Hi, > > This is my first attempt to post patches to Wine and it would be great if you could review these. > > First patch (0001-2-2-ntdll-Added-try-except-blocks-to-RtlCaptureStackBa.txt) is a correction to a crash situation where application crashes if RtlCaptureStackBackTrace() is called and one of the frame pointers of call stack is invalid, pointing usually to 0xffffffff, sometimes to other invalid locations. > The correction wraps the offending code inside __try â�� __except blocks so that a resulting access violation is caught before it can crash the calling application. > > Second patch (0002-1-2-ntdll-tests-Added-a-test-for-RtlCaptureStackBackTr.txt) is a test case for the correction. > It changes one of the frame pointers to invalid value and then calls RtlCaptureStackBackTrace(). The test fails on Wineâ��s HEAD (without the first patch), but passes successfully on Windows 7. Havenâ��t tested this on other OSes. I created a new signal.c file for the test case under ntdll/tests since I couldnâ��t find more fitting place. > > This crash can be reproduced with Anarchy Online game when trying to open an in-game browser. Also reproducible with AwesomiumGL sample application from Awesomiumâ��s SDK. The crash (among other problems) is reported to bugzilla at: http://bugs.winehq.org/show_bug.cgi?id=25206. > > Best Regards, > Janne Hakonen Hi and welcome to wine development! First i see is that your mail application wrapped the patch ( that is when long likes brake down into the next line ). Then you have a whitespace issue in the makefile i think... further you should consider using IsBadReadPtr instead of try...except, but i'm not sure if that would work. If you solved the issues, you can send the patch to wine-patches(a)winehq.org and can track the status at source.winehq.org/patches -- Best Regards, André Hentschel

2 1

Correction to crash inside RtlCaptureStackBackTrace() + test case
by Janne Hakonen 23 Jan '11

23 Jan '11

Hi, This is my first attempt to post patches to Wine and it would be great if you could review these. First patch (0001-2-2-ntdll-Added-try-except-blocks-to-RtlCaptureStackBa.txt) is a correction to a crash situation where application crashes if RtlCaptureStackBackTrace() is called and one of the frame pointers of call stack is invalid, pointing usually to 0xffffffff, sometimes to other invalid locations. The correction wraps the offending code inside __try – __except blocks so that a resulting access violation is caught before it can crash the calling application. Second patch (0002-1-2-ntdll-tests-Added-a-test-for-RtlCaptureStackBackTr.txt) is a test case for the correction. It changes one of the frame pointers to invalid value and then calls RtlCaptureStackBackTrace(). The test fails on Wine’s HEAD (without the first patch), but passes successfully on Windows 7. Haven’t tested this on other OSes. I created a new signal.c file for the test case under ntdll/tests since I couldn’t find more fitting place. This crash can be reproduced with Anarchy Online game when trying to open an in-game browser. Also reproducible with AwesomiumGL sample application from Awesomium’s SDK. The crash (among other problems) is reported to bugzilla at: http://bugs.winehq.org/show_bug.cgi?id=25206. Best Regards, Janne Hakonen

1 0

Re: gdi32/tests: Skip font metrics test on non-English systems
by Dmitry Timoshkov 23 Jan '11

23 Jan '11

"Greg Geldorp" <ggeldorp(a)vmware.com> wrote: > --- > dlls/gdi32/tests/font.c | 6 ++++++ > 1 files changed, 6 insertions(+), 0 deletions(-) That test is supposed to pass at least on russian locale. On which locale does the test fail for you? -- Dmitry.

2 4