Skip to content

Comments

Support Python 3.13 and introduce pypa/cibuildwheel#509

Merged
philomath213 merged 8 commits intomainfrom
cibuildwheel
Feb 19, 2025
Merged

Support Python 3.13 and introduce pypa/cibuildwheel#509
philomath213 merged 8 commits intomainfrom
cibuildwheel

Conversation

@philomath213
Copy link
Member

@philomath213 philomath213 commented Feb 18, 2025

Description

Affected Dependencies

None

How has this been tested?

  • I have tested cibuildwheel in a fork (I had to remove the other workflows to speedup the debugging and testing).
    You can check the tests here Windowns, Linux, MacOS

Checklist

@philomath213 philomath213 added the Build Related to the build of the package label Feb 18, 2025
Copy link
Member

@youben11 youben11 left a comment

Choose a reason for hiding this comment

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

We still have to test it, but it looks good! I left some minor comments.
Do you know what's causing that job to fail?

- name: Run gtest
timeout-minutes: 40
run: bazel test --test_output=all --spawn_strategy=standalone --test_timeout=1500 //tests/cpp/...
run:
Copy link
Member

Choose a reason for hiding this comment

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

I thought you need a | here? Like run: |

@philomath213
Copy link
Member Author

We still have to test it

I have tested cibuildwheel in a fork (I had to remove the other workflows to speedup the debugging and testing).
You can check the tests here Windowns, Linux, MacOS

Do you know what's causing that job to fail?

We were using the legacy system WORKSPACE of Bazel, I think it has been sunset. (Related issue)
We have to migrate to bzlmod system if we want to keep supporting Bazel build.

@philomath213 philomath213 merged commit e8731d2 into main Feb 19, 2025
32 of 34 checks passed
@philomath213 philomath213 deleted the cibuildwheel branch February 27, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build Related to the build of the package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

'tenseal._tenseal_cpp' Module Not Found

2 participants