On Tuesday 22 May 2007 22:06, Paul Vriens wrote:
Peter Oberndorfer wrote:
Some tests on http://test.winehq.org/data/ seem to crash silently.
Hi,
Remember that not all 'Crashes' on the mentioned site are real crashes. Most Vista tests I'm running have 'Crash' mentioned on those pages. The reason is that the generated results file is bigger than 1.5 MB and this means the file will only be sent partly.
If you're talking about the 'failed' ones with no good data (as it looks from you're patch), ignore the above.
Yes i'm talking about those.
I did some work on the parser (dissect Perl-script) a few months ago. You're output will not be handled properly (from a first glance). A normal test looks like:
a:b start <filename> <cvs-version> .. .. .. .. b: w tests executed (x marked as todo, y failure[s]), z skipped. a:b done (0)
As you're new output doesn't result in the 'done' line the parser will not correctly use that information. Maybe another line should be added at the end of you're output:
From what i have seen the lines with start <filename> <cvs-version> a:b done (0) are not output from the test itself, but wintetest which runs the tests. so they should still show up. But i havent verified this.
The only problem i see now is that we can't set the exit code and rune winedbg. So how could we inform winetest that the test crashed so it can set a proper done status, that can be examined by dissect. Or dissect finds crashed tests by a certain string that is only output on a crash. Maybe you have a better idea?
a:b done (some number not being 0 or 258)
The 258 is used for tests that timeout (0 being for tests that have run without crashes or timeouts). Obviously the parser (in this case it's actually the gather Perl-script) has to be changed to accommodate this new number.
Cheers,
Paul.
Greetings Peter