[PATCHv2 1/3] gdb: work around prompt corruption caused by bracketed-paste-mode
Simon Marchi
simon.marchi@polymtl.ca
Sun Mar 27 00:59:32 GMT 2022
> Thanks for the awesome steps to reproduce! This made it super easy to
> track down the problem.
>
> Basically the problem is that I only took readline 8+ into account
> when writing the test, this spin of ubuntu is on readline 7.
>
> The patch below changes the test FAIL into a KFAIL, which, if you were
> on readline 8 would be the best we could do.
>
> But given you're on readline 7, we should be able to do better,
> getting this to a PASS! But that will require changes to GDB itself.
>
> I'd like to propose that first we merge the patch below, this removes
> the FAIL, then next week I'll post a follow on patch for GDB that
> should get this test PASSing for readline 7.
>
> Thoughts?
Hi,
I am fine with the patch below. Unfortunately I am not up to speed on
what this issue with the prompt was, so I can't really help further.
Thanks for coming back with a "fix" quickly, that helps.
Just one nit in the commit message below.
> commit 7a0add556420e2ef814a6cd58501a832fdb1fb90
> Author: Andrew Burgess <aburgess@redhat.com>
> Date: Sat Mar 26 13:41:33 2022 +0000
>
> gdb/testsuite: fix test failure when building against readline v7
>
> The test added in the commit:
>
> commit a6b413d24ccc5d76179bab866834e11fd6fec294
> Date: Fri Mar 11 14:44:03 2022 +0000
>
> gdb: work around prompt corruption caused by bracketed-paste-mode
>
> Was not written with readline 7 in mind, only readline 8+. Between
> readline 7 and 8 the escape sequence used to disable bracketed paste
> mode changed, an additional '\r' character was added to the end. In
> fact, it was the addition of this '\r' character that triggered the
> issue for which the above commit is part of the solution.
>
> Anyway, the test tries to spot the case where the output from GDB is
> not perfect, but does have the above work around applied. However,
> the pattern in the test assume that the problematic '\r' will be
assume -> assumes
Simon
More information about the Gdb-patches
mailing list