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.