Skip to content

feat: initial support for riscv64 - #2366

Merged
henryiii merged 4 commits into
pypa:mainfrom
mayeut:riscv64
Apr 28, 2025
Merged

feat: initial support for riscv64#2366
henryiii merged 4 commits into
pypa:mainfrom
mayeut:riscv64

Conversation

@mayeut

@mayeut mayeut commented Apr 19, 2025

Copy link
Copy Markdown
Member

Towards #2263

While riscv64 would need a PEP in order to publish to PyPI, the community is already using a patch version of cibuildwheel to populate a repository hosted on GitLab: https://gitlab.com/riseproject/python/wheel_builder.

In order to get things moving forward, initial support could be added to cibuildwheel while waiting for full support once a PEP is available.

IMHO, the fact that those wheels can't be uploaded to PyPI isn't a showstopper here (in fact, the same is true of pyodide wheels). The only thing I'm concerned about with this PR is that building "all" platforms on Linux would now include something not upload able to PyPI so maybe it should be protected by an enable flag ?

@joerick

joerick commented Apr 20, 2025

Copy link
Copy Markdown
Contributor

I expect only a few people have set CIBW_ARCHS_LINUX: all but I did find a few.

However, making this an enable group is a good idea IMO, since it's kind of experimental... In theory, something about these wheels might change before the PEP is approved. That way it matches the logic of the prerelease enable group. cpython-experimental-riscv would work for me as a name.

@joerick joerick left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! I think we'd want to document the option in the docs for CIBW_ENABLE, in options.md.

Comment thread cibuildwheel/logger.py Outdated
@henryiii
henryiii requested a review from Copilot April 28, 2025 16:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@henryiii
henryiii merged commit 5f8d06f into pypa:main Apr 28, 2025
@mayeut
mayeut deleted the riscv64 branch May 8, 2025 12:45
xen0n added a commit to xen0n/pygit2 that referenced this pull request Jun 16, 2026
RISC-V wheels have been supported by cibuildwheel and PyPI for a while,
so let's build binary wheels for it as many of the boards currently on
the market are rather slow to make building on the spot rather painful.

Human note: The entirety of the commit message is written manually.

See: pypa/cibuildwheel#2366
See: pypi/warehouse#18243
Assisted-by: GPT-5.5
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.

4 participants