From c347c3bbba2b3eea2442b641a11a381dea709cd8 Mon Sep 17 00:00:00 2001 From: Ralph Corderoy Date: Fri, 5 May 2017 13:11:15 +0100 Subject: [PATCH] configure.ac: Enable assert(3) by default. Revert 167e542b that disabled assert(3) by default after private email discussion with David Levine. I suggested it being enabled by default in git so those running nmh from git were testing them for us, with it being disabled as part of the release process so packagers continue with it disabled. David pointed out that would make the release intentionally different from what had been long tested so the default is now enabled assertions everywhere unless disabled by a packager. --- configure.ac | 5 ----- 1 file changed, 5 deletions(-) diff --git a/configure.ac b/configure.ac index d1a755ca..8651dd1b 100644 --- a/configure.ac +++ b/configure.ac @@ -214,11 +214,6 @@ test "$nmh_cv_has_q_unused_arguments" = "yes" && \ AM_LDFLAGS="${AM_LDFLAGS:+$AM_LDFLAGS }-Qunused-arguments" AC_SUBST([AM_LDFLAGS]) -dnl This hack turns off assertions by default, assuming -dnl that configure still uses this shell variable. Without -dnl it, AC_HEADER_ASSERT enables assertions by default. -test -z "$enable_assert" && enable_assert=no - AC_HEADER_ASSERT dnl ------------------ -- 2.56.0