On Wed, 31 Jul 2019, Jacek Caban wrote: [...]
debugger.c:1143: Test failed: Got 10958580 failures from child process. debugger.c:1091: Test failed: unexpected event pid
This is not related to the patch. It looks like we need to increase the timeout in existing tests.
Do you mean the 2 minutes test timeout? We really don't want the tests to take that long.
On slightly related note, I feel like Vista is overrepresented on TestBot. It's the least interesting Windows version. While XP is still interesting sometimes, I don't see any value in testing on Vista. Debugger tests are an example of a case where Vista seems to behave worse than any other Windows version (XP including) and it just adds extra useless work of working around its weirdness. I'd suggest to plan to get rid of Vista from TestBot. Currently, due to multiple locale VMs, it has the highest number of VMs.
I don't think that matters. For tests I would say the only results that matter are 0 test failures, and 1+ test failures. Running a test more times on a given platform would not change that, except for random failures which are probably random on any platform anyway.
So if test failures on Vista are not important the only option is to not run the tests on that platform at all.
If there's a general agreement that's what we want I have no objection.
I think it would make more sense to have win10 in multiple locales instead.
I thought that required a special Windows version but apparently not. So sure, I can add locales there.