Update OpenEXR and spill out its deps

Devsh-Graphics-Programming / Nabla

Vulkan, OptiX and CUDA Interoperation Modular Rendering Library and Framework for PC/Linux/Android

http://devsh.eu

Apache License 2.0

493 stars 59 forks source link

Closed tad1 closed 3 months ago

tad1 commented 3 months ago

update OpenEXR to latest add IMath and Deflate submodule force OpenEXR to use local deps (IMath & Deflate) instead of using fetchContent

compiles Nabla w/ and w/o _NBL_COMPILE_WITH_OPEN_EXR_

[x] make OpenEXR optional dep based on _NBL_COMPILE_WITH_OPEN_EXR_
[x] fix Deflate cmake warrnings; once again make OpenEXR use our Deflate ~~- [ ] remove unnecesary OpenEXR targets~~

tad1 commented 3 months ago

currently, IMath & Deflate submodules point to OP's repo, @AnastaZIuk suggested to fork IMath; should Deflate also be forked?

AnastaZIuk commented 3 months ago

currently, IMath & Deflate submodules point to OP's repo, @AnastaZIuk suggested to fork IMath; should Deflate also be forked?

actually as long as we don't plan to modify their CMake we don't have to fork