Daniel Gustafsson <dan...@yesql.se> writes:
> If popen fails in pipe_read_line we invoke perror for the error message, and
> pipe_read_line is in turn called by find_other_exec which is used in both
> frontend and backend code. This is an old codepath, and it seems like it
> ought
> to be replaced with the more common logging tools we now have like in the
> attached diff (which also makes the error translated as opposed to now). Any
> objections to removing this last perror() call?
I didn't check your replacement code in detail, but I think we have
a policy against using perror, mainly because we can't count on it
to localize consistently with ereport et al. My grep confirms this
is the only use, so +1 for removing it.
regards, tom lane
