diff options
| author | Mike McGreevy <mamcgree@hdfgroup.org> | 2009-03-09 17:50:53 (GMT) |
|---|---|---|
| committer | Mike McGreevy <mamcgree@hdfgroup.org> | 2009-03-09 17:50:53 (GMT) |
| commit | da5058310c324dcce93dc9328ef2bd53bf1fed02 (patch) | |
| tree | fb079da5b113904f61c922ed02f59fe84e8e9d9f | |
| parent | 793503a3cd26b3435ee27b092d9fe7f2cb93ce29 (diff) | |
| download | hdf5-da5058310c324dcce93dc9328ef2bd53bf1fed02.zip hdf5-da5058310c324dcce93dc9328ef2bd53bf1fed02.tar.gz hdf5-da5058310c324dcce93dc9328ef2bd53bf1fed02.tar.bz2 | |
[svn-r16554] 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
| -rwxr-xr-x | configure | 33 | ||||
| -rw-r--r-- | configure.in | 33 | ||||
| -rw-r--r-- | release_docs/RELEASE.txt | 3 |
3 files changed, 4 insertions, 65 deletions
@@ -1,5 +1,5 @@ #! /bin/sh -# From configure.in Id: configure.in 16525 2009-03-01 15:35:48Z hdftest . +# From configure.in Id: configure.in 16553 2009-03-08 12:53:10Z lrknox . # Guess values for system-dependent variables and create Makefiles. # Generated by GNU Autoconf 2.61 for HDF5 1.9.35. # @@ -21620,37 +21620,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 32d9d59..229209d 100644 --- a/configure.in +++ b/configure.in @@ -1142,39 +1142,6 @@ case "X-$enable_production" in 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 1190df9..06a8108 100644 --- a/release_docs/RELEASE.txt +++ b/release_docs/RELEASE.txt @@ -230,6 +230,9 @@ Bug Fixes since HDF5-1.8.0 release 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. - Fixed error with 'make check install' failing due to h5dump needing other tools built first. MAM - 2008/10/24. - When using shared szip, it is no longer necessary to specify |