Remove --disable-maintainer-mode option from 'configure'. (Bug#11555)

It is confusingly named and rarely useful.  See, for example,
<http://lists.gnu.org/archive/html/emacs-devel/2011-12/msg00089.html>.
* INSTALL.BZR: Don't mention --disable-maintainer-mode.
* Makefile.in (MAINTAINER_MODE_FLAG): Remove; all uses removed.
* configure.in: Remove --disable-maintainer-mode.
(USE_MAINTAINER_MODE, MAINT): Remove.
* admin/make-tarball.txt: Don't worry about maintainer mode.
* etc/NEWS: Mention this.

diff --git a/ChangeLog b/ChangeLog
index ef790cd8d30fb32eaa7c460eae5daaf495999abf..95d9ecbf01b626574d66098b91d18621c1cafc5e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2012-06-01  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Remove --disable-maintainer-mode option from 'configure'.  (Bug#11555)
+       It is confusingly named and rarely useful.  See, for example,
+       <http://lists.gnu.org/archive/html/emacs-devel/2011-12/msg00089.html>.
+       * INSTALL.BZR: Don't mention --disable-maintainer-mode.
+       * Makefile.in (MAINTAINER_MODE_FLAG): Remove; all uses removed.
+       * configure.in: Remove --disable-maintainer-mode.
+       (USE_MAINTAINER_MODE, MAINT): Remove.
+
 2012-05-28  Paul Eggert  <eggert@cs.ucla.edu>
 
        Make 'configure' a bit smaller and faster.

diff --git a/INSTALL.BZR b/INSTALL.BZR
index 93e53fd09aad5b14126b2837497567694170fc37..675eaaf12dffc700ba3facf4a5b69b63d7f01906 100644 (file)
--- a/INSTALL.BZR
+++ b/INSTALL.BZR
@@ -30,7 +30,7 @@ options you can set):
 
 If you want later builds to go faster, at the expense of sometimes
 doing the wrong thing if you update the build procedure, you can
-invoke "./configure -C --disable-maintainer-mode" instead.
+invoke "./configure -C" instead.
 
 Some of the files that are included in the Emacs tarball, such as
 byte-compiled Lisp files, are not stored in Bazaar.  Therefore, to

diff --git a/Makefile.in b/Makefile.in
index 699c4794d92263f1facb31bcff75c869b13f7c45..fb2530d13df59da41e0a51c22e854d621dd6da1d 100644 (file)
--- a/Makefile.in
+++ b/Makefile.in
@@ -66,10 +66,8 @@ CDPATH=
 
 # ==================== Things `configure' Might Edit ====================
 
-MAINTAINER_MODE_FLAG = --disable-maintainer-mode
-@MAINT@MAINTAINER_MODE_FLAG = --enable-maintainer-mode
 cache_file = @cache_file@
-CONFIGURE_FLAGS = --cache-file=$(cache_file) $(MAINTAINER_MODE_FLAG)
+CONFIGURE_FLAGS = --cache-file=$(cache_file)
 
 CC=@CC@
 CFLAGS=@CFLAGS@
@@ -360,16 +358,17 @@ config.status: ${srcdir}/configure ${srcdir}/lisp/version.el
            ./configure $(CONFIGURE_FLAGS); \
        fi
 
-AUTOCONF_INPUTS = @MAINT@ $(srcdir)/configure.in $(srcdir)/aclocal.m4
+AUTOCONF_INPUTS = $(srcdir)/configure.in $(srcdir)/aclocal.m4
 
 $(srcdir)/configure: $(AUTOCONF_INPUTS)
        cd ${srcdir} && autoconf
 
-ACLOCAL_INPUTS = @MAINT@ $(srcdir)/m4/gnulib-comp.m4
+ACLOCAL_INPUTS = $(srcdir)/m4/gnulib-comp.m4
 $(srcdir)/aclocal.m4: $(ACLOCAL_INPUTS)
        cd $(srcdir) && aclocal -I m4
 
-AUTOMAKE_INPUTS = @MAINT@ $(srcdir)/aclocal.m4 $(srcdir)/lib/Makefile.am $(srcdir)/lib/gnulib.mk
+AUTOMAKE_INPUTS = $(srcdir)/aclocal.m4 $(srcdir)/lib/Makefile.am \
+  $(srcdir)/lib/gnulib.mk
 $(srcdir)/lib/Makefile.in: $(AUTOMAKE_INPUTS)
        cd $(srcdir) && automake --gnu -a -c lib/Makefile
 am--refresh: $(srcdir)/aclocal.m4 $(srcdir)/configure $(srcdir)/src/config.in

diff --git a/admin/ChangeLog b/admin/ChangeLog
index 0c3bbf93873245e9a66bdf92752f1c789188211d..3ed52c681544fef11dfa6a3cb75802fd04fbbddf 100644 (file)
--- a/admin/ChangeLog
+++ b/admin/ChangeLog
@@ -1,3 +1,8 @@
+2012-06-01  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Remove --disable-maintainer-mode option from 'configure'.  (Bug#11555)
+       * make-tarball.txt: Don't worry about maintainer mode.
+
 2012-05-28  Paul Eggert  <eggert@cs.ucla.edu>
 
        * CPP-DEFINES: Remove HAVE_SYSINFO.

diff --git a/admin/make-tarball.txt b/admin/make-tarball.txt
index 0acd29bbc1363791832b4596cd9e5f43650d1104..ea9c4a3c2d17312b5887fa77cc57f89c0d6a1515 100644 (file)
--- a/admin/make-tarball.txt
+++ b/admin/make-tarball.txt
@@ -28,13 +28,7 @@ For each step, check for possible errors.
     refer to a newer release of Emacs.  (This is probably needed only
     when preparing a major Emacs release, or branching for it.)
 
-5.  Edit configure.in so that maintainer-mode is off by default.
-    (FIXME - need to find a better way of dealing with this.
-     Or maybe it's fine and indeed correct to leave it on?
-     See http://lists.gnu.org/archive/html/emacs-devel/2011-03/msg00859.html
-     and subsequent.)
-
-     autoreconf -i -I m4 --force
+5.   autoreconf -i -I m4 --force
      make bootstrap
 
 6.  Commit etc/AUTHORS, all the files changed by M-x set-version, and

diff --git a/configure.in b/configure.in
index 9b805bf9a5fbfb5f2996d4e38a624ac6041d1e1f..055db30772e3ccb60866503ead39875d4107e200 100644 (file)
--- a/configure.in
+++ b/configure.in
@@ -235,19 +235,6 @@ AC_ARG_ENABLE(asserts,
       USE_XASSERTS=$enableval,
       USE_XASSERTS=no)
 
-AC_ARG_ENABLE(maintainer-mode,
-[AS_HELP_STRING([--disable-maintainer-mode],
-                [disable make rules and dependencies not useful (and sometimes
-                confusing) to the casual installer])],
-      USE_MAINTAINER_MODE=$enableval,
-      USE_MAINTAINER_MODE=yes)
-if test $USE_MAINTAINER_MODE = yes; then
-  MAINT=
-else
-  MAINT=#
-fi
-AC_SUBST(MAINT)
-
 AC_ARG_ENABLE(locallisppath,
 [AS_HELP_STRING([--enable-locallisppath=PATH],
                 [directories Emacs should search for lisp files specific

diff --git a/etc/ChangeLog b/etc/ChangeLog
index 964fce10ec19b7525e8620c4058f8fab69dcbdf3..25d794a01535abbea09287b42437095f6a2b2e8a 100644 (file)
--- a/etc/ChangeLog
+++ b/etc/ChangeLog
@@ -1,3 +1,8 @@
+2012-06-01  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Remove --disable-maintainer-mode option from 'configure'.  (Bug#11555)
+       * NEWS: Mention this.
+
 2012-06-01  Andrew Beals  <andrew.beals@gmail.com>  (tiny change)
 
        * spook.lines: Additions.  (Bug#11598)

diff --git a/etc/NEWS b/etc/NEWS
index f152228ec964b0db1f132495f7a0904e1d07260a..3ffbbd24d45a36478a1dcb921223937c2a3d1484 100644 (file)
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -29,6 +29,9 @@ possibly-questionable C code.  On a recent GNU system there should be
 no warnings; on older and on non-GNU systems the generated warnings
 may be useful.
 
+** The configure option '--disable-maintainer-mode' has been removed,
+as it was confusingly-named and rarely useful.
+
 ---
 ** Emacs uses libtinfo in preference to libncurses, if available.