[Emacs-diffs] master 650defd: Revert "Make --without-pop the default."

[Eli Zaretskii]

branch: master
commit 650defd4e74ad00dd16e61c1fc5b4f497821c3d2
Author: Eli Zaretskii <address@hidden>
Commit: Eli Zaretskii <address@hidden>

    Revert "Make --without-pop the default."
    
    This reverts commit 9319de675e395517f9a7b50cae1a3aad9cd0abc2.
---
 INSTALL      |  7 +++++--
 configure.ac | 31 +++++++++++++++++++++----------
 etc/NEWS     | 12 +++---------
 3 files changed, 29 insertions(+), 21 deletions(-)

diff --git a/INSTALL b/INSTALL
index 9b6c04f..a99f107 100644
--- a/INSTALL
+++ b/INSTALL
@@ -265,14 +265,17 @@ uses the GNU Mailutils 'movemail' program to retrieve 
mail.  Otherwise
 the Emacs build procedure builds and installs an auxiliary 'movemail'
 program, a limited and insecure substitute that Emacs can use when
 Mailutils is not installed; when this happens, there are several
-configure options such as --with-mail-unlink that provide fine-grained
+configure options such as --without-pop that provide fine-grained
 control over Emacs 'movemail' construction.
 
 The Emacs mail reader RMAIL is configured to be able to read mail from
 a POP3 server by default.  Versions of the POP protocol older than
 POP3 are not supported.  While POP3 support is typically enabled,
 whether Emacs actually uses POP3 is controlled by individual users;
-see the Rmail chapter of the Emacs manual.
+see the Rmail chapter of the Emacs manual.  Unless you configure
+--with-mailutils, it is a good idea to configure --without-pop so that
+users are less likely to inadvertently read email via insecure
+channels.
 
 For image support you may have to download, build, and install the
 appropriate image support libraries for image types other than XBM and
diff --git a/configure.ac b/configure.ac
index 648000e..fad7fb0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -229,17 +229,19 @@ AC_DEFUN([OPTION_DEFAULT_ON], [dnl
    m4_bpatsubst([with_$1], [[^0-9a-z]], [_])=$with_features])dnl
 ])dnl
 
+# FIXME: The default options '--without-mailutils --with-pop' result
+# in a movemail implementation that supports only unencrypted POP3
+# connections.  Encrypted connections should be the default.
+
 OPTION_DEFAULT_OFF([mailutils],
-  [rely on GNU Mailutils being installed, so that --with-pop through
-   --with-mailhost are irrelevant])
+  [rely on GNU Mailutils being installed; this makes the options
+   --without-pop through --with-mailhost irrelevant])
 if test "$with_mailutils" = no; then
   with_mailutils=
 fi
 AC_SUBST([with_mailutils])
 
-OPTION_DEFAULT_OFF([pop],
-  [support POP mail retrieval with movemail (not recommended, as its
-   email retrieval is unencrypted)])
+OPTION_DEFAULT_ON([pop],[don't support POP mail retrieval with movemail])
 if test "$with_pop" = yes; then
    AC_DEFINE(MAIL_USE_POP)
 fi
@@ -5479,11 +5481,15 @@ fi
 
 AC_OUTPUT
 
-if test ! "$with_mailutils" && test "$with_pop" = yes; then
+if test ! "$with_mailutils"; then
   case $opsys in
     cygwin | mingw32)
       # Don't suggest GNU Mailutils, as it hasn't been ported.
-      emacs_fix_movemail="use '$0' (without '--with-pop')";;
+      if test "$with_pop" = yes; then
+       emacs_fix_movemail="use '$0 --without-pop'"
+      else
+       emacs_fix_movemail=
+      fi;;
     *)
       emacs_fix_movemail="use '$0 --with-mailutils'"
       case `(movemail --version) 2>/dev/null` in
@@ -5493,9 +5499,14 @@ if test ! "$with_mailutils" && test "$with_pop" = yes; 
then
       esac;;
   esac
 
-  AC_MSG_WARN([This configuration installs a 'movemail' program
-that retrieves POP3 email via only unencrypted channels.
-To avoid the problem you can $emacs_fix_movemail.])
+  if test "$with_pop" = yes; then
+    AC_MSG_WARN([This configuration installs a 'movemail' program
+that retrieves POP3 email via only insecure channels.
+To fix this you can $emacs_fix_movemail.])
+  elif test "$emacs_fix_movemail"; then
+    AC_MSG_NOTICE([This configuration installs a 'movemail' program that cannot
+retrieve POP3 email.  You might want to $emacs_fix_movemail.])
+  fi
 fi
 
 test "$MAKE" = make || AC_MSG_NOTICE([Now you can run '$MAKE'.])
diff --git a/etc/NEWS b/etc/NEWS
index bf50d1a..cd98f53 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -32,15 +32,9 @@ version 2.6.6 or later.
 
 ** The new option 'configure --with-mailutils' causes Emacs to rely on
 GNU Mailutils 'movemail' to retrieve email.  By default, the Emacs
-build procedure continues to build and install a limited 'movemail'
-substitute.  Although --with-mailutils is recommended, it is not yet
-the default due to backward-compatibility concerns.
-
-** The option 'configure --without-pop' is now the default.
-(This option's setting matters only if --with-mailutils is not used.)
-Using --with-pop (the previous default) preserves the historical
-behavior where Emacs 'movemail' retrieves network email only via
-unencrypted channels.
+build procedure continues to build and install a limited and insecure
+'movemail' substitute.  Although --with-mailutils is recommended, it
+is not yet the default due to backward-compatibility concerns.
 
 ** The new option 'configure --enable-gcc-warnings=warn-only' causes
 GCC to issue warnings without stopping the build.  This behavior is