[Pacemaker] [patch] low: unset CFLAGS in a dash-friendly way

Simon Horman horms at verge.net.au
Mon Jul 5 08:19:10 UTC 2010


# HG changeset patch
# User Simon Horman <horms at verge.net.au>
# Date 1278317844 -32400
# Node ID e823bf55e0d875bcd9dc1668f24547345b9bdf81
# Parent  7499efa9c2a3f8c5c6396dc6c3631e9f3898547e
low: unset CFLAGS in a dash-friendly way

According to the dash man page, the only way to unexport a variable
is to unset it. This also works in bash.

For the record, the reason that CFLAGS is exported from the environment to
configure and configure appends -Werror to the CFLAGS then it will be
re-exported by configure and has been observed to cause trouble when
configure runs a separate configure is the libltdl directory.

See:	Debian Bug #582874 - http://bugs.debian.org/582874
	Debian Bug #582875 - http://bugs.debian.org/582875

Signed-off-by: Simon Horman <horms at verge.net.au>

diff -r 7499efa9c2a3 -r e823bf55e0d8 configure.in
--- a/configure.in	Wed Jan 07 15:02:59 2009 +0100
+++ b/configure.in	Mon Jul 05 17:17:24 2010 +0900
@@ -1475,11 +1475,12 @@
 dnl no effect. However if CFLAGS was exported from the user's
 dnl environment, then the new CFLAGS will also be exported
 dnl to sub processes. This causes a problem when configure
-dnl is run in the libltdl directory. Horms 16th July 2002
-dnl
-
+dnl is run in the libltdl directory.
 if export | fgrep " CFLAGS=" > /dev/null; then
-	export -n CFLAGS || true # We don't want to bomb out if this fails
+	SAVED_CFLAGS="$CFLAGS"
+	unset CFLAGS
+	CFLAGS="$SAVED_CFLAGS"
+	unset SAVED_CFLAGS
 fi
 
 if test "$GCC" = yes; then




More information about the Pacemaker mailing list