Improve documentation of configure's readline/libedit switches.

configure

@@ -892,8 +892,8 @@ Optional Packages:
   --with-ldap             build with LDAP support
   --with-bonjour          build with Bonjour support
   --with-openssl          build with OpenSSL support
+  --without-readline      do not use GNU Readline nor BSD Libedit for editing
   --with-libedit-preferred  prefer BSD Libedit over GNU Readline
-  --without-readline      do not use GNU Readline / BSD Libedit line editing
   --without-zlib          do not use Zlib
   --with-gnu-ld           assume the C compiler uses GNU ld [default=no]
 
@@ -4086,38 +4086,6 @@ echo "${ECHO_T}$with_openssl" >&6
 
 
 
-#
-# Prefer libedit
-#
-
-
-
-
-# Check whether --with-libedit-preferred or --without-libedit-preferred was given.
-if test "${with_libedit_preferred+set}" = set; then
-  withval="$with_libedit_preferred"
-
-  case $withval in
-    yes)
-      :
-      ;;
-    no)
-      :
-      ;;
-    *)
-      { { echo "$as_me:$LINENO: error: no argument expected for --with-libedit-preferred option" >&5
-echo "$as_me: error: no argument expected for --with-libedit-preferred option" >&2;}
-   { (exit 1); exit 1; }; }
-      ;;
-  esac
-
-else
-  with_libedit_preferred=no
-
-fi;
-
-
-
 #
 # Readline
 #
@@ -4160,6 +4128,38 @@ echo "$as_me: WARNING: *** Readline does not work on MinGW --- disabling" >&2;}
 fi
 
 
+#
+# Prefer libedit
+#
+
+
+
+
+# Check whether --with-libedit-preferred or --without-libedit-preferred was given.
+if test "${with_libedit_preferred+set}" = set; then
+  withval="$with_libedit_preferred"
+
+  case $withval in
+    yes)
+      :
+      ;;
+    no)
+      :
+      ;;
+    *)
+      { { echo "$as_me:$LINENO: error: no argument expected for --with-libedit-preferred option" >&5
+echo "$as_me: error: no argument expected for --with-libedit-preferred option" >&2;}
+   { (exit 1); exit 1; }; }
+      ;;
+  esac
+
+else
+  with_libedit_preferred=no
+
+fi;
+
+
+
 #
 # Zlib
 #
@@ -22955,8 +22955,8 @@ with_pam) ;;
 with_ldap) ;;
 with_bonjour) ;;
 with_openssl) ;;
-with_libedit_preferred) ;;
 with_readline) ;;
+with_libedit_preferred) ;;
 with_zlib) ;;
 
        with_gnu_ld) ;;

configure.in

@@ -1,5 +1,5 @@
 dnl Process this file with autoconf to produce a configure script.
-dnl $PostgreSQL: pgsql/configure.in,v 1.476 2006/09/27 16:29:45 tgl Exp $
+dnl $PostgreSQL: pgsql/configure.in,v 1.477 2006/10/01 23:47:16 tgl Exp $
 dnl
 dnl Developers, please strive to achieve this order:
 dnl
@@ -508,18 +508,11 @@ AC_MSG_RESULT([$with_openssl])
 AC_SUBST(with_openssl)
 
 
-#
-# Prefer libedit
-#
-PGAC_ARG_BOOL(with, libedit-preferred, no,
-              [  --with-libedit-preferred  prefer BSD Libedit over GNU Readline])
-
-
 #
 # Readline
 #
 PGAC_ARG_BOOL(with, readline, yes,
-              [  --without-readline      do not use GNU Readline / BSD Libedit line editing])
+              [  --without-readline      do not use GNU Readline nor BSD Libedit for editing])
 # readline on MinGW has problems with backslashes in psql and other bugs.
 # This is particularly a problem with non-US code pages.
 # Therefore disable its use until we understand the cause. 2004-07-20
@@ -531,6 +524,13 @@ if test "$PORTNAME" = "win32"; then
 fi
 
 
+#
+# Prefer libedit
+#
+PGAC_ARG_BOOL(with, libedit-preferred, no,
+              [  --with-libedit-preferred  prefer BSD Libedit over GNU Readline])
+
+
 #
 # Zlib
 #

doc/src/sgml/installation.sgml

@@ -1,4 +1,4 @@
-<!-- $PostgreSQL: pgsql/doc/src/sgml/installation.sgml,v 1.262 2006/09/16 00:30:14 momjian Exp $ -->
+<!-- $PostgreSQL: pgsql/doc/src/sgml/installation.sgml,v 1.263 2006/10/01 23:47:16 tgl Exp $ -->
 
 <chapter id="installation">
  <title><![%standalone-include[<productname>PostgreSQL</>]]>
@@ -105,6 +105,9 @@ su - postgres
       <indexterm>
        <primary>readline</primary>
       </indexterm>
+      <indexterm>
+       <primary>libedit</primary>
+      </indexterm>
 
       The <acronym>GNU</> <productname>Readline</> library (for
       simple line editing and command history retrieval) is
@@ -866,22 +869,25 @@ su - postgres
       </varlistentry>
 
       <varlistentry>
-       <term><option>--with-libedit-preferred</option></term>
+       <term><option>--without-readline</option></term>
        <listitem>
         <para>
-         Favors the use of the BSD-licensed <application>libedit</> library
+         Prevents use of the <application>Readline</> library
-         rather than GPL-licensed <application>Readline</>.
+         (and <application>libedit</> as well).  This option disables
+         command-line editing and history in
+         <application>psql</application>, so it is not recommended.
         </para>
        </listitem>
       </varlistentry>
 
       <varlistentry>
-       <term><option>--without-readline</option></term>
+       <term><option>--with-libedit-preferred</option></term>
        <listitem>
         <para>
-         Prevents use of the <application>Readline</> library.  This disables
+         Favors the use of the BSD-licensed <application>libedit</> library
-         command-line editing and history in
+         rather than GPL-licensed <application>Readline</>.  This option
-         <application>psql</application>, so it is not recommended.
+         is significant only if you have both libraries installed; the
+         default in that case is to use <application>Readline</>.
         </para>
        </listitem>
       </varlistentry>