Page MenuHome

cmake/deps: update alembic to 1.7.16
ClosedPublic

Authored by Ray Molenkamp (LazyDodo) on Feb 5 2021, 5:09 PM.

Details

Summary

straight forward version bump, the only thing significant is
that alembic was using boost to get shared/unique_ptr and
they are seemingly no longer CI'ing with ALEMBIC_LIB_USES_BOOST
so that gave build errors on both linux and windows.

after consulting with sybren it was decided that dropping the
boost dep and relying on the STL implementation will be the
way to go.

Diff Detail

Repository
rB Blender
Branch
tmp_alembic (branched from master)
Build Status
Buildable 12661
Build 12661: arc lint + arc unit

Event Timeline

Ray Molenkamp (LazyDodo) requested review of this revision.Feb 5 2021, 5:09 PM
Ray Molenkamp (LazyDodo) created this revision.
This revision is now accepted and ready to land.Feb 5 2021, 5:11 PM

@Ray Molenkamp (LazyDodo) On a second note, wouldn't it be nice to make this change explicit (-DALEMBIC_LIB_USES_BOOST=OFF)?

Obviously my bad, but I just had alembic fail on me because cmake was still referencing boost.

It shouldn't? but if you want to explicitly tell it not to that be fine with me

took a closer look, this flag is off by default?

Ah never mind, it builds as is it is right now and that counts.

This revision was automatically updated to reflect the committed changes.