]> granicus.if.org Git - graphviz/commitdiff
remove legacy __eprintf dependency check
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 19 Jun 2021 00:07:05 +0000 (17:07 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 25 Jun 2021 04:09:44 +0000 (21:09 -0700)
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.

configure.ac

index 257386c4563a7ea755355872fc12455f9e934824..e355c5ab296ca7ba86bb28e2a9bb2bc89931e874 100644 (file)
@@ -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