]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Remove workaround for ancient incompatibility between readline and libedit.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 14 Mar 2015 17:43:00 +0000 (13:43 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 14 Mar 2015 17:43:26 +0000 (13:43 -0400)
commit396ef6fd8de2222cb8151e0da8710fd42514e5c2
tree726645fd1cc37da97d303666ceb61176712234df
parenta07bd32f2b130a67bf285f8ef5f1fe4ebcc3a5b8
Remove workaround for ancient incompatibility between readline and libedit.

GNU readline defines the return value of write_history() as "zero if OK,
else an errno code".  libedit's version of that function used to have a
different definition (to wit, "-1 if error, else the number of lines
written to the file").  We tried to work around that by checking whether
errno had become nonzero, but this method has never been kosher according
to the published API of either library.  It's reportedly completely broken
in recent Ubuntu releases: psql bleats about "No such file or directory"
when saving ~/.psql_history, even though the write worked fine.

However, libedit has been following the readline definition since somewhere
around 2006, so it seems all right to finally break compatibility with
ancient libedit releases and trust that the return value is what readline
specifies.  (I'm not sure when the various Linux distributions incorporated
this fix, but I did find that OS X has been shipping fixed versions since
10.5/Leopard.)

If anyone is still using such an ancient libedit, they will find that psql
complains it can't write ~/.psql_history at exit, even when the file was
written correctly.  This is no worse than the behavior we're fixing for
current releases.

Back-patch to all supported branches.
src/bin/psql/input.c