Skip to content

Add information about lack of possibility of cross compile Poetry #1416

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.

Sign up for GitHub

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

Merged
joerick merged 11 commits into from
Jul 9, 2023
Merged

Add information about lack of possibility of cross compile Poetry #1416

Changes from 4 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
5be1d17
add information about lack of possibility of cross compile Poetry pro…
Czaki Feb 10, 2023
cc0bb5d
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 10, 2023
96cffc2
Update docs/faq.md
Czaki Feb 10, 2023
c7643f0
update information about poetry state
Czaki Feb 14, 2023
567fe2a
Apply suggestions from code review
Czaki Feb 14, 2023
ed32041
fix note text
Czaki Feb 16, 2023
983347f
Add reference to the new 'wheel tags' command
joerick Jun 18, 2023
3192cdc
Copy pass on this text
joerick Jun 18, 2023
1ce2091
Copy edit of the whole section, make it a bit easier to follow, add s…
joerick Jun 18, 2023
8f569c4
Merge remote-tracking branch 'origin/main' into poerty_problem
joerick Jun 23, 2023
f44d278
Update docs/faq.md
joerick Jul 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion docs/faq.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,18 @@ Linux wheels are built in [`manylinux`/`musllinux` containers](https://github.co

### Building macOS wheels for Apple Silicon {: #apple-silicon}

`cibuildwheel` supports cross-compiling `universal2` and `arm64` wheels on `x86_64` runners. With the introduction of Apple Silicon, you now have several choices for wheels for Python 3.8+:
`cibuildwheel` supports cross-compiling `universal2` and `arm64` wheels on `x86_64` runners.

!!! note
Currently it is not fully supported to cross-compile project that uses Poetry as a build backend.
Reladed issue [here](https://github.com/python-poetry/poetry/issues/7107).
Some packages can be built with arm64 wheels, but their file names will be incorrect,
with the platform tag showing `x86_64` instead of `arm64`.
As a workaround, the file can be renamed before running delocate to repair the wheel.
Copy link
Contributor

@henryiii henryiii Feb 14, 2023

Choose a reason for hiding this comment

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

The internal tags will still be broken? You'd need pypa/wheel#422 I believe.

Copy link
Contributor

@henryiii henryiii Feb 14, 2023

Choose a reason for hiding this comment

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

I'd remove this renaming suggestion and just say the tags have to be corrected. An example is in https://github.com/scikit-build/cmake-python-distributions/blob/master/scripts/repair_wheel.py.

Copy link
Contributor Author

@Czaki Czaki Feb 16, 2023

Choose a reason for hiding this comment

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

Now better?

For further details, please refer to this (https://github.com/pypa/cibuildwheel/pull/1416#discussion_r1105217074)."


With the introduction of Apple Silicon, you now have several choices for wheels for Python 3.8+:

#### `x86_64`

Expand Down