[svn-r16555] Purpose:

    Bug Fix

Description:

    Removing the code from configure which strips the '-g' flag from CFLAGS
    when in production mode. The current default CFLAGS in production mode
    does not include '-g', as intended, but we should allow users to
    override this and enable '-g' by setting the CFLAGS environment
    variable if desired. Note that this applies to FCFLAGS and CXXFLAGS
    as well.

Tested:

    kagiso, linew, liberty

3 files changed, 4 insertions, 65 deletions

diff --git a/configure b/configure
index 188bc6e..749d071 100755
--- a/configure
+++ b/configure
@@ -1,5 +1,5 @@
 #! /bin/sh
-# From configure.in Id: configure.in 16519 2009-02-26 22:25:57Z pvn .
+# From configure.in Id: configure.in 16526 2009-03-02 01:52:02Z hdftest .
 # Guess values for system-dependent variables and create Makefiles.
 # Generated by GNU Autoconf 2.61 for HDF5 1.8.2-post8.
 #
@@ -21618,37 +21618,6 @@ case "X-$enable_production" in
     enable_production="yes"
     { echo "$as_me:$LINENO: result: production" >&5
 echo "${ECHO_T}production" >&6; }
-
-            CFLAGS_temp=""
-    if test -n "$CFLAGS"; then
-      for d in $CFLAGS ; do
-        if test "X$d" != "X-g"; then
-          CFLAGS_temp="$CFLAGS_temp $d"
-        fi
-      done
-      CFLAGS=$CFLAGS_temp
-    fi
-
-    CXXFLAGS_temp=""
-    if test -n "$CXXFLAGS"; then
-      for d in $CXXFLAGS ; do
-        if test "X$d" != "X-g"; then
-          CXXFLAGS_temp="$CXXFLAGS_temp $d"
-        fi
-      done
-      CXXFLAGS=$CXXFLAGS_temp
-    fi
-
-    FCFLAGS_temp=""
-    if test -n "$FCFLAGS"; then
-      for d in $FCFLAGS ; do
-        if test "X$d" != "X-g"; then
-          FCFLAGS_temp="$FCFLAGS_temp $d"
-        fi
-      done
-      FCFLAGS=$FCFLAGS_temp
-    fi
-
     CONFIG_MODE=production
     H5_CFLAGS="$H5_CFLAGS $PROD_CFLAGS"
     H5_CPPFLAGS="$H5_CPPFLAGS $PROD_CPPFLAGS"
diff --git a/configure.in b/configure.in
index 75d74f0..6d72e85 100644
--- a/configure.in
+++ b/configure.in
@@ -1142,39 +1142,6 @@ case "X-$enable_production" in
   X-|X-yes)
     enable_production="yes"
     AC_MSG_RESULT([production])
-
-    dnl Remove the "-g" flag from CFLAGS if it's in there.
-    dnl
-    CFLAGS_temp=""
-    if test -n "$CFLAGS"; then
-      for d in $CFLAGS ; do
-        if test "X$d" != "X-g"; then
-          CFLAGS_temp="$CFLAGS_temp $d"
-        fi
-      done
-      CFLAGS=$CFLAGS_temp
-    fi
-
-    CXXFLAGS_temp=""
-    if test -n "$CXXFLAGS"; then
-      for d in $CXXFLAGS ; do
-        if test "X$d" != "X-g"; then
-          CXXFLAGS_temp="$CXXFLAGS_temp $d"
-        fi
-      done
-      CXXFLAGS=$CXXFLAGS_temp
-    fi
-
-    FCFLAGS_temp=""
-    if test -n "$FCFLAGS"; then
-      for d in $FCFLAGS ; do
-        if test "X$d" != "X-g"; then
-          FCFLAGS_temp="$FCFLAGS_temp $d"
-        fi
-      done
-      FCFLAGS=$FCFLAGS_temp
-    fi
-
     CONFIG_MODE=production
     H5_CFLAGS="$H5_CFLAGS $PROD_CFLAGS"
     H5_CPPFLAGS="$H5_CPPFLAGS $PROD_CPPFLAGS"
diff --git a/release_docs/RELEASE.txt b/release_docs/RELEASE.txt
index 448cae4..89e6dae 100644
--- a/release_docs/RELEASE.txt
+++ b/release_docs/RELEASE.txt
@@ -111,6 +111,9 @@ Bug Fixes since HDF5-1.8.2
 
     Configuration
     -------------
+    - Configure no longer removes the '-g' flag from CFLAGS when in production
+      mode if it has been explicitly set in the CFLAGS environment variable 
+      prior to configuration. MAM - 2009/03/09 - BZ #1401
 
     Library
     -------