From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sat, 19 Jun 2021 00:07:05 +0000 (-0700)
Subject: remove legacy __eprintf dependency check
X-Git-Tag: 2.48.0~40^2~13
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=37b56a3d4309b62c154ee4c176454ae3227ae3b3;p=graphviz

remove legacy __eprintf dependency check

This check was forcing assertions off if using them depends on __eprintf. It is
unclear to me what the motivation for this is and it was introduced in the
initial revision with no further information. As far as I am aware, this is no
longer relevant. Modern libc is intended to provide everything necessary to
support assert, unless you are building freestanding which Graphviz does not
support.
---

diff --git a/configure.ac b/configure.ac
index 257386c45..e355c5ab2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2799,26 +2799,6 @@ AC_LINK_IFELSE([AC_LANG_PROGRAM([[#include <stdlib.h>
 	],[AC_MSG_RESULT(no)
 	])
 
-# -----------------------------------
-
-AC_MSG_CHECKING(if assert needs __eprintf)
-cat > conftest.$ac_ext <<EOF
-#include "confdefs.h"
-#include "assert.h"
-int main(int argc, char **argv) {
-(void)argv;
-assert(argc);
-; return 0; }
-EOF
-${CC-cc} -c $CFLAGS $CPPFLAGS conftest.$ac_ext 1>&5
-if test "x`nm conftest.o |$EGREP __eprintf`" != "x"; then
-    AC_MSG_RESULT(yes)
-    CFLAGS="$CFLAGS -DNDEBUG"
-else
-    AC_MSG_RESULT(no)
-fi
-rm -f conftest*
-
 # -----------------------------------------------------------------------
 
 # This is a historical artifact ... there are no other choices these days