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

Add soversion to IlmImfCtl too #153

Merged
merged 3 commits into from
Jun 6, 2024
Merged

Conversation

paolostivanin
Copy link
Contributor

@paolostivanin paolostivanin commented May 30, 2024

This improves the packaging experience for us distro packagers 😄

@michaeldsmith
Copy link
Collaborator

@paolostivanin thanks for the contribution. Do you think we should also add this to https://github.com/ampas/CTL/blob/master/lib/dpx/CMakeLists.txt ?

I think Github recently updated the Mac workflow runners from Intel to Apple Silicon and now some Mac CI is failing, I will try to fix it.

@paolostivanin
Copy link
Contributor Author

good question. I didn't receive any warning about that lib (I'm an opensuse packager).
I'd have to check the build system if that lib is being built at all, but I'll be afk until Sunday:)

@michaeldsmith michaeldsmith merged commit bc50ecd into ampas:master Jun 6, 2024
54 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.

2 participants