Re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-18 Thread Dan Kegel
Two more tips: 1) don't send HTML email to wine mailing lists 2) please be sure to run the conformance tests before submitting patches, and fix any failures Happily, your patch seems to cause a test to succeed: ... ../../../tools/runtest -q -P wine -M cmd.exe -T ../../.. -p cmd.exe_test.exe.so ba

Re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Nowres Rafid
On 17/08/2011 14:42, Dan Kegel wrote: Frédéric Delanoy wote: Nowres Rafid wrote: how can I generate the expected file (.exp)? It already exists in the source tree I think Nowres meant, how can he update the .exp file to match the expected output of his new test. The answer is: run the new te

re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Dan Kegel
Frédéric Delanoy wote: > Nowres Rafid wrote: >> how can I generate the expected file (.exp)? > It already exists in the source tree I think Nowres meant, how can he update the .exp file to match the expected output of his new test. The answer is: run the new test, save its output, and add it to t

Re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Frédéric Delanoy
2011/8/17 Nowres Rafid : > On 17/08/2011 14:02, Frédéric Delanoy wrote: >> >> On Fri, Aug 12, 2011 at 19:14, Nowres Rafid >>  wrote: >>> >>> fixing cd ..@space@ >>> >>> please ignore the previous patch. >> >> You didn't include any test in your patch. >> All patches including a fix must have tests

Re: cmd: fixing an issue with extra whitespaces after cd parameter (retry)

2011-08-17 Thread Frédéric Delanoy
On Fri, Aug 12, 2011 at 19:14, Nowres Rafid wrote: > fixing cd ..@space@ > > please ignore the previous patch. You didn't include any test in your patch. All patches including a fix must have tests confirming the bug is fixed