On Mon, Sep 28, 2026 at 12:54:18PM -0400, Andrew Dunstan wrote:
> I think the right way is to have a wrapper function as in the attached, so
> instead of calling IPC::Run::run you would call ipc_run ...
> 
> Note that this patch only touches the sites that will actually be affected
> by this. If we wanted to be consistent we would replace calls in a further
> 23 files.

That seems much better in the long-run, thanks.  The
ipc_run_text_mode() being published in Utils.pm does not seem
necessary, though.  This is a part that I'm sure could become
confusing if we allow code outside of Utils.pm to use it.
--
Michael

Attachment: signature.asc
Description: PGP signature

Reply via email to