Skip to content

Fix wheel builds for i686 Linux#13967

Merged
1ucian0 merged 1 commit intoQiskit:stable/1.4from
mtreinish:fix-ci
Mar 7, 2025
Merged

Fix wheel builds for i686 Linux#13967
1ucian0 merged 1 commit intoQiskit:stable/1.4from
mtreinish:fix-ci

Conversation

@mtreinish
Copy link
Copy Markdown
Member

Summary

In the 1.4.1 release the i686 linux wheel builds failed because the recent release of rustup 1.28.0 and 1.28.1 added a runtime dependency on libatomic to download and install the rust compiler. The i686 manylinux container we run the wheel builds in didn't contain this library so rustup errored and crashed the build. This commit fixes this issue by installing libatomic prior to running rustup. This fix was used in a throwaway branch to push out the 1.4.1 release on PyPI, but this fixes it for real for when we publish 1.4.2.

Details and comments

In the 1.4.1 release the i686 linux wheel builds failed because the
recent release of rustup 1.28.0 and 1.28.1 added a runtime dependency on
libatomic to download and install the rust compiler. The i686 manylinux
container we run the wheel builds in didn't contain this library so
rustup errored and crashed the build. This commit fixes this issue by
installing libatomic prior to running rustup. This fix was used in a
throwaway branch to push out the 1.4.1 release on PyPI, but this fixes
it for real for when we publish 1.4.2.
@mtreinish mtreinish requested a review from a team as a code owner March 6, 2025 18:11
@qiskit-bot
Copy link
Copy Markdown
Collaborator

One or more of the following people are relevant to this code:

  • @Qiskit/terra-core

@mtreinish mtreinish added type: qa Issues and PRs that relate to testing and code quality Changelog: None Do not include in the GitHub Release changelog. labels Mar 6, 2025
@mtreinish mtreinish added this to the 1.4.2 milestone Mar 6, 2025
Copy link
Copy Markdown
Collaborator

@Cryoris Cryoris left a comment

Choose a reason for hiding this comment

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

LGTM, this fixed the wheels for the 1.4.1 release (see e5a7844)

@mtreinish mtreinish added the ci: test wheels Run the wheel-build scripts as an additional CI run for this PR label Mar 6, 2025
@1ucian0 1ucian0 added this pull request to the merge queue Mar 7, 2025
Merged via the queue into Qiskit:stable/1.4 with commit 8a26423 Mar 7, 2025
@mtreinish mtreinish deleted the fix-ci branch April 22, 2025 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: None Do not include in the GitHub Release changelog. ci: test wheels Run the wheel-build scripts as an additional CI run for this PR type: qa Issues and PRs that relate to testing and code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants