Re: [HACKERS] Should libedit be preferred to libreadline? - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: [HACKERS] Should libedit be preferred to libreadline?
Date
Msg-id 200512021823.jB2INZX26304@candle.pha.pa.us
Whole thread Raw
In response to Re: [HACKERS] Should libedit be preferred to libreadline?  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [HACKERS] Should libedit be preferred to libreadline?
List pgsql-patches
Tom Lane wrote:
> Chris Browne <cbbrowne@acm.org> writes:
> > To my mind, giving BIG weight to the opinions of the relatively small
> > set of individuals that manage PostgreSQL packages for the popular
> > distributions of Linux and *BSD seems fairly appropriate.
>
> The packagers are bright enough to adapt to whatever we do --- it's
> the people who build their own from source that I'm worried about.
> --with-readline has worked fine for libedit users for a long time,
> and suddenly changing its semantics strikes me as a bad idea.
>
> The other problem with the "let's be deterministic" argument is that
> it rests on a fallacy, which is that configure can reliably tell the
> difference between libreadline and libedit.  Darwin, for example, goes
> to some lengths to confuse matters.
>
> (I think I'd actually be for the determinism point of view if it could
> provide an #ifdef flag saying which library is in use --- then we could
> fix the write_history return value problem we're seeing on Darwin ---
> but I don't think we can do it short of a behavioral probe during
> configure.)

Let me add one more thing into the mix.  Right now, you have to say
--without-readline if you don't want any command-line editing.
(Remember, configure will currently fail if it doesn't find readline or
libedit.)  How will we do that if we have separate flags for readline
and libedit?

Right now, --with-readline is on by default, and fails if readline or
libedit can not be found.  If we change to two flags, then we will have
to say --without-readline --with-libedit to get libedit to work, and
--without-readline and --without-libedit to configure for no line
editing capability.  Does anyone want to try to explain that in an email
over and over again for 8.2?  :-)

The basic problem is that with two deterministic flags the default
values for those flags are unclear.  We really want to default to
looking for either of them, but want to the ability to give preference
to one over the other.  Deterministic is great, but how do we do that
with reasonable defaults?

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: Numeric 508 datatype
Next
From: Tom Lane
Date:
Subject: Re: [HACKERS] Should libedit be preferred to libreadline?