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

Remove duplicate cmake dependency from skbuild plugin #1958

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mgorny
Copy link

@mgorny mgorny commented Jan 20, 2025

Remove the duplicate cmake dependency that was added by openexr_skbuild_plugin.py. Scikit-build-core is adding a dependency on CMake if necessary itself, and adding one unconditionally has a side effect of installing a local PyPI version of CMake that overrides the system CMake that scikit-build-core would be using instead. This can be particularly problematic when downstream patching of CMake is required on the system in question.

Fixes #1957

Remove the duplicate `cmake` dependency that was added by
`openexr_skbuild_plugin.py`.  Scikit-build-core is adding a dependency
on CMake if necessary itself, and adding one unconditionally has a side
effect of installing a local PyPI version of CMake that overrides
the system CMake that scikit-build-core would be using instead.  This
can be particularly problematic when downstream patching of CMake
is required on the system in question.

Fixes AcademySoftwareFoundation#1957

Signed-off-by: Michał Górny <[email protected]>
Copy link

CLA Not Signed

@mgorny
Copy link
Author

mgorny commented Jan 20, 2025

I don't seem to be able to sign the CLA since both EasyCLA and Linux Foundation's support system are both throwing internal server errors.

Copy link
Member

@cary-ilm cary-ilm left a comment

Choose a reason for hiding this comment

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

Thanks, I had independently determined those lines were unnecessary but hadn't acted on it yet, so looks good to me

@cary-ilm
Copy link
Member

That's odd about the EasyCLA and LF support, could you possibly try it again, in case it was a glitch on their end? Thanks!

@mgorny
Copy link
Author

mgorny commented Jan 22, 2025

Yep, still failing. EasyCLA:

failed

Support system:

fail2

I'm going to try mailing them.

@cary-ilm
Copy link
Member

I filed a LF support ticket but didn't get much help, beyond the suggestion to try a different browser?

@mgorny
Copy link
Author

mgorny commented Jan 23, 2025

Oh, thanks for letting me know, that made me go through some debugging and I've found the problem:

$ resolvectl query cdn.platform.linuxfoundation.org
cdn.platform.linuxfoundation.org: resolve call failed: DNSSEC validation failed: failed-auxiliary

I just wished people would actually give useful error messages such as "can't resolve API domain" instead of "internal errors", sigh.

@cary-ilm
Copy link
Member

@mgorny, did you get past the EasyCLA error? Are you able to sign the form? Thanks!

@mgorny
Copy link
Author

mgorny commented Jan 27, 2025

@mgorny, did you get past the EasyCLA error? Are you able to sign the form? Thanks!

Yes, it's waiting for our CLA manager to approve.

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.

openexr_skbuild_plugin.py adds an unnecessary cmake build requirement
2 participants