Re: [PATCH v7 0/2] MR8200: cmd: COPY should output file names as they are copied.
please amend the tests in the `---------- Testing copy part` (after line 3387 in test_builtins.cmd), the tests you've amended are meant to test the return codes, not really the final behavior) and likely all these tests up to the ascii/binary part should be amended to check whether the names are printed or not if needed, you can work around builtin find.exe missing the /c switch by using ``` :count_lines @set "lines=0" @for /F %%a in (%1) do set /A lines+=1 @exit /b %lines% ``` using `call :count_lines <file>` and you'll get the number of lines in %errorlevel% variable (but IMO in most of the cases testing for a file name printed should serve the same purpose) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8200#note_105647
participants (1)
-
eric pouech (@epo)