autoconf-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[SCM] GNU Autoconf source repository branch, master, updated. v2.63-240-


From: Jim Meyering
Subject: [SCM] GNU Autoconf source repository branch, master, updated. v2.63-240-g9077bf1
Date: Wed, 10 Dec 2008 16:00:42 +0000

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU Autoconf source repository".

http://git.sv.gnu.org/gitweb/?p=autoconf.git;a=commitdiff;h=9077bf12d8f0b0b6c6cdf94a1ef7837b9a2f5b6c

The branch, master has been updated
       via  9077bf12d8f0b0b6c6cdf94a1ef7837b9a2f5b6c (commit)
      from  0aae415c0c873d72b660927e6d1b648a7b438afe (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9077bf12d8f0b0b6c6cdf94a1ef7837b9a2f5b6c
Author: Jim Meyering <address@hidden>
Date:   Wed Dec 10 14:45:35 2008 +0100

    AC_HEADER_ASSERT: don't say assertions are disabled when they're not
    
    * lib/autoconf/headers.m4 (AC_HEADER_ASSERT): Do not make configure
    report "checking whether to enable assertions... no", when they are
    in fact enabled.  This is solely a bug in the output of configure.
    In spite of saying "no", NDEBUG was not defined in that case.
    Also, as noted by Eric Blake, leave assertions enabled upon
    --enable-assert=INVALID.

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog               |   10 ++++++++++
 lib/autoconf/headers.m4 |   13 +++++++------
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index fb31bfd..704ec47 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2008-12-10  Jim Meyering  <address@hidden>
+
+       AC_HEADER_ASSERT: don't say assertions are disabled when they're not
+       * lib/autoconf/headers.m4 (AC_HEADER_ASSERT): Do not make configure
+       report "checking whether to enable assertions... no", when they are
+       in fact enabled.  This is solely a bug in the output of configure.
+       In spite of saying "no", NDEBUG was not defined in that case.
+       Also, as noted by Eric Blake, leave assertions enabled upon
+       --enable-assert=INVALID.
+
 2008-12-09  Eric Blake  <address@hidden>
 
        Fix m4_location inside m4_wrap with m4 1.4.5.
diff --git a/lib/autoconf/headers.m4 b/lib/autoconf/headers.m4
index 7fe94af..0bf4f60 100644
--- a/lib/autoconf/headers.m4
+++ b/lib/autoconf/headers.m4
@@ -437,13 +437,14 @@ AC_DEFUN([AC_HEADER_ASSERT],
   AC_ARG_ENABLE([assert],
     [AS_HELP_STRING([--disable-assert], [turn off assertions])],
     [ac_enable_assert=$enableval
-     AS_IF([test "x$enableval" = xno],
-      [AC_DEFINE([NDEBUG], [1],
-       [Define to 1 if assertions should be disabled.])],
+     AS_IF(dnl
+      [test "x$enableval" = xno],
+        [AC_DEFINE([NDEBUG], [1],
+          [Define to 1 if assertions should be disabled.])],
       [test "x$enableval" != xyes],
-      [AC_MSG_WARN([invalid argument supplied to --enable-assert])
-      ac_enable_assert=no])],
-    [ac_enable_assert=no])
+        [AC_MSG_WARN([invalid argument supplied to --enable-assert])
+        ac_enable_assert=yes])],
+    [ac_enable_assert=yes])
   AC_MSG_RESULT([$ac_enable_assert])
 ])
 


hooks/post-receive
--
GNU Autoconf source repository




reply via email to

[Prev in Thread] Current Thread [Next in Thread]