Re: [PATCH] testsuite: The expect framework might introduce CR in output

2025-01-11 Thread Torbjorn SVENSSON
On 2025-01-12 01:04, Jonathan Wakely wrote: On Sat, 11 Jan 2025, 19:14 Torbjorn SVENSSON, mailto:torbjorn.svens...@foss.st.com>> wrote: On 2025-01-11 20:05, Jonathan Wakely wrote: > > > On Sat, 11 Jan 2025, 18:31 Torbjörn SVENSSON, > mailto:torbjorn.svens...@fos

Re: [PATCH] testsuite: The expect framework might introduce CR in output

2025-01-11 Thread Jonathan Wakely
On Sat, 11 Jan 2025, 19:14 Torbjorn SVENSSON, wrote: > > > On 2025-01-11 20:05, Jonathan Wakely wrote: > > > > > > On Sat, 11 Jan 2025, 18:31 Torbjörn SVENSSON, > > mailto:torbjorn.svens...@foss.st.com>> > > wrote: > > > > Ok for trunk and releases/gcc-14? > > > > > > OK, thanks > > Oh, mid-a

Re: [PATCH] testsuite: The expect framework might introduce CR in output

2025-01-11 Thread Torbjorn SVENSSON
On 2025-01-11 20:05, Jonathan Wakely wrote: On Sat, 11 Jan 2025, 18:31 Torbjörn SVENSSON, mailto:torbjorn.svens...@foss.st.com>> wrote: Ok for trunk and releases/gcc-14? OK, thanks Oh, mid-air-collision. Thanks for the fast review Jonathan! I suppose my v2 should also be ok as it

[PATCH] testsuite: The expect framework might introduce CR in output

2025-01-11 Thread Torbjörn SVENSSON
Changes since v1: - Found out that 27_io/print/3.cc has the same kind of issue. Ok for trunk and releases/gcc-14? -- When running tests using the "sim" config, the command is launched in non-readonly mode and the text retrieved from the expect command will then replace all LF with CRLF. (The pr

Re: [PATCH] testsuite: The expect framework might introduce CR in output

2025-01-11 Thread Jonathan Wakely
On Sat, 11 Jan 2025, 18:31 Torbjörn SVENSSON, wrote: > Ok for trunk and releases/gcc-14? > OK, thanks > -- > > When running tests using the "sim" config, the command is launched in > non-readonly mode and the text retrieved from the expect command will > then replace all LF with CRLF. (The pr

[PATCH] testsuite: The expect framework might introduce CR in output

2025-01-11 Thread Torbjörn SVENSSON
Ok for trunk and releases/gcc-14? -- When running tests using the "sim" config, the command is launched in non-readonly mode and the text retrieved from the expect command will then replace all LF with CRLF. (The problem can be found in sim_load where it calls remote_spawn without an input file).