Am Sonntag, 30. Juli 2006 15:56 schrieb Lars Gullik Bjønnes:
> Georg Baum <[EMAIL PROTECTED]> writes:
> 
> | Why? CPPFLAGS should always be set IMO if the user did not set them.
> 
> No. If _you_ play with CXXFLAGS, then _you_ are in charge and we bow
> to your decisions.

Of course.

> Hmm... but you don't play with CPPFLAGS... should possibly be an
> additional check for CPPFLAGS then.

Exactly. The attached patch does exactly that. I am going to commit this 
soon.

> btw. Why do you play with CXXFLAGS anyway?

-fmessage-length=0. The default for gcc is 0, but for g++ 72. g++ error 
messages tend to be unreadable anyway for code with templates, and this 
does not get better if g++ tries to be smart and wraps them to 72 
characters (my terminals are usually rather wide).


Georg
Index: config/lyxinclude.m4
===================================================================
--- config/lyxinclude.m4	(Revision 14515)
+++ config/lyxinclude.m4	(Arbeitskopie)
@@ -243,6 +243,8 @@ if test x$GXX = xyes; then
         CXXFLAGS="-pg $CXXFLAGS"
         LDFLAGS="-pg $LDFLAGS"
     fi
+  fi
+  if test "$ac_env_CPPFLAGS_set" != set; then
     if test x$enable_warnings = xyes ; then
         case $gxx_version in
             3.1*|3.2*|3.3*)

Reply via email to