[PATCH v4 0/1] MR7252: cmd: add support for right-to-left handle redirection (2<&1)

[Damjan Jovanovic (＠dj)]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57795

Closes #57795

--
  v4: cmd: add support for right-to-left handle redirection (2<&1)
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7252

[PATCH v4 1/1] cmd: add support for right-to-left handle redirection (2<&1)

[Damjan Jovanovic]

From: Damjan Jovanovic <damjan.jov(a)gmail.com>

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57795
---
 programs/cmd/tests/test_builtins.cmd     |  5 +++++
 programs/cmd/tests/test_builtins.cmd.exp |  3 +++
 programs/cmd/wcmdmain.c                  | 18 +++++++++++++++---
 3 files changed, 23 insertions(+), 3 deletions(-)

diff --git a/programs/cmd/tests/test_builtins.cmd b/programs/cmd/tests/test_builtins.cmd
index 8552d51930a..9e690aeaa49 100644
--- a/programs/cmd/tests/test_builtins.cmd
+++ b/programs/cmd/tests/test_builtins.cmd
@@ -176,6 +176,11 @@ del foo
 echo foo> foo
 echo foo7 7>> foo || (echo not supported & del foo)
 if exist foo (type foo) else echo not supported
+echo --- right-to-left redirection
+1>foo-out 2>foo-err 1<&2 echo foo
+type foo-out 2>NUL || echo good
+type foo-err 2>NUL || echo bad
+erase /q foo-out foo-err
 echo --- redirect at beginning of line
> foo (echo foo)
>  type foo
> diff --git a/programs/cmd/tests/test_builtins.cmd.exp b/programs/cmd/tests/test_builtins.cmd.exp
> index 292c24985e8..5764df2c466 100644
> --- a/programs/cmd/tests/test_builtins.cmd.exp
> +++ b/programs/cmd/tests/test_builtins.cmd.exp
> @@ -204,6 +204,9 @@ food2
>  food21
>  foo7(a)space@@space@@or_broken(a)not supported(a)space@
>  foo(a)or_broken@not supported
> +--- right-to-left redirection
> +good
> +foo
>  --- redirect at beginning of line
>  foo
>  foo1
> diff --git a/programs/cmd/wcmdmain.c b/programs/cmd/wcmdmain.c
> index b19f8453438..3e248a6d36a 100644
> --- a/programs/cmd/wcmdmain.c
> +++ b/programs/cmd/wcmdmain.c
> @@ -2627,8 +2627,20 @@ static void lexer_push_command(struct node_builder *builder,

if (*p == L'<')
             {
-                filename = WCMD_parameter(p + 1, 0, NULL, FALSE, FALSE);
-                tkn_pmt.redirection = redirection_create_file(REDIR_READ_FROM, 0, filename);
+                unsigned fd = 0;
+
+                if (p > redirs && p[-1] >= L'0' && p[-1] <= L'9') fd = p[-1] - L'0';
+                p++;
+                if (*p == L'&' && (p[1] >= L'0' && p[1] <= L'9'))
+                {
+                    tkn_pmt.redirection = redirection_create_clone(fd, p[1] - L'0');
+                    p++;
+                }
+                else
+                {
+                    filename = WCMD_parameter(p + 1, 0, NULL, FALSE, FALSE);
+                    tkn_pmt.redirection = redirection_create_file(REDIR_READ_FROM, 0, filename);
+                }
             }
             else
             {
@@ -2984,7 +2996,7 @@ enum read_parse_line WCMD_ReadAndParseLine(const WCHAR *optionalcmd, CMD_NODE **
 
                 /* If a redirect is immediately followed by '&' (ie. 2>&1) then
                     do not process that ampersand as an AND operator */
-                if (thisChar == '>' && *(curPos+1) == '&') {
+                if ((thisChar == '>' || thisChar == '<') && *(curPos+1) == '&') {
                     curCopyTo[(*curLen)++] = *(curPos+1);
                     curPos++;
                 }
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/7252

[PATCH v4 1/1] cmd: add support for right-to-left handle redirection (2<&1)

[Marvin]

Hi,

It looks like your patch introduced the new failures shown below.
Please investigate and fix them before resubmitting your patch.
If they are not new, fixing them anyway would help a lot. Otherwise
please ask for the known failures list to be updated.

The tests also ran into some preexisting test failures. If you know how
to fix them that would be helpful. See the TestBot job for the details:

The full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=151254

Your paranoid android.


=== debian11b (64 bit WoW report) ===

user32:
msg.c:7004: Test failed: SetFocus(hwnd) on a button: 3: the winevent_hook 0x8005 was expected, but got hook 0x0005 instead
msg.c:7004: Test failed: SetFocus(hwnd) on a button: 3: the winevent_hook 0x8005 was expected, but got winevent_hook 0x0003 instead
msg.c:7004: Test failed: SetFocus(hwnd) on a button: 3: the winevent_hook 0x8005 was expected, but got msg 0x030f instead
msg.c:7004: Test failed: SetFocus(hwnd) on a button: 3: the winevent_hook 0x8005 was expected, but got msg 0x001c instead
msg.c:7004: Test failed: SetFocus(hwnd) on a button: 3: the winevent_hook 0x8005 was expected, but got msg 0x0086 instead
msg.c:7004: Test failed: SetFocus(hwnd) on a button: 3: the winevent_hook 0x8005 was expected, but got msg 0x0006 instead
msg.c:7004: Test failed: SetFocus(hwnd) on a button: 3: the winevent_hook 0x8005 was expected, but got hook 0x0009 instead

[Damjan Jovanovic (＠dj)]

On Wed Feb  5 01:09:09 2025 +0000, eric pouech wrote:
> thanks for the update, your new first test is way cleaner...
> (just trying to simplify at most the tests and avoid calling external
> findstr, you could do
> `type foo-out 2> NUL || echo good`
> `type foo-err 2> NUL || echo bad`
> should output
> `good`
> `foo`
> (does work on native, should work on wine but didn't test it though)
> Good idea, that's what I've done now. Thank you.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7252#note_93669

[PATCH v4 0/1] MR7252: cmd: add support for right-to-left handle redirection (2<&1) - approved

[eric pouech (＠epo)]

This merge request was approved by eric pouech.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7252