From 20caceae78851c552e7d0c1fa75a528556e9eab2 Mon Sep 17 00:00:00 2001 From: Larry Gritz Date: Fri, 12 Mar 2021 18:44:17 -0800 Subject: [PATCH] The OPENEXR_INSTALL option needs to be defined before it's used This was causing OpenEXR builds (since #955 was merged) to NOT INSTALL. Oh boy. Signed-off-by: Larry Gritz --- CMakeLists.txt | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index bcab323cd9..1e8089d7b2 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -32,6 +32,12 @@ add_subdirectory(cmake) # Add all source in subdirectories ####################################### +option(OPENEXR_INSTALL "Install OpenEXR libraries" ON) +option(OPENEXR_INSTALL_TOOLS "Install OpenEXR tools" ON) +if(OPENEXR_INSTALL_TOOLS AND NOT OPENEXR_INSTALL) + message(SEND_ERROR "OPENEXR_INSTALL_TOOLS requires OPENEXR_INSTALL") +endif() + # Include these two modules without enable/disable options add_subdirectory(src/lib) add_subdirectory(src/bin) @@ -48,12 +54,6 @@ if(OPENEXR_INSTALL_EXAMPLES) add_subdirectory( src/examples ) endif() -option(OPENEXR_INSTALL "Install OpenEXR libraries" ON) -option(OPENEXR_INSTALL_TOOLS "Install OpenEXR tools" ON) -if(OPENEXR_INSTALL_TOOLS AND NOT OPENEXR_INSTALL) - message(SEND_ERROR "OPENEXR_INSTALL_TOOLS requires OPENEXR_INSTALL") -endif() - # If you want to use ctest to configure, build and # upload the results, cmake has builtin support for # submitting to CDash, or any server who speaks the