[Gnash-commit] gnash ChangeLog configure.ac

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/06/06 07:50:13

Modified files:
        .              : ChangeLog configure.ac 

Log message:
                * configure.ac: don't set CXXFLAGS as a side-effect of the 
SIZET_FMT
                  tests if not already set by user.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.3464&r2=1.3465
http://cvs.savannah.gnu.org/viewcvs/gnash/configure.ac?cvsroot=gnash&r1=1.332&r2=1.333

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.3464
retrieving revision 1.3465
diff -u -b -r1.3464 -r1.3465
--- ChangeLog   6 Jun 2007 06:58:18 -0000       1.3464
+++ ChangeLog   6 Jun 2007 07:50:12 -0000       1.3465
@@ -1,5 +1,7 @@
 2007-06-06 Sandro Santilli <address@hidden>
 
+       * configure.ac: don't set CXXFLAGS as a side-effect of the SIZET_FMT
+         tests if not already set by user.
        * Makefile.am: Add print of CXXFLAGS at 'make dumpconfig' time.
 
 2007-06-05 Tomas Groth Christensen <address@hidden>

Index: configure.ac
===================================================================
RCS file: /sources/gnash/gnash/configure.ac,v
retrieving revision 1.332
retrieving revision 1.333
diff -u -b -r1.332 -r1.333
--- configure.ac        5 Jun 2007 13:56:27 -0000       1.332
+++ configure.ac        6 Jun 2007 07:50:13 -0000       1.333
@@ -15,7 +15,7 @@
 dnl  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 dnl  
 
-dnl $Id: configure.ac,v 1.332 2007/06/05 13:56:27 strk Exp $
+dnl $Id: configure.ac,v 1.333 2007/06/06 07:50:13 strk Exp $
 
 AC_PREREQ(2.50)
 AC_INIT(gnash, cvs)
@@ -63,6 +63,10 @@
 
 dnl AC_DEFINE(SIZET_FMT, ["%z"], [size_t printf modifier is z])
 
+dnl NOTE: Using AC_TRY_COMPILE is bogus when cross-compiling !
+dnl       We need to know the size of size_t in the *target* host, 
+dnl       not the build host (right?)
+dnl
 AC_LANG_PUSH(C++)
 save_CXXFLAGS="$CXXFLAGS"
 CXXFLAGS="$CXXFLAGS -Werror -Wformat"
@@ -92,7 +96,11 @@
   AC_DEFINE(SIZET_FMT, ["%ld"], [size_t printf modifier is ld])
 )
 
-CXXFLAGS="$save_CXXFLAGS"
+if test x"$ac_env_CXXFLAGS_set" = "xset"; then
+       CXXFLAGS="$save_CXXFLAGS"
+else
+       unset CXXFLAGS; 
+fi
 AC_LANG_POP(C++)
 
 dnl This is primarily used when compiling for a similar architecture,