https://bugs.winehq.org/show_bug.cgi?id=57615
Bug ID: 57615 Summary: winetricks-test failing after recent cmd parsing changes Product: Wine Version: 9.12 Hardware: x86-64 OS: Linux Status: NEW Keywords: download, regression, source Severity: normal Priority: P2 Component: cmd Assignee: wine-bugs@winehq.org Reporter: austinenglish@gmail.com CC: eric.pouech@gmail.com Regression SHA1: bf9a465e9566578b40c08223436ce53f597bedd0 Distribution: Debian
Created attachment 77733 --> https://bugs.winehq.org/attachment.cgi?id=77733 stripped down winetricks-test showing the problem
https://github.com/Winetricks/winetricks/issues/2312
bf9a465e9566578b40c08223436ce53f597bedd0 is the first bad commit commit bf9a465e9566578b40c08223436ce53f597bedd0 (HEAD) Author: Eric Pouech epouech@codeweavers.com Date: Thu Jun 27 16:21:35 2024 +0200
cmd: Implement semantic for chaining in ||, | and && operators.
Signed-off-by: Eric Pouech epouech@codeweavers.com
programs/cmd/tests/test_builtins.cmd.exp | 106 +++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------------- programs/cmd/wcmd.h | 2 ++ programs/cmd/wcmdmain.c | 42 ++++++++++++++++++++++++++++++++++++++---- 3 files changed, 93 insertions(+), 57 deletions(-)
After this commit, winetricks-test fails (it's failing when trying to run): wine cmd.exe /c "chcp 65001 > nul && echo %AppData%"
Running that same command directly in winetricks (i.e., outside of winetricks-test works fine).
I narrowed down the winetricks-test script, and found that it's failing whenever stdin is redirected (winetricks-test gets the list of winetricks commands to run, filters out anything blacklisted, then writes that to a file that is then read in a loop to run the tests).
Removing `chcp 65001 > nul &&` also serves as a workaround (that was recently added to improve support for a unicode username, https://github.com/Winetricks/winetricks/pull/2278).
I'm not sure what the proper fix is here (or if it's a wine/winetricks problem).