|
From: Dominik V. <dom...@gm...> - 2003-06-10 14:37:10
Attachments:
valgrind-1.9.6_cflags.patch
|
The Makefile.am's in 1.9.6 override thew user settings for CFLAGS and CXXFLAGS and use the deprecated INCLUDES automake directive. Instead they should use AM_CFLAGS, AM_CXXFLAGS and AM_CPPFLAGS (patch attached). With the patch you say $ make CFLAGS="-O2" without later compile errors. Bye Dominik ^_^ ^_^ |
|
From: Dominik V. <dom...@gm...> - 2003-06-10 14:40:02
|
... and the fixed ChengeLog entry: 2003-06-10 Dominik Vogt <dom...@gm...> * addrcheck/Makefile.am: * auxprogs/Makefile.am: * cachegrind/Makefile.am: * cachegrind/tests/Makefile.am: * corecheck/Makefile.am: * corecheck/tests/Makefile.am: * coregrind/Makefile.am: * coregrind/demangle/Makefile.am: * helgrind/Makefile.am: * lackey/Makefile.am: * memcheck/Makefile.am: * memcheck/tests/Makefile.am: * none/Makefile.am: * none/tests/Makefile.am: * tests/Makefile.am: use AM_CPPFLAGS, AM_CFLAGS and AM_CXXFLAGS instead of INCLUDES, CFLAGS and CXXFLAGS to allow specifying them as environment variables a la 'make CFLAGS=-O2'. moved -DVG_LIBDIR definition from CFLAGS to AM_CPPFLAGS Bye Dominik ^_^ ^_^ |
|
From: Nicholas N. <nj...@ca...> - 2003-06-29 17:23:19
|
On Tue, 10 Jun 2003, Dominik Vogt wrote: > The Makefile.am's in 1.9.6 override thew user settings for CFLAGS > and CXXFLAGS and use the deprecated INCLUDES automake directive. > Instead they should use AM_CFLAGS, AM_CXXFLAGS and AM_CPPFLAGS > (patch attached). With the patch you say > > $ make CFLAGS="-O2" > > without later compile errors. Oh, I fixed this a while ago using your patch. Thanks for pointing out the problem. N |