[Cmake-commits] CMake branch, next, updated. v2.8.7-3441-g74df567

Brad King brad.king at kitware.com
Mon Apr 9 16:28:43 EDT 2012


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "CMake".

The branch, next has been updated
       via  74df56764f2ec779745e4e8945c10ccdad79b399 (commit)
       via  5adb6b7a34fc67ed49d8f70c1b5b57ca808ef2c4 (commit)
      from  4b7a92a3bf30b7bc866fa18f042167ac8fb89077 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=74df56764f2ec779745e4e8945c10ccdad79b399
commit 74df56764f2ec779745e4e8945c10ccdad79b399
Merge: 4b7a92a 5adb6b7
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Mon Apr 9 16:28:41 2012 -0400
Commit:     CMake Topic Stage <kwrobot at kitware.com>
CommitDate: Mon Apr 9 16:28:41 2012 -0400

    Merge topic 'check_include_files_output' into next
    
    5adb6b7 CheckIncludeFiles: Shorten check description message


http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=5adb6b7a34fc67ed49d8f70c1b5b57ca808ef2c4
commit 5adb6b7a34fc67ed49d8f70c1b5b57ca808ef2c4
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Mon Apr 9 15:57:02 2012 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Mon Apr 9 15:57:02 2012 -0400

    CheckIncludeFiles: Shorten check description message
    
    Since commit 44d007b6 (CheckIncludeFiles: fix status output, 2012-02-01)
    check_include_files reports the list of files tested instead of the name
    of the variable storing the result.  Some projects incrementally test
    and concatenate very long lists leading to long messages that do not
    provide much information.  Users report confusion especially when the
    lines wrap.
    
    For lists of more than two files produce messages of the format
    
     Looing for N include files first.h, ..., last.h
    
    where N is the list length and "..." is literal.  Leave the log file
    entries and cache entry description unchanged as they should have the
    full detail of the check performed.

diff --git a/Modules/CheckIncludeFiles.cmake b/Modules/CheckIncludeFiles.cmake
index 642d962..a94644e 100644
--- a/Modules/CheckIncludeFiles.cmake
+++ b/Modules/CheckIncludeFiles.cmake
@@ -13,7 +13,7 @@
 #  CMAKE_REQUIRED_INCLUDES = list of include directories
 
 #=============================================================================
-# Copyright 2003-2009 Kitware, Inc.
+# Copyright 2003-2012 Kitware, Inc.
 #
 # Distributed under the OSI-approved BSD License (the "License");
 # see accompanying file Copyright.txt for details.
@@ -44,7 +44,17 @@ MACRO(CHECK_INCLUDE_FILES INCLUDE VARIABLE)
     CONFIGURE_FILE("${CMAKE_ROOT}/Modules/CMakeConfigurableFile.in"
       "${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/CMakeTmp/CheckIncludeFiles.c" @ONLY IMMEDIATE)
 
-    MESSAGE(STATUS "Looking for include files ${INCLUDE}")
+    SET(_INCLUDE ${INCLUDE}) # remove empty elements
+    IF("${_INCLUDE}" MATCHES "^([^;]+);.+;([^;]+)$")
+      LIST(LENGTH _INCLUDE _INCLUDE_LEN)
+      SET(_description "${_INCLUDE_LEN} include files ${CMAKE_MATCH_1}, ..., ${CMAKE_MATCH_2}")
+    ELSEIF("${_INCLUDE}" MATCHES "^([^;]+);([^;]+)$")
+      SET(_description "include files ${CMAKE_MATCH_1}, ${CMAKE_MATCH_2}")
+    ELSE()
+      SET(_description "include file ${_INCLUDE}")
+    ENDIF()
+
+    MESSAGE(STATUS "Looking for ${_description}")
     TRY_COMPILE(${VARIABLE}
       ${CMAKE_BINARY_DIR}
       ${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/CMakeTmp/CheckIncludeFiles.c
@@ -54,14 +64,14 @@ MACRO(CHECK_INCLUDE_FILES INCLUDE VARIABLE)
       "${CHECK_INCLUDE_FILES_INCLUDE_DIRS}"
       OUTPUT_VARIABLE OUTPUT)
     IF(${VARIABLE})
-      MESSAGE(STATUS "Looking for include files ${INCLUDE} - found")
+      MESSAGE(STATUS "Looking for ${_description} - found")
       SET(${VARIABLE} 1 CACHE INTERNAL "Have include ${INCLUDE}")
       FILE(APPEND ${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/CMakeOutput.log 
         "Determining if files ${INCLUDE} "
         "exist passed with the following output:\n"
         "${OUTPUT}\n\n")
     ELSE(${VARIABLE})
-      MESSAGE(STATUS "Looking for include files ${INCLUDE} - not found.")
+      MESSAGE(STATUS "Looking for ${_description} - not found.")
       SET(${VARIABLE} "" CACHE INTERNAL "Have includes ${INCLUDE}")
       FILE(APPEND ${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/CMakeError.log 
         "Determining if files ${INCLUDE} "

-----------------------------------------------------------------------

Summary of changes:
 Modules/CheckIncludeFiles.cmake |   18 ++++++++++++++----
 1 files changed, 14 insertions(+), 4 deletions(-)


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list