diff options
| author | Sergey Sharybin <sergey.vfx@gmail.com> | 2019-06-26 20:50:38 +0300 |
|---|---|---|
| committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2019-06-26 20:50:38 +0300 |
| commit | d93558e91488f1c110a3a2e2e3399e2917f77e13 (patch) | |
| tree | ee7c1d372aaad4f7657d3be986e6d0997b3df935 /build_files/build_environment/cmake/osl.cmake | |
| parent | a7e32dbcb08c452fd170fcc71022278bb7985e72 (diff) | |
Make deps: Fix detection/linking of PugiXML on Linux
This time both full `make deps` and final compilation is tested on
a freshly installed CentOS 7.
The thing is: OpenImageIO is not configured to use an external PugiXML
library, so it was compiling its own.
At the same time the OpenShadingLanguage library was commanded to use
an externally compiled PugiXML. This caused some sort of discrepancy
which lead to Blender-link-time errors. Could be linking error, could
be namespace related, could be ABI related. In any case since we do
have PugiXML in the OpenImageIO already lets just stick to it.
Diffstat (limited to 'build_files/build_environment/cmake/osl.cmake')
| -rw-r--r-- | build_files/build_environment/cmake/osl.cmake | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/build_files/build_environment/cmake/osl.cmake b/build_files/build_environment/cmake/osl.cmake index 7b5909e3700..b8268f90c33 100644 --- a/build_files/build_environment/cmake/osl.cmake +++ b/build_files/build_environment/cmake/osl.cmake @@ -81,11 +81,6 @@ if(WIN32) ${OSL_EXTRA_ARGS} -DPUGIXML_HOME=${LIBDIR}/pugixml ) -elseif(UNIX AND NOT APPLE) - set(OSL_EXTRA_ARGS - ${OSL_EXTRA_ARGS} - -DPUGIXML_HOME=${LIBDIR}/pugixml - ) elseif(APPLE) # Make symbol hiding consistent with OIIO which defaults to OFF, # avoids linker warnings on macOS @@ -116,9 +111,17 @@ add_dependencies( external_zlib external_flexbison external_openimageio - external_pugixml ) +if(UNIX AND NOT APPLE) + # Rely on PugiXML compiled with OpenImageIO +else() + add_dependencies( + external_osl + external_pugixml + ) +endif() + if(WIN32) if(BUILD_MODE STREQUAL Release) ExternalProject_Add_Step(external_osl after_install |