Skip to content

Commit

Permalink
Merge pull request #7 from v4hn/wrong_foundation_check
Browse files Browse the repository at this point in the history
Poco detection is too confident when seeing /usr/include/Foundation
  • Loading branch information
Mirza Shah committed May 22, 2013
2 parents 547a4c3 + 0fe94c4 commit 00b6930
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions cmake/FindPoco.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ SET(SUFFIX_FOR_LIBRARY_PATH
#
# Look for an installation.
#
FIND_PATH(Poco_INCLUDE_DIR NAMES Foundation/include/Poco/AbstractCache.h PATH_SUFFIXES ${SUFFIX_FOR_INCLUDE_PATH} PATHS
FIND_PATH(Poco_INCLUDE_DIR NAMES Foundation/include/Poco/SharedLibrary.h PATH_SUFFIXES ${SUFFIX_FOR_INCLUDE_PATH} PATHS

# Look in other places.
${POCO_DIR_SEARCH}
Expand All @@ -121,7 +121,7 @@ SET(Poco_FOUND 0)

# Now try to get the include and library path.
IF(Poco_INCLUDE_DIR)
IF(EXISTS "${Poco_INCLUDE_DIR}/Foundation")
IF(EXISTS "${Poco_INCLUDE_DIR}/Foundation/include/Poco/SharedLibrary.h")
SET(Poco_INCLUDE_DIRS
${Poco_INCLUDE_DIR}/CppUnit/include
${Poco_INCLUDE_DIR}/Foundation/include
Expand Down
4 changes: 2 additions & 2 deletions cmake/PocoConfig.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ SET(SUFFIX_FOR_LIBRARY_PATH
#
# Look for an installation.
#
FIND_PATH(Poco_INCLUDE_DIR NAMES Foundation/include/Poco/AbstractCache.h PATH_SUFFIXES ${SUFFIX_FOR_INCLUDE_PATH} PATHS
FIND_PATH(Poco_INCLUDE_DIR NAMES Foundation/include/Poco/SharedLibrary.h PATH_SUFFIXES ${SUFFIX_FOR_INCLUDE_PATH} PATHS

# Look in other places.
${POCO_DIR_SEARCH}
Expand All @@ -121,7 +121,7 @@ SET(Poco_FOUND 0)

# Now try to get the include and library path.
IF(Poco_INCLUDE_DIR)
IF(EXISTS "${Poco_INCLUDE_DIR}/Foundation")
IF(EXISTS "${Poco_INCLUDE_DIR}/Foundation/include/Poco/SharedLibrary.h")
SET(Poco_INCLUDE_DIRS
${Poco_INCLUDE_DIR}/CppUnit/include
${Poco_INCLUDE_DIR}/Foundation/include
Expand Down

0 comments on commit 00b6930

Please sign in to comment.