Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Outdated vdb_tool OpenEXR import target #1978

Open
efleurant opened this issue Dec 4, 2024 · 0 comments
Open

[BUG] Outdated vdb_tool OpenEXR import target #1978

efleurant opened this issue Dec 4, 2024 · 0 comments
Labels

Comments

@efleurant
Copy link

Environment

Rocky Linux 9.4
master, v11.0.0, v12.0.0:
GCC: 11.4.1
cmake: 3.27.8
C++ standard: c++ 17

Describe the bug

I noticed that the CMake configuration for vdb_tool uses OpenEXR::IlmImf as one of the imported targets. This target was previously part of OpenEXR but is no longer available in recent versions, as the project has consolidated its libraries. The correct target to use should now be OpenEXR::OpenEXR.

You might want to update the CMakeLists.txt as follows to ensure compatibility with newer OpenEXR versions:

target_link_libraries(vdb_tool_common INTERFACE OpenEXR::IlmImf)

if(OPENVDB_TOOL_USE_EXR)
  target_compile_definitions(vdb_tool_common INTERFACE "VDB_TOOL_USE_EXR")
  find_package(OpenEXR REQUIRED)
  target_link_libraries(vdb_tool_common INTERFACE OpenEXR::OpenEXR)
endif()

Thanks in advance,
Cheers

@efleurant efleurant added the bug label Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant