Re: TAP testing for psql's tab completion code

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: TAP testing for psql's tab completion code
Date: 2019-12-29 17:42:36
Message-ID: alpine.DEB.2.21.1912291816440.14206@pseudo
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Hello Tom,

>> If you have to install IO::Pty anyway, ISTM you can also install Expect.
>
> My point is precisely that buildfarm owners *won't* have to install
> IO::Pty; it comes in a default Perl install almost everywhere.
> I'm afraid that's not true of Expect.

Hmmm. That is a good argument.

> Now in both cases we could avoid raising the bar by allowing the
> script to "skip" if the module isn't there.

Yep.

>> IO::Pty documentation says that it is "mainly used by Expect", which is a
>> clue that IO::Pty is not much better than Expect as a dependency.
>
> You're just guessing, not looking at facts on the ground. [...]

I'm not guessing what the documentation says:-) But for the consequences,
indeed I was guessing.

> Well, actually, it's possible that on some of these boxes it was pulled
> in by the IPC::Run package,

Ah, you are guessing right, IPC::Run requires IO::Pty, so it should be
available everywhere the buildfarm scripts already run. Maybe.

I've looked at your PoC implementation:

I'm not fan of relying on the configure stuff ("with_readline"), in my
Expect version I tested if history capabilities are available from psql
itself.

I did not paid attention not to overwrite the psql history file, though.

For the psql coverage patch, I was more ambitious and needed less
assumption about the configuration, I only forced -X.

--
Fabien.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Janes 2019-12-29 17:46:50 Re: [PATCH] Increase the maximum value track_activity_query_size
Previous Message Tom Lane 2019-12-29 17:27:13 Re: Incremental View Maintenance: ERROR: out of shared memory