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

Added cmake constraint. Force our dummy wheel or fail for version above. #60

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ccoulombe
Copy link
Contributor

CMake Python wheel are duplicate of the system cmake, in order for python packages to build with cmake, it provides cmake, cpack and ctest binaries
Having cmake wheels are unnecessary and takes a very long time to build, and fails randomly sometimes

Having a dummy wheel where we constraint pip to install our dummy wheel (3.23.1) or fail if a version above our module is requested.

Meet the requirements : cmake, cmake<3.24, cmake>3.21
Fails when :

  • version is lower, ie cmake==3.22. We then need to sed the build requirement this is not often
  • version is above ie cmake>3.27. Means we need to install that version has some specific features might be used.

@ccoulombe ccoulombe requested a review from mboisson September 22, 2023 19:24
@mboisson
Copy link
Member

Hum, I just thought that we need to have a separate constraint file for build-node, otherwise this will eventually make its way to production without having the actual dummy wheel on clusters.

@ccoulombe
Copy link
Contributor Author

Hmm ok, then we can let this PR rest until then

Added `constraints-cmake.txt` file
Keep but emptied `constraints-buildnode.txt` in case we like to add new constraints only on build-node
Updated 2020 and 2023 pip config files to use constraints-cmake.txt file
@ccoulombe ccoulombe self-assigned this Apr 24, 2024
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