Skip to content

Revert "Ignore cffi installation error"#285

Merged
messense merged 1 commit intoPyO3:mainfrom
mkniewallner:unignore-cffi-install-error
Sep 7, 2024
Merged

Revert "Ignore cffi installation error"#285
messense merged 1 commit intoPyO3:mainfrom
mkniewallner:unignore-cffi-install-error

Conversation

@mkniewallner
Copy link
Copy Markdown
Contributor

This reverts commit f61caa2.

cffi now supports, and provides wheels for, Python 3.13 (https://github.com/python-cffi/cffi/releases/tag/v1.17.0), so this should work as expected on 3.13.

Tested on https://github.com/mkniewallner/mkv-playground/pull/9/files where cffi was correctly installed:

Install maturin
[...]
  Collecting cffi
    Downloading cffi-1.17.1-cp313-cp313-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (1.5 kB)
  Collecting pycparser (from cffi)
    Downloading pycparser-2.22-py3-none-any.whl.metadata (943 bytes)
  Downloading cffi-1.17.1-cp313-cp313-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (479 kB)
  Downloading pycparser-2.22-py3-none-any.whl (117 kB)
  Installing collected packages: pycparser, cffi
  Successfully installed cffi-1.17.1 pycparser-2.22

This reverts commit f61caa2.

`cffi` now supports, and provide wheels for, Python 3.13
(https://github.com/python-cffi/cffi/releases/tag/v1.17.0), so this
should work as expected on 3.13.
@mkniewallner mkniewallner marked this pull request as ready for review September 6, 2024 21:55
@messense messense merged commit b234962 into PyO3:main Sep 7, 2024
@mkniewallner mkniewallner deleted the unignore-cffi-install-error branch September 7, 2024 08:16
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