Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: Add build recipe for ZLIB #4420

Merged
merged 2 commits into from
Sep 16, 2024

Conversation

zachlewis
Copy link
Contributor

Required for building redistributable python wheels on Windows Wheels runners.

@lgritz
Copy link
Collaborator

lgritz commented Sep 13, 2024

I looked at the "all local builds" CI test, and it's still using the system zlib, not exercising this new builder script.

I think it's because you named this file build_zlib.cmake, but that doesn't match the checked_find_package(ZLIB). Needs to be build_ZLIB.cmake.

I think maybe if it was working for you, it was only because you were using a case-insensitive file system?

@zachlewis
Copy link
Contributor Author

Needs to be build_ZLIB.cmake.

I'm dumb. I had this set up properly in one branch, and copy-pasted the contents to a file in another branch without checking the filename. Good catch...!

Required for building redistributable python wheels on Windows Wheels runners.

Signed-off-by: Zach Lewis <[email protected]>
Copy link
Collaborator

@lgritz lgritz left a comment

Choose a reason for hiding this comment

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

LGTM

@lgritz lgritz merged commit c658c8f into AcademySoftwareFoundation:master Sep 16, 2024
25 of 27 checks passed
zachlewis added a commit to zachlewis/OpenImageIO that referenced this pull request Sep 17, 2024
Required for building redistributable python wheels on Windows Wheels
runners.

Signed-off-by: Zach Lewis <[email protected]>
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.

2 participants