GafferHQ / dependencies

All 3rd party dependencies necessary for building Gaffer
33 stars 26 forks source link

OpenEXR : Update to version 3.1.12 #256

Closed murraystevenson closed 8 months ago

murraystevenson commented 8 months ago

Bumping up from 3.1.9 as there are bug fixes necessary for proper DWA compression support in ExrCore.

Cortex will require https://github.com/ImageEngine/cortex/pull/1407 in order to build with this version.

johnhaddon commented 8 months ago

I just noticed that we're still on OIIO 2.4, quite a long way behind the latest 2.5.8.0. That latest version will auto-detect EXR version and set attribute( "openexr:core", true ) automatically if it's happy. I think that would be better than us setting it in Gaffer...

Maybe I've forgotten some reason we can't upgrade OIIO though?

murraystevenson commented 8 months ago

Maybe I've forgotten some reason we can't upgrade OIIO though?

I think we only avoided OIIO 2.5.x to have one less moving part while performing the VFX platform related dependencies updates. Now that the dust has settled there, and in light of the ExrCore related work, it does make sense to give it a try. We'll likely need to bump OSL to 1.12.14.0 at the same time, but we should probably do that anyway. I can take a look tomorrow.

johnhaddon commented 8 months ago

I think it's my turn in the tag-teaming of updates, so I've updated the PR to bump OIIO and OSL. Will get some builds going now.