[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Readline not being able to handle long lines
From: |
Mihail Konev |
Subject: |
Re: [PATCH] Readline not being able to handle long lines |
Date: |
Tue, 29 Nov 2016 13:27:27 +0500 |
On Tue, Nov 29, 2016 at 12:47:26PM +0500, Mihail Konev wrote:
> On Tue, Nov 29, 2016 at 08:27:00AM +0500, Mihail Konev wrote:
> > Msys2 on windows, at least, should do it without a problem.
>
> Tried both it (bash 4.3.46) and arch linux in qemu (bash 4.3.46).
> Neither reproduced.
>
> Nevertheless, MSYS2 was exhibiting the bug all the time, most likely,
> with bash 4.3.46.
>
> Perhaps this has to do with ^Z and fg ...
>
> ... but the real linux system exhibits it, straight from the console login.
> And in terminal emulators too.
>
> And the patch helps on f459cbd8be37b28be1dc90315e0ab51d7f211301,
> i.e. bash 4.4.5
>
> Would now try to hit this on MSYS2 again :)
It was triggered by PS1.
Buggy:
PS1='\[\e]0;\w\a\]\n\[\e[32m\]\u@\h \[\e[35m\]$MSYSTEM\[\e[0m\]
\[\e[33m\]\w\[\e[0m\]\n\$ '
Buggy (replace the ^-sequences in the second line):
PS1="[\${PWD##*/}]\$ " # set the prompt
PS1="^[]0;\$PWD^G^M$PS1" # set the window title
Does not trigger:
PS1='\[\e]0;WINDOW_TITLE_HERE \w\a\][${PWD##*/}]\$ '
Does not trigger:
PS1='[bash]$ '
For the second one, replacing ${PWD##*/} with 'bash' does not help.
But commenting out the set-window-title does.
Escape sequences were all googled/copypasted.
- [PATCH] Readline not being able to handle long lines, Mihail Konev, 2016/11/28
- Re: [PATCH] Readline not being able to handle long lines, Mihail Konev, 2016/11/28
- Re: [PATCH] Readline not being able to handle long lines, Chet Ramey, 2016/11/28
- Re: [PATCH] Readline not being able to handle long lines, Mihail Konev, 2016/11/28
- Re: [PATCH] Readline not being able to handle long lines, Mihail Konev, 2016/11/29
- Re: [PATCH] Readline not being able to handle long lines,
Mihail Konev <=
- Re: [PATCH] Readline not being able to handle long lines, Mihail Konev, 2016/11/29
- Re: [PATCH] Readline not being able to handle long lines, Greg Wooledge, 2016/11/29
- Re: [PATCH] Readline not being able to handle long lines, Mihail Konev, 2016/11/29
- Re: [PATCH] Readline not being able to handle long lines, Mihail Konev, 2016/11/29