AcademySoftwareFoundation / aswf-docker

Common container configuration
Apache License 2.0
148 stars 36 forks source link

For CY2022, the corresponding ocio-ci images must now have OSL installed #125

Closed hodoulp closed 2 years ago

hodoulp commented 2 years ago

For the ocio-ci images dedicated for CY2022, here are few improvements:

aloysbaillet commented 2 years ago

Hi @hodoulp , unfortunately the the current system doesn't nicely support having different requirements for different years... Adding that capability is in progress in the Conan branch, but that branch is far from ready.

That said I'd be happy to update the OCIO requirements and only release the new 2022 images with these changes, and not release the updates for years 2019-2021, but I some of the information (such as the content displayed there: https://github.com/AcademySoftwareFoundation/aswf-docker/tree/master/ci-ocio#aswfci-ocio20199) will look incorrect...

Would that be acceptable for you?

hodoulp commented 2 years ago

You can add OSL to the requirement list for the OCIO images, only release CY2022 and that's will be acceptable.

Note: To be consistent for the other images, I do not know what should be the OSL version for the older OCIO images. But that's clearly not an issue for OCIO as only CY2022 will use OSL.

aloysbaillet commented 2 years ago

Thanks I'll do that: just add OSL and not remove tbb/boost just yet!