Allow building against an external NanoGUI installation (#525)

autodesk-forks · Nov 19, 2020 · 76a30db · 76a30db
1 parent c4a6c7f
commit 76a30db
Showing 1 changed file with 56 additions and 31 deletions.
diff --git a/source/MaterialXView/CMakeLists.txt b/source/MaterialXView/CMakeLists.txt
@@ -1,36 +1,62 @@
-if (NOT IS_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/NanoGUI/ext/glfw/src")
-    message(FATAL_ERROR "Building the MaterialX viewer requires the NanoGUI submodule "
-        "to be present. Update your repository by calling the following:\n"
-        "git submodule update --init --recursive")
-endif()
+option(MATERIALX_NANOGUI_EXTERNAL "Build aginst an external install of NanoGUI (NANOGUI_ROOT may also need to be set)" OFF)
 
-file(GLOB materialx_source "${CMAKE_CURRENT_SOURCE_DIR}/*.cpp")
-file(GLOB materialx_headers "${CMAKE_CURRENT_SOURCE_DIR}/*.h*")
+if (MATERIALX_NANOGUI_EXTERNAL)
+    find_path(NANOGUI_INCLUDE_DIRS
+        NAMES
+            nanogui/nanogui.h
+        HINTS
+            "${NANOGUI_ROOT}/include"
+            "$ENV{NANOGUI_ROOT}/include")
+    find_library(NANOGUI_LIBRARIES
+        NAMES
+            nanogui
+        HINTS
+            "${NANOGUI_ROOT}/lib"
+            "$ENV{NANOGUI_ROOT}/lib"
+            "${NANOGUI_ROOT}/lib64"
+            "$ENV{NANOGUI_ROOT}/lib64")
 
-set(NANOGUI_BUILD_EXAMPLE OFF CACHE BOOL " " FORCE)
-set(NANOGUI_BUILD_SHARED OFF CACHE BOOL " " FORCE)
-set(NANOGUI_BUILD_PYTHON OFF CACHE BOOL " " FORCE)
-set(NANOGUI_INSTALL OFF CACHE BOOL " " FORCE)
+    if (NOT NANOGUI_INCLUDE_DIRS OR NOT NANOGUI_LIBRARIES)
+        message(FATAL_ERROR "Could not find external NanoGUI installation, is NANOGUI_ROOT set?")
+    endif()
+else()
+    if (NOT IS_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/NanoGUI/ext/glfw/src")
+        message(FATAL_ERROR "Building the MaterialX viewer requires the NanoGUI submodule "
+            "to be present. Update your repository by calling the following:\n"
+            "git submodule update --init --recursive")
+    endif()
 
-# Locally disable additional warnings for NanoGUI and its dependencies
-set(PREV_CMAKE_C_FLAGS ${CMAKE_C_FLAGS})
-set(PREV_CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS})
-if(MSVC)
-    add_compile_options(-wd4389 -D_CRT_SECURE_NO_WARNINGS -D_SCL_SECURE_NO_WARNINGS -DEIGEN_DONT_VECTORIZE)
-elseif(APPLE)
-    add_compile_options(-Wno-objc-multiple-method-names -DGL_SILENCE_DEPRECATION)
-elseif(UNIX AND CMAKE_CXX_COMPILER_ID MATCHES "GNU")
-    add_compile_options(-Wno-format-truncation -Wno-implicit-fallthrough -Wno-int-in-bool-context
-                        -Wno-maybe-uninitialized -Wno-misleading-indentation)
-    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-deprecated-copy")
-endif()
+    set(NANOGUI_BUILD_EXAMPLE OFF CACHE BOOL " " FORCE)
+    set(NANOGUI_BUILD_SHARED OFF CACHE BOOL " " FORCE)
+    set(NANOGUI_BUILD_PYTHON OFF CACHE BOOL " " FORCE)
+    set(NANOGUI_INSTALL OFF CACHE BOOL " " FORCE)
+
+    # Locally disable additional warnings for NanoGUI and its dependencies
+    set(PREV_CMAKE_C_FLAGS ${CMAKE_C_FLAGS})
+    set(PREV_CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS})
+    if(MSVC)
+        add_compile_options(-wd4389 -D_CRT_SECURE_NO_WARNINGS -D_SCL_SECURE_NO_WARNINGS -DEIGEN_DONT_VECTORIZE)
+    elseif(APPLE)
+        add_compile_options(-Wno-objc-multiple-method-names -DGL_SILENCE_DEPRECATION)
+    elseif(UNIX AND CMAKE_CXX_COMPILER_ID MATCHES "GNU")
+        add_compile_options(-Wno-format-truncation -Wno-implicit-fallthrough -Wno-int-in-bool-context
+                            -Wno-maybe-uninitialized -Wno-misleading-indentation)
+        set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-deprecated-copy")
+    endif()
 
-add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/NanoGUI external/NanoGUI)
-set_property(TARGET nanogui nanogui-obj glfw glfw_objects PROPERTY FOLDER "External")
+    add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/NanoGUI external/NanoGUI)
+    set_property(TARGET nanogui nanogui-obj glfw glfw_objects PROPERTY FOLDER "External")
 
-# Restore warnings for MaterialXView
-set(CMAKE_C_FLAGS ${PREV_CMAKE_C_FLAGS})
-set(CMAKE_CXX_FLAGS ${PREV_CMAKE_CXX_FLAGS})
+    set(NANOGUI_INCLUDE_DIRS "${CMAKE_CURRENT_SOURCE_DIR}/../;${CMAKE_CURRENT_SOURCE_DIR}/NanoGUI/include")
+    set(NANOGUI_LIBRARIES "nanogui")
+
+    # Restore warnings for MaterialXView
+    set(CMAKE_C_FLAGS ${PREV_CMAKE_C_FLAGS})
+    set(CMAKE_CXX_FLAGS ${PREV_CMAKE_CXX_FLAGS})
+endif()
+
+file(GLOB materialx_source "${CMAKE_CURRENT_SOURCE_DIR}/*.cpp")
+file(GLOB materialx_headers "${CMAKE_CURRENT_SOURCE_DIR}/*.h*")
 
 add_definitions(${NANOGUI_EXTRA_DEFS})
 
@@ -41,14 +67,13 @@ target_link_libraries(
     MaterialXFormat
     MaterialXGenGlsl
     MaterialXRenderGlsl
-    nanogui
+    ${NANOGUI_LIBRARIES}
     ${NANOGUI_EXTRA_LIBS})
 
 target_include_directories(
     MaterialXView
     PRIVATE
-    ${CMAKE_CURRENT_SOURCE_DIR}/../
-    ${CMAKE_CURRENT_SOURCE_DIR}/NanoGUI/include
+    ${NANOGUI_INCLUDE_DIRS}
     ${NANOGUI_EXTRA_INCS})
 
 if(MATERIALX_BUILD_OIIO AND OPENIMAGEIO_ROOT_DIR)