Skip to content
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.

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

python@3.8: add arm64 compatibility patches #57425

Merged
merged 1 commit into from
Jul 4, 2020
Merged

python@3.8: add arm64 compatibility patches #57425

merged 1 commit into from
Jul 4, 2020

Conversation

claui
Copy link
Contributor

@claui claui commented Jul 4, 2020

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

This set of patches includes the following upstream pull requests:

Adding the patches before upstream has released them is warranted here because python@3.8 is widely used as a dependency, and the patches are needed to enable testing dependent formulae on arm64.

Supersedes Homebrew/formula-patches#287.

This set of patches includes the following upstream pull requests:

- python/cpython#21114
  "Support `arm64` in Mac/Tools/pythonw"

- python/cpython#21224
  "allow python to build for macosx-11.0-arm64"

- python/cpython#21249
  "ctypes fixes for arm64 Mac OS"

Adding the patches before upstream has released them is warranted here
because `python@3.8` is widely used as a dependency, and the patch is
needed to enable testing dependent formulae on arm64.

CC: Sean Molenaar <1484494+SMillerDev@users.noreply.github.com>
@claui claui added the 11 Big Sur is specifically affected label Jul 4, 2020
@dawidd6
Copy link
Member

dawidd6 commented Jul 4, 2020

I think building x86_64 bottles for this PR is unnecessary, since it is a change for ARM. So I would propose cancelling the jobs and just merge it via UI, to not occupy the runners.

@claui
Copy link
Contributor Author

claui commented Jul 4, 2020

@dawidd6 Good point. Workflow cancelled.

@SMillerDev SMillerDev merged commit e9700c9 into Homebrew:master Jul 4, 2020
@SMillerDev
Copy link
Member

Merged, thanks @claui

@claui claui deleted the python3.8-arm branch July 4, 2020 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Big Sur is specifically affected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants