On 07/13/2011 01:26 PM, Marcin Baczyński wrote:
---

  Without this patch CFLAGS=-fstrict-aliasing ./autogen.sh ... has no effect,
  as the flag is followed by -fno-strict-aliasing added by configure.

  configure.ac |    8 ++++++++
  1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/configure.ac b/configure.ac
index eace790..ef9f4b2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -17,6 +17,10 @@ AC_INIT([Mesa],[mesa_version],
  AC_CONFIG_AUX_DIR([bin])
  AC_CANONICAL_HOST

+dnl Save user CFLAGS and CXXFLAGS so one can override the default ones
+USER_CFLAGS="$CFLAGS"
+USER_CXXFLAGS="$CXXFLAGS"
+
  dnl Versions for external dependencies
  LIBDRM_REQUIRED=2.4.24
  LIBDRM_RADEON_REQUIRED=2.4.24
@@ -1906,6 +1910,10 @@ dnl Restore LDFLAGS and CPPFLAGS
  LDFLAGS="$_SAVE_LDFLAGS"
  CPPFLAGS="$_SAVE_CPPFLAGS"

+dnl Add user CFLAGS and CXXFLAGS
+CFLAGS="$CFLAGS $USER_CFLAGS"
+CXXFLAGS="$CXXFLAGS $USER_CXXFLAGS"
+
  dnl Substitute the config
  AC_CONFIG_FILES([configs/autoconf])


Looks OK to me, but an autoconf expert should probably verify that this is the right fix. Anyone???

-Brian

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to