GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — improve or remove parsing of Tkinter.__version__ required for newer python versions · Issue #139 · matplotlib/matplotlib

improve or remove parsing of Tkinter.__version__ required for newer python versions #139

New issue
Copy link
New issue
Copy link
Closed
Closed
improve or remove parsing of Tkinter.__version__ required for newer python versions#139
Copy link
Assignees
efiring
Milestone
v1.1.0

Description

@juliantaylor
juliantaylor
opened on Jun 8, 2011
Issue body actions

due to the cpython hg transition the version attribute of tkinter has been removed in python 3 and in the python 2.7 rc1 it does not contain a number anymore.
see http://bugs.python.org/issue12236
http://hg.python.org/cpython/file/f3553d5083f8/Lib/tkinter/__init__.py

This breaks the matplotlib build which parses version and expects a number:

OPTIONAL BACKEND DEPENDENCIES
                libpng: 1.2.44
Traceback (most recent call last):
  File "./setup.py", line 162, in <module>
    if check_for_tk() or (options['build_tkagg'] is True):
  File "/build/buildd/matplotlib-1.0.1/setupext.py", line 832, in check_for_tk
    (Tkinter.__version__.split()[-2], Tkinter.TkVersion, Tkinter.TclVersion))
IndexError: list index out of range

Activity

  1. ddale commented on Jun 8, 2011

    @ddale
    ddale
    on Jun 8, 2011
    Contributor
    More actions

    This issue was already addressed in commit 04b4e50 on v1.0.x-maint and 9bcca38 on master. Thank you for the report.

  2. ddale
    closed this as completedon Jun 8, 2011
  3. efiring
    reopened this on Jun 9, 2011
  4. efiring commented on Jun 9, 2011

    @efiring
    efiring
    on Jun 9, 2011
    Member
    More actions

    Those commits don't take care of version being missing in py3k. Since master is periodically merged into py3k, I am inclined to fix master to handle the missing version case along with everything else.

  5. ddale commented on Jun 9, 2011

    @ddale
    ddale
    on Jun 9, 2011
    Contributor
    More actions

    My mistake, thanks for catching.

  6. ghost assigned
    efiring
    on Jun 15, 2011
  7. efiring commented on Jun 15, 2011

    @efiring
    efiring
    on Jun 15, 2011
    Member
    More actions

    Fixed in master with e4a34df

  8. efiring
    closed this as completedon Jun 15, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

  • @efiring
    efiring

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    • v1.1.0

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    You are visitor 01402503 — GeezerVison does not store the pages you look up.
    Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.