From: Ralph Corderoy <ralph@inputplus.co.uk>
Date: Wed, 19 Oct 2016 13:39:53 +0000 (+0100)
Subject: Remove ./configure's --enable-debug support;  it did nothing.
X-Git-Url: https://diplodocus.org/git/nmh/commitdiff_plain/8eacff500716cab8256b032c0f4dd29bc2dc16bb?ds=sidebyside;hp=390acba226dd2bce2335ffed2318a79d554861da

Remove ./configure's --enable-debug support;  it did nothing.
David on nmh-workers said "that looks like a relic that can be removed".
---

diff --git a/INSTALL b/INSTALL
index 6ea100d8..c6a4077a 100644
--- a/INSTALL
+++ b/INSTALL
@@ -195,9 +195,6 @@ Options for configure
 --mandir=DIR     (DEFAULT is ${prefix}/man)
      nmh's man pages are installed here.
 
---enable-debug
-     Enable debugging support.
-
 --with-editor=EDITOR  (DEFAULT is vi)
      specify the full path of the default editor to use.  If this
      option is not given, then the configuration process will search
diff --git a/Makefile.am b/Makefile.am
index 3a04f781..7cc5387b 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -744,9 +744,6 @@ rpm: dist
 ## targets because they will be used so infrequently, if at all.
 ##
 gcov:
-	@if ./config.status --config | grep -e --enable-debug >/dev/null; then \
-	   echo For best results with gcov, configure without --enable-debug; \
-	 fi
 	@echo rebuilding with AM_CFLAGS=--coverage . . .
 	@(make clean  &&  make AM_CFLAGS=--coverage) > /dev/null  && \
 	  make check AM_CFLAGS=--coverage
diff --git a/configure.ac b/configure.ac
index 4ed79bbc..9a8b9977 100644
--- a/configure.ac
+++ b/configure.ac
@@ -23,10 +23,6 @@ dnl --------------------------
 dnl CHECK COMMAND LINE OPTIONS
 dnl --------------------------
 
-dnl Do you want to debug nmh?
-AC_ARG_ENABLE([debug],
-  AS_HELP_STRING([--enable-debug],[enable nmh code debugging]))
-
 dnl Do you want client-side support for using SASL for authentication?
 dnl Note that this code will be enabled for both POP and SMTP
 AC_ARG_WITH([cyrus-sasl], AS_HELP_STRING([--with-cyrus-sasl],
diff --git a/docs/contrib/build_nmh b/docs/contrib/build_nmh
index e25d480c..73111439 100755
--- a/docs/contrib/build_nmh
+++ b/docs/contrib/build_nmh
@@ -239,7 +239,7 @@ config_opts="--prefix=$config_prefix"
 [ "$config_oauth" = y ]  &&  \
   config_opts="$config_opts --with-oauth"
 [ $config_debug = y ]  &&  \
-  config_opts="$config_opts --enable-debug --enable-assert"
+  config_opts="$config_opts --enable-assert"
 
 #### dotlocking, the usual default, requires chgrp and chmod of inc.
 installpriv=
diff --git a/docs/pending-release-notes b/docs/pending-release-notes
index b7f6138a..8e1aa0d7 100644
--- a/docs/pending-release-notes
+++ b/docs/pending-release-notes
@@ -62,6 +62,7 @@ NEW FEATURES
 - Fewer lseek(2)s will be used when reading headers in the common case.
 - Single-character headers can be reliably formatted, etc., instead of
   apparently being missing.
+- ./configure's --enable-debug has been removed;  it did nothing.
 
 -----------------
 OBSOLETE FEATURES