Re: [PATCH 5/5] cmd: rename reference file from .out to .exp to avoid clash with gnu make builtin rule
On 02/16/2010 10:08 PM, Dan Kegel wrote: Without this, editing the .cmd file and then running make would cause make to overwrite the .out file with a copy of the .cmd file! Evidently '.out' is not appropriate as a suffix for a nongenerated source file. The arbitrarily-chosen .exp suffix works better (if anyone asks, it stands for 'expected output'). --- programs/cmd/tests/batch.c |4 ++-- programs/cmd/tests/rsrc.rc |4 ++-- programs/cmd/tests/test_builtins.cmd.exp | 18 ++ programs/cmd/tests/test_builtins.cmd.out | 18 -- 4 files changed, 22 insertions(+), 22 deletions(-) create mode 100644 programs/cmd/tests/test_builtins.cmd.exp delete mode 100644 programs/cmd/tests/test_builtins.cmd.out Hi Dan, These last changes introduce several test failures on some W2K, XP and W2K3 boxes but on all Wine ones: http://test.winehq.org/data/tests/cmd.exe:batch.html One thing I've noticed is that the succeeding ones run 12 tests whereas the failing ones run 14 tests. Could you have a look? -- Cheers, Paul.
Re: [PATCH 5/5] cmd: rename reference file from .out to .exp to avoid clash with gnu make builtin rule
On 02/18/2010 09:50 AM, Paul Vriens wrote: On 02/16/2010 10:08 PM, Dan Kegel wrote: Without this, editing the .cmd file and then running make would cause make to overwrite the .out file with a copy of the .cmd file! Evidently '.out' is not appropriate as a suffix for a nongenerated source file. The arbitrarily-chosen .exp suffix works better (if anyone asks, it stands for 'expected output'). --- programs/cmd/tests/batch.c | 4 ++-- programs/cmd/tests/rsrc.rc | 4 ++-- programs/cmd/tests/test_builtins.cmd.exp | 18 ++ programs/cmd/tests/test_builtins.cmd.out | 18 -- 4 files changed, 22 insertions(+), 22 deletions(-) create mode 100644 programs/cmd/tests/test_builtins.cmd.exp delete mode 100644 programs/cmd/tests/test_builtins.cmd.out Hi Dan, These last changes introduce several test failures on some W2K, XP and W2K3 boxes but on all Wine ones: http://test.winehq.org/data/tests/cmd.exe:batch.html One thing I've noticed is that the succeeding ones run 12 tests whereas the failing ones run 14 tests. Could you have a look? Nice, I just ran the latest tests again on winetestbot (as you also did before submitting) and they don't fail. So the big issue will be to reproduce these failures. I wasn't able on Windows, but I am on Wine when doing something like: wine winetest-latest.exe cmd.exe:batch Note that make batch.ok works just fine. I've build my own winetest and there I can trigger the failures on Wine as well. On to some debugging exercise. -- Cheers, Paul.
Re: [PATCH 5/5] cmd: rename reference file from .out to .exp to avoid clash with gnu make builtin rule
On 02/18/2010 11:36 AM, Paul Vriens wrote: On 02/18/2010 09:50 AM, Paul Vriens wrote: On 02/16/2010 10:08 PM, Dan Kegel wrote: Without this, editing the .cmd file and then running make would cause make to overwrite the .out file with a copy of the .cmd file! Evidently '.out' is not appropriate as a suffix for a nongenerated source file. The arbitrarily-chosen .exp suffix works better (if anyone asks, it stands for 'expected output'). --- programs/cmd/tests/batch.c | 4 ++-- programs/cmd/tests/rsrc.rc | 4 ++-- programs/cmd/tests/test_builtins.cmd.exp | 18 ++ programs/cmd/tests/test_builtins.cmd.out | 18 -- 4 files changed, 22 insertions(+), 22 deletions(-) create mode 100644 programs/cmd/tests/test_builtins.cmd.exp delete mode 100644 programs/cmd/tests/test_builtins.cmd.out Hi Dan, These last changes introduce several test failures on some W2K, XP and W2K3 boxes but on all Wine ones: http://test.winehq.org/data/tests/cmd.exe:batch.html One thing I've noticed is that the succeeding ones run 12 tests whereas the failing ones run 14 tests. Could you have a look? Nice, I just ran the latest tests again on winetestbot (as you also did before submitting) and they don't fail. So the big issue will be to reproduce these failures. I wasn't able on Windows, but I am on Wine when doing something like: wine winetest-latest.exe cmd.exe:batch Note that make batch.ok works just fine. I've build my own winetest and there I can trigger the failures on Wine as well. On to some debugging exercise. I think, I got it. It's a case-sensitivity issue. When I run the crosscompiled winetest on my Wine box I get for 'workdir': C:\users\paul\Temp\wct The test.out file however contains: C:\users\paul\temp\wct\ (notice the lower case 't' for temp). The same is probably true for several Windows boxes. On the winetestbot tests are run from the C:\winetest directory. Winetest.exe however uses the %TEMP%\wct directory which contains (XP/W2K3) something like 'Document and Settings'. -- Cheers, Paul.
Re: [PATCH 5/5] cmd: rename reference file from .out to .exp to avoid clash with gnu make builtin rule
On 02/18/2010 01:32 PM, Paul Vriens wrote: On 02/18/2010 11:36 AM, Paul Vriens wrote: On 02/18/2010 09:50 AM, Paul Vriens wrote: On 02/16/2010 10:08 PM, Dan Kegel wrote: Without this, editing the .cmd file and then running make would cause make to overwrite the .out file with a copy of the .cmd file! Evidently '.out' is not appropriate as a suffix for a nongenerated source file. The arbitrarily-chosen .exp suffix works better (if anyone asks, it stands for 'expected output'). --- programs/cmd/tests/batch.c | 4 ++-- programs/cmd/tests/rsrc.rc | 4 ++-- programs/cmd/tests/test_builtins.cmd.exp | 18 ++ programs/cmd/tests/test_builtins.cmd.out | 18 -- 4 files changed, 22 insertions(+), 22 deletions(-) create mode 100644 programs/cmd/tests/test_builtins.cmd.exp delete mode 100644 programs/cmd/tests/test_builtins.cmd.out Hi Dan, These last changes introduce several test failures on some W2K, XP and W2K3 boxes but on all Wine ones: http://test.winehq.org/data/tests/cmd.exe:batch.html One thing I've noticed is that the succeeding ones run 12 tests whereas the failing ones run 14 tests. Could you have a look? Nice, I just ran the latest tests again on winetestbot (as you also did before submitting) and they don't fail. So the big issue will be to reproduce these failures. I wasn't able on Windows, but I am on Wine when doing something like: wine winetest-latest.exe cmd.exe:batch Note that make batch.ok works just fine. I've build my own winetest and there I can trigger the failures on Wine as well. On to some debugging exercise. I think, I got it. It's a case-sensitivity issue. When I run the crosscompiled winetest on my Wine box I get for 'workdir': C:\users\paul\Temp\wct The test.out file however contains: C:\users\paul\temp\wct\ (notice the lower case 't' for temp). The same is probably true for several Windows boxes. On the winetestbot tests are run from the C:\winetest directory. Winetest.exe however uses the %TEMP%\wct directory which contains (XP/W2K3) something like 'Document and Settings'. Just sent a patch to fix the Wine failures. Forget the remarks about Windows and case-sensitivity, that needs more investigation. -- Cheers, Paul.
