Thanks to visit codestin.com
Credit goes to github.com

Skip to content

loading TCL / Tk symbols dynamically #6442

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
merged 14 commits into from
May 24, 2016
Merged

loading TCL / Tk symbols dynamically #6442

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
edc80a3
WIP: loading TCL / Tk symbols dynamically
matthew-brett May 17, 2016
64f01dd
RF: refactor C code to Michael D's commments
matthew-brett May 17, 2016
89535be
NF: add Windows support for TCL dynamic loading
matthew-brett May 18, 2016
fc5d060
RF: make dynamic loading of TCL the one true way
matthew-brett May 18, 2016
1c85968
RF: disable build-time link against TCL/Tk
matthew-brett May 18, 2016
830d7a3
RF: find TCL / Tk symbols correctly on Windows
matthew-brett May 19, 2016
4c5a3ab
RF: short-circuit the Tk function tests
matthew-brett May 20, 2016
94bb457
WIP: Add excerpts from TCL / Tk header
matthew-brett May 20, 2016
3f5407a
RF: check, edit, refactor Tcl / tk mini header
matthew-brett May 21, 2016
573aeb5
RF: remove Tcl / Tk header / lib discovery
matthew-brett May 21, 2016
626a1d2
RF: remove build-time check of Tkinter
matthew-brett May 22, 2016
0c2c5a0
RF: force TkAgg / Agg extensions with messages
matthew-brett May 23, 2016
de99a76
BF: fix goofy double "installing" message
matthew-brett May 23, 2016
aa275c6
TST: add test of tkagg backend import
matthew-brett May 24, 2016
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
Prev Previous commit
Next Next commit
RF: remove build-time check of Tkinter 
We don't need Tkinter at build time.

We could move this Tkinter version check to
`matplotlib/backends/tkagg.py` as a run-time check, but it's very
unlikely that any Python we support would be linked against Tcl / Tk <
8.4.  For example, the Python 2.5 Python.org OSX install links against
Tcl / Tk 8.5, and the default Tcl / Tk on CentOS 5.11 is 8.4.
  • Loading branch information
@matthew-brett
matthew-brett committed May 23, 2016
commit 626a1d276463faadc0268722335db093531d8efd
22 changes: 1 addition & 21 deletions setupext.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1525,28 +1525,8 @@ class BackendTkAgg(OptionalBackendPackage):
name = "tkagg"

def check_requirements(self):
try:
if PY3min:
import tkinter as Tkinter
else:
import Tkinter
except ImportError:
raise CheckFailed('TKAgg requires Tkinter.')
except RuntimeError:
raise CheckFailed('Tkinter present but import failed.')
else:
if Tkinter.TkVersion < 8.3:
raise CheckFailed("Tcl/Tk v8.3 or later required.")

try:
tk_v = Tkinter.__version__.split()[-2]
except (AttributeError, IndexError):
# Tkinter.__version__ has been removed in python 3
tk_v = 'not identified'

BackendAgg.force = True

return "version %s" % tk_v
return ""

def get_extension(self):
sources = [
Expand Down