Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
> But now that I read the patch again, I'm not sure why this needs to
> touch libpq. The formatting of error messages in psql should be handled
> in psql.
Maybe in an ideal world that'd be the case, but psql has always just
depended on PQerrorMessage(). I don't think this patch should be
tasked with changing that. I'm not even sure that doing so would be
a good idea: I think the idea there is that whatever we believe is a
nice error-reporting option for psql ought to be easily available to
other libpq clients too.
(Note: I haven't read the patch and don't mean to be saying that it's
necessarily OK. But I'm fine with the idea that something like this
involves touching libpq more than psql.)
regards, tom lane
