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

bpo-34956: Fix macOS _tkinter use of Tcl/Tk in /Library/Frameworks #20171

Merged
merged 1 commit into from
May 18, 2020

Conversation

ned-deily
Copy link
Member

@ned-deily ned-deily commented May 18, 2020

_tkinter now builds and links with non-system Tcl and Tk frameworks if they
are installed in /Library/Frameworks as had been the case on older releases
of macOS. If a macOS SDK is explicitly configured, by using ./configure
--enable-universalsdk= or -isysroot, only a Library/Frameworks directory in
the SDK itself is searched. The default behavior can still be overridden with
configure --with-tcltk-includes and --with-tcltk-libs.

https://bugs.python.org/issue34956

_tkinter now builds and links with non-system Tcl and Tk frameworks if they
are installed in /Library/Frameworks as had been the case on older releases
of macOS. If a macOS SDK is explicitly configured, by using ./configure
--enable-universalsdk= or -isysroot, only a Library/Frameworks directory in
the SDK itself is searched. The default behavior can still be overridden with
configure --with-tcltk-includes and --with-tcltk-libs.
@ned-deily ned-deily merged commit 1731d6d into python:master May 18, 2020
@ned-deily ned-deily deleted the bpo-34956-tk-library-fw branch May 18, 2020 08:33
arturoescaip pushed a commit to arturoescaip/cpython that referenced this pull request May 24, 2020
…ythonGH-20171)

_tkinter now builds and links with non-system Tcl and Tk frameworks if they
are installed in /Library/Frameworks as had been the case on older releases
of macOS. If a macOS SDK is explicitly configured, by using ./configure
--enable-universalsdk= or -isysroot, only a Library/Frameworks directory in
the SDK itself is searched. The default behavior can still be overridden with
configure --with-tcltk-includes and --with-tcltk-libs.
claui added a commit to Homebrew/formula-patches that referenced this pull request Jul 15, 2020
This set of patches includes the following upstream pull requests,
in this order:

- PR 20171, "Fix _tkinter use"
  python/cpython#20171
  (prerequisite for patch #21249 to apply)

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

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

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

The patches for 20171 and 21249 have been minimally modified in order
to backport them to 3.8.3.

Note that these have been successfully tested for `python@3.8`
but not for `python@3.7`.

The patch directive should be surrounded by an `if Hardware::CPU.arm?`
block.
claui pushed a commit to xvilo/homebrew-core that referenced this pull request Jul 15, 2020
This replaces the three unmerged PR patches with a hosted
formula patch.

This includes the following upstream pull requests:

- python/cpython#20171, "Fix _tkinter use" (prerequisite for 21249)
- 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"

See also:

- Homebrew/formula-patches#292
BrewTestBot pushed a commit to Homebrew/homebrew-core that referenced this pull request Jul 16, 2020
This replaces the three unmerged PR patches with a hosted
formula patch.

This includes the following upstream pull requests:

- python/cpython#20171, "Fix _tkinter use" (prerequisite for 21249)
- 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"

See also:

- Homebrew/formula-patches#292

Closes #57997.

Signed-off-by: Claudia Pellegrino <1239874+claui@users.noreply.github.com>
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.

3 participants