find_path: Fix location of <dir/header.h> in a framework on OS X

After finding it in `foo.Framework/Headers/dir/header.h`, we should
report the `foo.Framework/Headers` directory, not
`foo.Framework/Headers/dir`, because the former is what actually
contains the path the caller wishes to include.
This commit is contained in:
Harry Mallon
2016-06-07 09:04:35 -04:00
committed by Brad King
parent eeac8430bf
commit 188baef00c
5 changed files with 9 additions and 1 deletions
+1 -1
View File
@@ -117,7 +117,7 @@ std::string cmFindPathCommand::FindHeaderInFramework(std::string const& file,
if (this->IncludeFileInPath) {
return fheader;
}
fheader = cmSystemTools::GetFilenamePath(fheader);
fheader.resize(fheader.size() - file.size());
return fheader;
}
return "";
@@ -0,0 +1 @@
-- SOME_INCLUDE_DIR='[^']*Tests/RunCMake/find_path/Frameworks/Foo.framework/Headers'
@@ -0,0 +1,3 @@
set(CMAKE_FRAMEWORK_PATH "${CMAKE_CURRENT_SOURCE_DIR}/Frameworks")
find_path(SOME_INCLUDE_DIR "Some/Dir/Header.h")
message(STATUS "SOME_INCLUDE_DIR='${SOME_INCLUDE_DIR}'")
@@ -3,3 +3,7 @@ include(RunCMake)
if(WIN32 OR CYGWIN)
run_cmake(PrefixInPATH)
endif()
if(APPLE)
run_cmake(FrameworksWithSubdirs)
endif()