https://testbot.winehq.org/JobDetails.pl?Key=11509 shows failures and crashes, but the nearly identical https://testbot.winehq.org/JobDetails.pl?Key=11510 does not. (The latter one just adds a call to GetProcessMemoryInfo() in the xyz() logging function.)
And the failure shown by the first patch has a last error on creating the batch file of 8, ERROR_NOT_ENOUGH_MEMORY, which is suspicious.
Not quite sure what's going on there.
On Mon, Jun 6, 2011 at 03:17, Dan Kegel dank@kegel.com wrote:
https://testbot.winehq.org/JobDetails.pl?Key=11509 shows failures and crashes, but the nearly identical https://testbot.winehq.org/JobDetails.pl?Key=11510 does not. (The latter one just adds a call to GetProcessMemoryInfo() in the xyz() logging function.)
And the failure shown by the first patch has a last error on creating the batch file of 8, ERROR_NOT_ENOUGH_MEMORY, which is suspicious.
Not quite sure what's going on there.
I made some tests to get a minimal patch where this occurs, and it's rather weird: problem occurs when the "rem" line precedes the "call :setError 0" call
WORKS https://testbot.winehq.org/JobDetails.pl?Key=11540 +echo -----------Testing mkdir ----------- +call :setError 0 +
https://testbot.winehq.org/JobDetails.pl?Key=11539 +echo -----------Testing mkdir ----------- +rem Creating an already existing directory must fail +
DOES NOT WORK https://testbot.winehq.org/JobDetails.pl?Key=11538 +echo -----------Testing mkdir ----------- +rem Creating an already existing directory must fail +call :setError 0 +
On Mon, Jun 6, 2011 at 03:17, Dan Kegel dank@kegel.com wrote:
https://testbot.winehq.org/JobDetails.pl?Key=11509 shows failures and crashes, but the nearly identical https://testbot.winehq.org/JobDetails.pl?Key=11510 does not. (The latter one just adds a call to GetProcessMemoryInfo() in the xyz() logging function.)
And the failure shown by the first patch has a last error on creating the batch file of 8, ERROR_NOT_ENOUGH_MEMORY, which is suspicious.
Not quite sure what's going on there.
Actually it seems using "md" instead of "mkdir" gives those problems, although they should be perfect synonymous Compare https://testbot.winehq.org/JobDetails.pl?Key=11608 (md version; some failures) and https://testbot.winehq.org/JobDetails.pl?Key=11611 (mkdir; no failure)
-
Dan Kegel -
Frédéric Delanoy