Only use Boost_INCLUDE_DIR if Boost was found. Prevents errors like:

"CMake Error: The following variables are used in this project, but they are set to NOTFOUND." ..meaning Boost was less optional than it should be. This didn't seem to happen on Linux, only on OS X
dbr · Jan 15, 2011 · 9804eb0 · 9804eb0
1 parent 5733569
commit 9804eb0
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/src/core/CMakeLists.txt b/src/core/CMakeLists.txt
@@ -6,9 +6,13 @@ include_directories(
     ${CMAKE_SOURCE_DIR}/export/
     ${CMAKE_BINARY_DIR}/export/
     ${YAML_CPP_INCLUDE_DIRS}
-    ${Boost_INCLUDE_DIR}
 )
 
+if(Boost_FOUND)
+    include_directories(
+        ${Boost_INCLUDE_DIR})
+endif()
+
 file(GLOB_RECURSE core_src_files "${CMAKE_SOURCE_DIR}/src/core/*.cpp")
 file(GLOB_RECURSE core_export_headers "${CMAKE_SOURCE_DIR}/export/OpenColorIO/*.h")
 

diff --git a/src/pyglue/CMakeLists.txt b/src/pyglue/CMakeLists.txt
@@ -25,8 +25,12 @@ if(PYTHON_OK)
         ${PYTHON_INCLUDE}
         ${CMAKE_SOURCE_DIR}/export/
         ${CMAKE_BINARY_DIR}/export/
-        ${Boost_INCLUDE_DIR}
     )
+
+    if(Boost_FOUND)
+        include_directories(
+            ${Boost_INCLUDE_DIR})
+    endif()
 
     # Exclude the 'lib' prefix from the name.
     if(NOT PYTHON_INCLUDE_LIB_PREFIX)

diff --git a/src/testbed/CMakeLists.txt b/src/testbed/CMakeLists.txt
@@ -2,9 +2,13 @@
 include_directories(
     ${CMAKE_SOURCE_DIR}/export/
     ${CMAKE_BINARY_DIR}/export/
-    ${Boost_INCLUDE_DIR}
 )
 
+if(Boost_FOUND)
+    include_directories(
+        ${Boost_INCLUDE_DIR})
+endif()
+
 add_executable(OpenColorIOTest
     main.cpp
 )