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

Jump to bottom

Fix Rosetta2 usage for shell commands #1195

Merged

joerick merged 1 commit into pypa:main from Jackenmen:fix_arch_x86_64_shell_true

Jul 29, 2022

Contributor

Jackenmen commented Jul 25, 2022

Fixes #1193

Instead of the suggested sh, I used /bin/sh as that's how Python's subprocess does it:
https://github.com/python/cpython/blob/a566912049722386478c075b83534fa9f7a1de45/Lib/subprocess.py#L1759-L1765

Successful build log: https://cirrus-ci.com/task/5802864923639808
Link to the CI configuration: https://raw.githubusercontent.com/jack1142/cibuildwheel/macos_arch_x86_64_issue/.cirrus.yml


          Fix Rosetta2 usage for shell commands

aa4ca42

joerick approved these changes

View reviewed changes

joerick merged commit f2c03fa into pypa:main

Jackenmen deleted the fix_arch_x86_64_shell_true branch

July 29, 2022 23:46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment