From 45a34cf267b823c6f1b62cce215b38546c18a031 Mon Sep 17 00:00:00 2001 From: Albert Cheng Date: Mon, 19 Aug 2002 15:28:05 -0500 Subject: [svn-r5878] Purpose: fix/feature Description: v1.4's configure had enable-trace default as on. That is not right for production code as it increase the object code and uses up compute time. Changed --enable-trace default to depend on --enable-debug. If debug is on, trace is on. Otherwise it is off. Also added the printing of CFLAGS, CPPFLAGS, and LDFLAGS in the summary. Platforms tested: eirene, arabica (pp). --- configure | 57 ++++++++++++++++++++++++++++++++++++--------------------- configure.in | 57 +++++++++++++++++++++++++++++++++++++-------------------- 2 files changed, 73 insertions(+), 41 deletions(-) diff --git a/configure b/configure index 46193b9..2da2646 100755 --- a/configure +++ b/configure @@ -1015,7 +1015,8 @@ Optional Features: specify a comma-separated list of package names without the leading H5 or the word no. The default is most packages. - --disable-trace Disable API tracing capability + --enable-trace Enable API tracing capability. Default=no if debug + is disabled. --enable-parallel=TYPE Search for MPI-IO and MPI support files Optional Packages: @@ -4101,7 +4102,7 @@ test "x$enable_libtool_lock" != xno && enable_libtool_lock=yes case $host in *-*-irix6*) # Find out which ABI we are using. - echo '#line 4104 "configure"' > conftest.$ac_ext + echo '#line 4105 "configure"' > conftest.$ac_ext if { (eval echo "$as_me:$LINENO: \"$ac_compile\"") >&5 (eval $ac_compile) 2>&5 ac_status=$? @@ -4642,7 +4643,7 @@ chmod -w . save_CFLAGS="$CFLAGS" CFLAGS="$CFLAGS -o out/conftest2.$ac_objext" compiler_c_o=no -if { (eval echo configure:4645: \"$ac_compile\") 1>&5; (eval $ac_compile) 2>out/conftest.err; } && test -s out/conftest2.$ac_objext; then +if { (eval echo configure:4646: \"$ac_compile\") 1>&5; (eval $ac_compile) 2>out/conftest.err; } && test -s out/conftest2.$ac_objext; then # The compiler can only warn and ignore the option if not recognized # So say no if there are warnings if test -s out/conftest.err; then @@ -6453,7 +6454,7 @@ else lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2 lt_status=$lt_dlunknown cat > conftest.$ac_ext < conftest.$ac_ext <&5 -echo $ECHO_N "checking for API tracing... $ECHO_C" >&6 +echo $ECHO_N "checking for API tracing... $ECHO_C" >&6; # Check whether --enable-trace or --disable-trace was given. if test "${enable_trace+set}" = set; then enableval="$enable_trace" @@ -29527,18 +29528,27 @@ if test "${enable_trace+set}" = set; then fi; - -if test -z "$TRACE" -o "X$TRACE" = "Xyes"; then - echo "$as_me:$LINENO: result: yes" >&5 +if test "X-$TRACE" = X- ; then + if test -z "$DEBUG_PKG" ; then + TRACE=no + else + TRACE=yes + fi +fi +case "X-$TRACE" in + X-yes) + echo "$as_me:$LINENO: result: yes" >&5 echo "${ECHO_T}yes" >&6 - TRACE_API=yes - CPPFLAGS="$CPPFLAGS -DH5_DEBUG_API" -else - echo "$as_me:$LINENO: result: no" >&5 + TRACE_API=yes + CPPFLAGS="$CPPFLAGS -DH5_DEBUG_API" + ;; + X-no|*) + echo "$as_me:$LINENO: result: no" >&5 echo "${ECHO_T}no" >&6 - TRACE_API=no - CPPFLAGS="$CPPFLAGS -UH5_DEBUG_API" -fi + TRACE_API=no + CPPFLAGS="$CPPFLAGS -UH5_DEBUG_API" + ;; +esac cat >confcache <<\_ACEOF # This file is a shell script that caches the results of configure @@ -31669,6 +31679,15 @@ case "X-$enable_production" in *) PRINT "$enable_production" ;; esac +PRINT_N " CFLAGS" +PRINT "$CFLAGS" + +PRINT_N " CPPFLAGS" +PRINT "$CPPFLAGS" + +PRINT_N " LDFLAGS" +PRINT "$LDFLAGS" + PRINT_N " Debug Mode" if test "X$DEBUG_PKG" = "X$all_packages"; then @@ -31689,11 +31708,7 @@ PRINT_N " Statically Linked Executables" IF_YES_NO "$STATIC_EXEC" PRINT_N " Tracing" -if test -z "$TRACE" -o "X$TRACE" = "Xyes"; then - PRINT "Yes" -else - PRINT "No" -fi +IF_YES_NO "$TRACE_API" PRINT "Languages:" diff --git a/configure.in b/configure.in index d2aaba7..2b4483d 100644 --- a/configure.in +++ b/configure.in @@ -1427,24 +1427,36 @@ fi dnl ---------------------------------------------------------------------- dnl Enable tracing of the API +dnl This must come after the enable-debug since it depends on debug. dnl -AC_MSG_CHECKING([for API tracing]) +AC_MSG_CHECKING(for API tracing); AC_ARG_ENABLE([trace], - [AC_HELP_STRING([--disable-trace], - [Disable API tracing capability])], - [TRACE=$enableval]) - -AC_SUBST([TRACE_API]) - -if test -z "$TRACE" -o "X$TRACE" = "Xyes"; then - AC_MSG_RESULT([yes]) - TRACE_API=yes - CPPFLAGS="$CPPFLAGS -DH5_DEBUG_API" -else - AC_MSG_RESULT([no]) - TRACE_API=no - CPPFLAGS="$CPPFLAGS -UH5_DEBUG_API" + [AC_HELP_STRING([--enable-trace], + [Enable API tracing capability. + Default=no if debug is disabled.])], + TRACE=$enableval) + +AC_SUBST(TRACE_API) +dnl Default to no if debug is disabled +if test "X-$TRACE" = X- ; then + if test -z "$DEBUG_PKG" ; then + TRACE=no + else + TRACE=yes + fi fi +case "X-$TRACE" in + X-yes) + AC_MSG_RESULT(yes) + TRACE_API=yes + CPPFLAGS="$CPPFLAGS -DH5_DEBUG_API" + ;; + X-no|*) + AC_MSG_RESULT(no) + TRACE_API=no + CPPFLAGS="$CPPFLAGS -UH5_DEBUG_API" + ;; +esac dnl Checkpoint the cache AC_CACHE_SAVE @@ -2023,6 +2035,15 @@ case "X-$enable_production" in *) PRINT "$enable_production" ;; esac +PRINT_N " CFLAGS" +PRINT "$CFLAGS" + +PRINT_N " CPPFLAGS" +PRINT "$CPPFLAGS" + +PRINT_N " LDFLAGS" +PRINT "$LDFLAGS" + PRINT_N " Debug Mode" if test "X$DEBUG_PKG" = "X$all_packages"; then @@ -2043,11 +2064,7 @@ PRINT_N " Statically Linked Executables" IF_YES_NO "$STATIC_EXEC" PRINT_N " Tracing" -if test -z "$TRACE" -o "X$TRACE" = "Xyes"; then - PRINT "Yes" -else - PRINT "No" -fi +IF_YES_NO "$TRACE_API" PRINT "Languages:" -- cgit v0.12