Skip to content

Conversation

@lgritz
Copy link
Collaborator

@lgritz lgritz commented Dec 9, 2025

aswftesting is no longer maintained.

aswftesting is no longer maintained.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
@lgritz lgritz requested a review from jfpanisset December 9, 2025 20:44
Copy link
Contributor

@jfpanisset jfpanisset left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That should get you the slimmer images without the baked in Conan download cache.

What's forcing you to remove the bundled OpenImageIO version in the 2024 images and having to rebuild your own with the REMOVE_INSTALLED_OPENIMAGEIO mechanism? I'd be happy to fix that so you don't have to spend CI time rebuilding OIIO.

@lgritz
Copy link
Collaborator Author

lgritz commented Dec 10, 2025

What's forcing you to remove the bundled OpenImageIO version in the 2024 images and having to rebuild your own with the REMOVE_INSTALLED_OPENIMAGEIO mechanism? I'd be happy to fix that so you don't have to spend CI time rebuilding OIIO.

It's often the case that I want to try combinations of a different OIIO than the one in the container, and sometimes it can be hard to guarantee that cmake only finds one and not any remnants of the others, creating an unholy mixture of headers from one version and libraries from another, or a Python module from one version but a library from another. Especially when one of the installs is in a "system area" that has to get included in various search paths. Sometimes the expedient way to solve this is just to remove the one I don't want from the container.

In this particular case, I wanted a newer OIIO for the sanitizer tests. I should really push the sanitizer tests to a more recent container, but I wanted to get this repair out quickly and not spend 3 days fighting some unrelated reason why it might not work in the 2025 or 2026 container. This was just the fastest way to an all-green lineup.

@lgritz lgritz merged commit 3a343a5 into AcademySoftwareFoundation:main Dec 10, 2025
51 of 52 checks passed
@jfpanisset
Copy link
Contributor

Makes perfect sense, just wanted to make sure there wasn't something specifically broken with the OIIO in the 2024 image which was forcing you to replace it.

@lgritzecho
Copy link
Contributor

lgritzecho commented Dec 10, 2025 via email

@lgritz lgritz deleted the lg-aswftesting branch December 13, 2025 00:17
lgritz added a commit to lgritz/OpenShadingLanguage that referenced this pull request Dec 13, 2025
…undation#2061)

aswftesting is no longer maintained.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
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