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

CI: update cibuildwheel==2.16.2 so we can have python 3.12 wheels #265

Merged
merged 5 commits into from
Nov 9, 2023

Conversation

fruch
Copy link

@fruch fruch commented Oct 18, 2023

cibuildwheel==2.16.2 supports building with python 3.12.0 release

fruch added 4 commits November 9, 2023 12:15
cibuildwheel==2.16.2 supports building with python 3.12.0 release
* few import needed to be ajusted/ignored
* need to update cython to latest version
since we want to support python 3.12, we need to remove
this pinning, cause those versions are casueing the cython
related unittets to fail:
```
 ImportError while importing test module '/project/tests/unit/cython/test_bytesio.py'.
  Hint: make sure your test modules/packages have valid Python names.
  Traceback:
  /opt/python/cp312-cp312/lib/python3.12/importlib/__init__.py:90: in import_module
      return _bootstrap._gcd_import(name[level:], package, level)
  /project/tests/unit/cython/test_bytesio.py:16: in <module>
      bytesio_testhelper = cyimport('tests.unit.cython.bytesio_testhelper')
  /project/tests/unit/cython/utils.py:29: in cyimport
      import pyximport
  ../venv/lib/python3.12/site-packages/pyximport/__init__.py:1: in <module>
      from .pyximport import *
  ../venv/lib/python3.12/site-packages/pyximport/pyximport.py:51: in <module>
      import imp
  E   ModuleNotFoundError: No module named 'imp'
```
eventlet is currently broken for python 3.12,
so until we have a fixed version.
we'll remove those tests from the build wheel action

Ref: eventlet/eventlet#795
@fruch fruch force-pushed the wheel_for_python3.12 branch from 604b2cc to a7bcd33 Compare November 9, 2023 10:16
switch from `python setup.py sdist` to `python -m build --sdist`

that's now the formal way to build, and not assume
we have `distutil` installed on that system.
@fruch fruch merged commit b1c6e6d into scylladb:master Nov 9, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant