Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

CMakeLists.txt misses includes #91

Open
bfloch opened this issue May 4, 2016 · 0 comments
Open

CMakeLists.txt misses includes #91

bfloch opened this issue May 4, 2016 · 0 comments

Comments

@bfloch
Copy link

bfloch commented May 4, 2016

To be able to build I needed to include the include sub directory, as the src does not contain the headers.
Also partially ILMBASE headers are included with an OpenEXR/ prefix and partially without.
I guess the proper fix would be to always prefix includes with OpenEXR/ but this patch does fix it from the CMake side.

In case you prefer a pull request let me know.

--- /mnt/studio/toolbox/rez/payload/field3d/Field3D-1.7.1/CMakeLists.txt    2016-02-16 18:32:23.000000000 -0500
+++ /mnt/studio/toolbox/rez/payload/field3d/Field3D-1.7.1/CMakeLists.txt.patched    2016-05-04 11:27:32.873606565 -0400
@@ -63,8 +63,10 @@
 # includes
 INCLUDE_DIRECTORIES ( . )
 INCLUDE_DIRECTORIES ( src )
+INCLUDE_DIRECTORIES ( include )
 INCLUDE_DIRECTORIES ( export )
 INCLUDE_DIRECTORIES ( ${ILMBASE_INCLUDE_DIRS} )
+INCLUDE_DIRECTORIES ( ${ILMBASE_INCLUDE_DIRS}/OpenEXR )
 INCLUDE_DIRECTORIES ( ${HDF5_INCLUDE_DIRS} )
 INCLUDE_DIRECTORIES ( ${Boost_INCLUDE_DIR} )
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant