-
Notifications
You must be signed in to change notification settings - Fork 616
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
Bump pypa/cibuildwheel from 2.1.1 to 2.11.2 #2431
Bump pypa/cibuildwheel from 2.1.1 to 2.11.2 #2431
Conversation
Bumps [pypa/cibuildwheel](https://github.com/pypa/cibuildwheel) from 2.1.1 to 2.11.2. - [Release notes](https://github.com/pypa/cibuildwheel/releases) - [Changelog](https://github.com/pypa/cibuildwheel/blob/main/docs/changelog.md) - [Commits](pypa/cibuildwheel@v2.1.1...v2.11.2) --- updated-dependencies: - dependency-name: pypa/cibuildwheel dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
c165d29
to
114c546
Compare
This looks like a
|
A newer version of pypa/cibuildwheel exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged. |
I see in the logs that fontconfig is now missing on the musl builds:
But I am installing it on the apk command:
But for some reason it isn't installing fontconfig, only libjpeg. |
I could be reading the logs wrong but I think it is trying to build numpy from source. Not something we want to do. |
This lets vispy know that we expect some tests to fail due to being in a CI environment (texture NaN handling of GPUs in virtual machines).
And how website building is failing:
Likely from my realtime examples. |
It is running against the osmesa backend...I'm not sure how to turn that off. |
osmesa was found in the build image, but does not succeed. Also try forcing timer to stop in ex03a_data_sources_timer.py in hopes of fixing website example building warning.
Many of the realtime examples have their own cleanup based on main windows closing
I'm not sure why the wheel building is hanging. I wonder if I should try splitting the musl, aarch, and x86 builds into separate jobs. @brisvag or @kmuehlbauer have you seen this done with cibuildwheel in other projects? |
I could be reading it wrong, but it looks like it is trying to build 32-bit (i686) wheels with musl. I don't think numpy has 32-bit wheels so it is trying to build it from source and taking forever. |
Ok @brisvag so looks like numpy doesn't have any musl wheels yet. See: So while we can disable 32-bit ones in the future I've tried just disabling all musl builds for now to get this PR moving. |
@brisvag I think I'm ready for this to be merged. Please review and merge when you have the time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are the actual code changes intended, or some merge error?
Ah yes, sorry. The website building started failing because one of the threads in my realtime examples wasn't being killed anymore. Then the fixes for that started making the pyqt6 environment fail because they were using PyQt5 so I added a skip in this case. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, just making sure :)
Bumps pypa/cibuildwheel from 2.1.1 to 2.11.2.
Release notes
Sourced from pypa/cibuildwheel's releases.
... (truncated)
Changelog
Sourced from pypa/cibuildwheel's changelog.
... (truncated)
Commits
a6732b1
Bump version: v2.11.236a6101
Merge pull request #1311 from henryiii/henryiii/fix/rootpip28672fee
Merge pull request #1300 from mayeut/gha-3.11b1b8ab5
Merge pull request #1322 from ankith26/update-delocate-default89a8cac
Merge pull request #1324 from pypa/stderr-interleavingdb12084
tests: include check for upgrade notice toodfb3e62
tests: use 3.11 finalf259bfd
tests: add a check in tests for the pip warningf252274
fix: hide root pip warning on Linux again1aa841b
Merge pull request #1323 from pypa/docs-38Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)