diff --git a/cmake/FindOpenColorIO.cmake b/cmake/FindOpenColorIO.cmake index 26b8229a1..db2b6ecd8 100644 --- a/cmake/FindOpenColorIO.cmake +++ b/cmake/FindOpenColorIO.cmake @@ -1,5 +1,6 @@ # # Copyright 2019 Pixar +# Modifications: Copyright (C) 2020 Olive Team # # Licensed under the Apache License, Version 2.0 (the "Apache License") # with the following modification; you may not use this file except in @@ -22,44 +23,14 @@ # language governing permissions and limitations under the Apache License. # -if(UNIX) - find_path(OCIO_BASE_DIR - include/OpenColorIO/OpenColorABI.h - HINTS - "${OCIO_LOCATION}" - "$ENV{OCIO_LOCATION}" - "/opt/ocio" - ) - find_path(OCIO_LIBRARY_DIR - libOpenColorIO.so - HINTS - "${OCIO_LOCATION}" - "$ENV{OCIO_LOCATION}" - "${OCIO_BASE_DIR}" - PATH_SUFFIXES - lib/ - DOC - "OpenColorIO library path" - ) -elseif(WIN32) - find_path(OCIO_BASE_DIR - include/OpenColorIO/OpenColorABI.h - HINTS - "${OCIO_LOCATION}" - "$ENV{OCIO_LOCATION}" - ) - find_path(OCIO_LIBRARY_DIR - OpenColorIO.lib - HINTS - "${OCIO_LOCATION}" - "$ENV{OCIO_LOCATION}" - "${OCIO_BASE_DIR}" - PATH_SUFFIXES - lib/ - DOC - "OpenColorIO library path" - ) -endif() +find_path(OCIO_BASE_DIR + include/OpenColorIO/OpenColorABI.h + HINTS + "${OCIO_LOCATION}" + "$ENV{OCIO_LOCATION}" + DOC + "OCIO root folder" +) find_path(OCIO_INCLUDE_DIR OpenColorIO/OpenColorABI.h @@ -70,13 +41,15 @@ find_path(OCIO_INCLUDE_DIR PATH_SUFFIXES include/ DOC - "OpenColorIO headers path" + "OCIO headers path" ) -list(APPEND OCIO_INCLUDE_DIRS ${OCIO_INCLUDE_DIR}) - find_library(OCIO_LIBRARY - OpenColorIO + NAMES + libOpenColorIO.so.2.0 # libOpenColorIO.so.2.0 (Linux) + OpenColorIO.2.0 # libOpenColorIO.2.0.dylib (macOS) + OpenColorIO_2_0 # OpenColorIO_2_0.lib (Windows) + OpenColorIO # (fallback) HINTS "${OCIO_LOCATION}" "$ENV{OCIO_LOCATION}" @@ -84,12 +57,10 @@ find_library(OCIO_LIBRARY PATH_SUFFIXES lib/ DOC - "OCIO's ${OCIO_LIB} library path" + "OCIO library path" ) -list(APPEND OCIO_LIBRARIES ${OCIO_LIBRARY}) - -if(OCIO_INCLUDE_DIRS AND EXISTS "${OCIO_INCLUDE_DIR}/OpenColorIO/OpenColorABI.h") +if(OCIO_INCLUDE_DIR AND EXISTS "${OCIO_INCLUDE_DIR}/OpenColorIO/OpenColorABI.h") file(STRINGS ${OCIO_INCLUDE_DIR}/OpenColorIO/OpenColorABI.h fullVersion REGEX @@ -97,14 +68,14 @@ if(OCIO_INCLUDE_DIRS AND EXISTS "${OCIO_INCLUDE_DIR}/OpenColorIO/OpenColorABI.h" string(REGEX MATCH "[0-9]+.[0-9]+.[0-9]+" OCIO_VERSION ${fullVersion}) endif() -# handle the QUIETLY and REQUIRED arguments and set OCIO_FOUND to TRUE if -# all listed variables are TRUE +# handle the QUIETLY and REQUIRED arguments and set OpenColorIO_FOUND to TRUE +# if all listed variables are TRUE include(FindPackageHandleStandardArgs) find_package_handle_standard_args(OpenColorIO REQUIRED_VARS - OCIO_LIBRARIES - OCIO_INCLUDE_DIRS + OCIO_LIBRARY + OCIO_INCLUDE_DIR VERSION_VAR OCIO_VERSION )