Skip to content

Conversation

@murraystevenson
Copy link
Contributor

This begins bumping our dependency versions to better align with VFX Platform 2024.

Notably, Boost hasn't been updated here as we'll first require some of the updates from ImageEngine/cortex#1460 merged and a new Cortex version released. For now, I've included a small patch to fix an issue between Boost 1.80 and Python 3.11, but this can be dropped as soon as we update to Boost 1.82.

We expect the Qt 6.5 update will be tackled independently after the remainder of the dependencies updates are in place. In the meantime I've bumped Qt 5.15 to the latest available as this includes some fixes that should help us restore macOS CI with Xcode 15.

We're put into a bit of a odd corner with Minizip's defined value of `MZ_VERSION_BUILD`, which OCIO relies on for compatibility with both Minizip 3.x and 4.x.

In Minizip 3.0.8 and above this is defined as an octal literal with apparently? hexadecimal values. Recent releases of Minizip 4 correctly prepend the hexadecimal value with 0x, but OCIO's version checks haven't caught up. So we're left either patching `MZ_VERSION_BUILD` to a valid octal value or updating to Minizip 4.x and patching OCIO's `#if MZ_VERSION_BUILD >= 040000` checks to `#if MZ_VERSION_BUILD >= 0x40000`. Of the two choices, I've gone with the one requiring the smallest patch.
This can be removed as soon as we update to Boost 1.82.

Patch taken from: boostorg/python@a218bab
@murraystevenson murraystevenson self-assigned this May 13, 2025
@boberfly
Copy link
Contributor

Just to note as of yesterday I am building/testing ImageEngine/cortex#1460 with boost-1.85 - for Cortex it'll need an IMATH_LIB_SUFFIX at least for Windows (-3_1 and not -3_2 that OpenEXR needs).

My custom Cortex lives at fb/RB-10.5 https://github.com/boberfly/cortex/blob/d81245ee673bd9636b42ec5a6ddb077b6ef561a9/SConstruct#L215

Cheers

@johnhaddon johnhaddon merged commit 8d49a59 into GafferHQ:main May 14, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants