Re: [PATCH 0/4] Match \r\r\n in testsuite

Joel Brobecker
2013-09-17T13:35:33Z

> Then, I restarted from the beginning again, and see if we can fix the
> problem by modifying testsuite. The requirement expressed in the
> previous patch review is that we should localize the changes to
> testsuite to match \r\r\n.
>
> This patch series is to change testsuite to match both \r\n and
> \r\r\n by modifying pattern to \r+\n in a localized manner. Patch 1/4
> is to replace gdb_test with gdb_test_sequence to match the output of
> "info tracepoints", so that the assumption of last character of each
> line is removed, so these patterns can be matched on all targets.
> Patch 2/4 is to transform \r\n in patterns to \r+\n, but \r\n in
> variables can't be transformed. Patch 3/4 transforms \r\n in variables.

How about we introduce a new official variable, similar to $hex,
$decimal, etc, which would correctly match new-line output based
on the host? I propose $eol, which is already used, but only in
2 testcases, so easy to adjust (remove "set eol ..." in those two
testcases).

--
Joel