[Cmake-commits] CMake branch, next, updated. v2.8.11.2-4453-gd96c6eb

Brad King brad.king at kitware.com
Mon Oct 7 08:25:45 EDT 2013


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  d96c6ebda35baa67dfcb0dd4d79d7e4d947cf4db (commit)
       via  6f98f4ac92a789a310cc7d9965556b4a8439654f (commit)
      from  85b897794bc87eb008bfc6dd30d3642ec70a07f8 (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=d96c6ebda35baa67dfcb0dd4d79d7e4d947cf4db
commit d96c6ebda35baa67dfcb0dd4d79d7e4d947cf4db
Merge: 85b8977 6f98f4a
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Mon Oct 7 08:25:32 2013 -0400
Commit:     CMake Topic Stage <kwrobot at kitware.com>
CommitDate: Mon Oct 7 08:25:32 2013 -0400

    Merge topic 'fix-install-include-dirs-processing' into next
    
    6f98f4a Genex: Fix processing multiple include directories for relative paths


http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=6f98f4ac92a789a310cc7d9965556b4a8439654f
commit 6f98f4ac92a789a310cc7d9965556b4a8439654f
Author:     Stephen Kelly <steveire at gmail.com>
AuthorDate: Mon Oct 7 13:24:41 2013 +0200
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Mon Oct 7 08:17:12 2013 -0400

    Genex: Fix processing multiple include directories for relative paths
    
    Re-insert the semicolon which was removed during splitting.
    
    Commit d777b8e7 (Genex: Allow relative paths in INSTALL_INTERFACE.,
    2013-07-25) introduced the prefixItems method to allow relative paths
    in the argument of the INSTALL_INTERFACE expression. That method was
    buggy in that it did not re-introduce the semicolon separator in
    the result.
    
    This bug also affects paths which are already absolute in user code.

diff --git a/Source/cmGeneratorExpression.cxx b/Source/cmGeneratorExpression.cxx
index 127cf6b..d73c72c 100644
--- a/Source/cmGeneratorExpression.cxx
+++ b/Source/cmGeneratorExpression.cxx
@@ -238,9 +238,12 @@ static void prefixItems(const std::string &content, std::string &result,
 {
   std::vector<std::string> entries;
   cmGeneratorExpression::Split(content, entries);
+  const char *sep = "";
   for(std::vector<std::string>::const_iterator ei = entries.begin();
       ei != entries.end(); ++ei)
     {
+    result += sep;
+    sep = ";";
     if (!cmSystemTools::FileIsFullPath(ei->c_str())
         && cmGeneratorExpression::Find(*ei) == std::string::npos)
       {
diff --git a/Tests/ExportImport/Export/CMakeLists.txt b/Tests/ExportImport/Export/CMakeLists.txt
index 1f23b2a..72ae78f 100644
--- a/Tests/ExportImport/Export/CMakeLists.txt
+++ b/Tests/ExportImport/Export/CMakeLists.txt
@@ -159,7 +159,7 @@ set_property(TARGET testLibRequired APPEND PROPERTY
     $<BUILD_INTERFACE:$<TARGET_PROPERTY:testLibIncludeRequired5,INTERFACE_INCLUDE_DIRECTORIES>>
     # Test that the below is non-fatal
     $<$<STREQUAL:one,two>:$<TARGET_PROPERTY:not_a_target,INTERFACE_INCLUDE_DIRECTORIES>>
-    $<INSTALL_INTERFACE:include/testLibIncludeRequired7>
+    $<INSTALL_INTERFACE:include/testLibIncludeRequired7;include/testLibIncludeRequired4>
 )
 
 set_property(TARGET testLibRequired APPEND PROPERTY

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

Summary of changes:


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list