Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Martin D Kealey
On Thu, 21 Aug 2025 at 05:48, Grisha Levit wrote: > You can reproduce this by adding a key binding to your terminal emulator > to send \003\033 for some key combination. > Or if using a terminal emulator that doesn't support programmable bindings (or indeed, just for the simplicity): stty intr

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Chet Ramey
On 8/20/25 3:36 PM, Chet Ramey wrote: = ==1078506==ERROR: AddressSanitizer: heap-use-after-free on address 0x7c315f7e217c at pc 0x55bc1d03b545 bp 0x7ffdbdb3c920 sp 0x7ffdbdb3c910 WRITE of size 4 at 0x7c315f7e217c thread T0      #0

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Dag-Erling Smørgrav
Collin Funk writes: > Chet Ramey writes: > > This means your bison is too old. configure should have told you. > This is a common MacOS issue. Since Apple is allergic to GPLv3+. Try using byacc instead. DES -- Dag-Erling Smørgrav - d...@des.no

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread penguin p
> Chet Ramey > What's the exact key sequence you're inputting, and are you still sending > ESC after ^C from your keyboard config? (And why do you do that?) > I assume you're typing ^R^C and your keyboard is sending ^R^C^[, right? Not exactly, when I tested Ctrl + R then Ctrl + C I was also pressi

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Grisha Levit
On Wed, Aug 20, 2025 at 3:48 PM Grisha Levit wrote: > You can reproduce this by adding a key binding to your terminal emulator > to send \003\033 for some key combination. This worked for me in both > Terminal.app and iTerm, though it might take a few rounds of entering > incremental search mode a

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Grisha Levit
On Wed, Aug 20, 2025 at 3:37 PM Chet Ramey wrote: > > On 8/20/25 1:45 PM, penguin p wrote: > > > > > https://cgit.git.savannah.gnu.org/cgit/bash.git/?h=devel > cgit.git.savannah.gnu.org/cgit/bash.git/?h=devel> > > > > Sorry, I am not able to build using the Mac, so I go into arch linux then > >

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Collin Funk
Chet Ramey writes: >> ./parse.y:501.23-27: symbol YYEOF is used, but is not defined as a >> token and has no rules > > This means your bison is too old. configure should have told you. This is a common MacOS issue. Since Apple is allergic to GPLv3+. See: $ /usr/bin/bison --version bison

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Chet Ramey
On 8/20/25 1:45 PM, penguin p wrote: > https://cgit.git.savannah.gnu.org/cgit/bash.git/?h=devel cgit.git.savannah.gnu.org/cgit/bash.git/?h=devel> Sorry, I am not able to build using the Mac, so I go into arch linux then build and tested over there. I pressed so hard I got this error ``` Th

Re: [PATCH] _rl_isearch_cleanup: avoid nested execution

2025-08-20 Thread Chet Ramey
On 8/19/25 10:12 PM, penguin p wrote: > Thanks for your careful analysis and fix. I applied a close variant, and > that will be in the next devel branch push. Please let me know when the new patch is ready. I can test it for you. https://cgit.git.savannah.gnu.org/cgit/bash.git/?h=devel -- `