Hi Bill:

I just discovered you did not include the regression fixup below in
cmake-2.8.1.  I spent a fair amount of effort making a simple test case at
your request, verifying that Brad's one-liner patch worked, etc.  I assume
you didn't include it due to an oversight or because the issue was
discovered and fixed too close to release, but could you please make sure
this regression fix is included in the next CMake release?

Alan

---------- Forwarded message ----------
Date: Mon, 15 Mar 2010 09:07:17 -0400
From: Brad King <[email protected]>
To: Alan W. Irwin <[email protected]>
Cc: Bill Hoffman <[email protected]>, CMake <[email protected]>
Subject: [PATCH] Support multiple arguments in CC,CXX,FC values

Teach compiler identification to support values such as

  export CC='gcc -g -O2'

by separating the arguments on spaces.  We already do this for the
values of CFLAGS, CXXFLAGS, and FFLAGS.
[...]
---

 Modules/CMakeDetermineCompilerId.cmake |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/Modules/CMakeDetermineCompilerId.cmake 
b/Modules/CMakeDetermineCompilerId.cmake
index bddd6a1..4a800a8 100644
--- a/Modules/CMakeDetermineCompilerId.cmake
+++ b/Modules/CMakeDetermineCompilerId.cmake
@@ -19,6 +19,7 @@
 FUNCTION(CMAKE_DETERMINE_COMPILER_ID lang flagvar src)
   # Make sure the compiler arguments are clean.
   STRING(STRIP "${CMAKE_${lang}_COMPILER_ARG1}" CMAKE_${lang}_COMPILER_ID_ARG1)
+  STRING(REGEX REPLACE " +" ";" CMAKE_${lang}_COMPILER_ID_ARG1 
"${CMAKE_${lang}_COMPILER_ID_ARG1}")

   # Make sure user-specified compiler flags are used.
   IF(CMAKE_${lang}_FLAGS)
--
1.6.6.1
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://www.cmake.org/mailman/listinfo/cmake

Reply via email to