Re: psql: Buggy interactions with wraparound in maximized gnome terminals.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: chet(dot)ramey(at)case(dot)edu
Cc: Tomas Zubiri <me(at)tomaszubiri(dot)com>, bug-readline(at)gnu(dot)org, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: psql: Buggy interactions with wraparound in maximized gnome terminals.
Date: 2019-12-21 21:40:29
Message-ID: 26175.1576964429@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Chet Ramey <chet(dot)ramey(at)case(dot)edu> writes:
> On 12/20/19 4:48 PM, Tomas Zubiri wrote:
>> This is probably hard to reproduce, but I entered a state where the
>> command I was writing was wrapping around at the 81st character while
>> in full screen, and the 1st character was being overwritten.This did
>> not affect the command sent to the server, but I would not have a way
>> to see what command I just entered.

> It seems like the readline SIGWINCH handler, which causes it to update its
> idea of the screen dimensions, isn't being called. I can't reproduce this
> using readline and bash, and I don't know what postgres does with its
> signal handling, or how it configures readline's signal handling (I assume
> it doesn't do anything special).

We don't touch SIGWINCH at all, so if readline sets up a handler for that,
it ought to behave as expected.

I notice though that we do this just before each readline() call:

/*
* Some versions of readline don't notice SIGWINCH signals that arrive
* when not actively reading input. The simplest fix is to always
* re-read the terminal size. This leaves a window for SIGWINCH to be
* missed between here and where readline() enables libreadline's
* signal handler, but that's probably short enough to be ignored.
*/
#ifdef HAVE_RL_RESET_SCREEN_SIZE
rl_reset_screen_size();
#endif

Is it conceivable that that bollixes things somehow, on some builds?
(I don't think the OP specified which readline and/or Postgres version
he was using.)

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tomas Zubiri 2019-12-21 22:37:20 Re: psql: Buggy interactions with wraparound in maximized gnome terminals.
Previous Message Chet Ramey 2019-12-21 20:31:46 Re: psql: Buggy interactions with wraparound in maximized gnome terminals.