python / cpython

The Python programming language
https://www.python.org/
Other
60k stars 29.04k forks source link

Lets Tkinter work with MacOSX native Tk #35685

Closed f061053c-37fe-4bf6-ad3e-9d9600654e34 closed 22 years ago

f061053c-37fe-4bf6-ad3e-9d9600654e34 commented 22 years ago
BPO 490100
Nosy @loewis, @jackjansen
Files
  • tkinter-patch-20011206: patch against Tkinter module
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields: ```python assignee = 'https://github.com/jackjansen' closed_at = created_at = labels = ['OS-mac'] title = 'Lets Tkinter work with MacOSX native Tk' updated_at = user = 'https://bugs.python.org/tonylownds' ``` bugs.python.org fields: ```python activity = actor = 'jackjansen' assignee = 'jackjansen' closed = True closed_date = None closer = None components = ['macOS'] creation = creator = 'tonylownds' dependencies = [] files = ['3810'] hgrepos = [] issue_num = 490100 keywords = ['patch'] message_count = 5.0 messages = ['38340', '38341', '38342', '38343', '38344'] nosy_count = 3.0 nosy_names = ['loewis', 'jackjansen', 'tonylownds'] pr_nums = [] priority = 'normal' resolution = 'accepted' stage = None status = 'closed' superseder = None type = None url = 'https://bugs.python.org/issue490100' versions = ['Python 2.3'] ```

    f061053c-37fe-4bf6-ad3e-9d9600654e34 commented 22 years ago

    There is a new Tcl/Tk in alpha that works on MacOSX's windowing layer natively. This patch adds calls necessary for Tkinter to work with it.

    The Tcl/Tk alpha can be picked up here:

    http://sourceforge.net/project/showfiles.php?group_id=10894

    NOTE: The amount of extra code needed to interface with Tcl/Tk will probably go down with the next alpha of Tcl/Tk.

    61337411-43fc-4a9c-b8d5-4060aede66d0 commented 22 years ago

    Logged In: YES user_id=21627

    Please review your patches carefully before submitting them. Why does this change PyOS_snprintf to sprintf?

    jackjansen commented 22 years ago

    Logged In: YES user_id=45365

    I assume the sprintf change was a mistake (I've undone it after I applied the patch). Aside from that the patch looks harmless to other platforms, but I haven't gotten it to work yet. It fails compilation with a missing X11/Xlib.h include. If I can get it to compile at least once I'll put it in CVS before 2.2 (even though it is only useful to the real die-hards: it requires a Tk alfa, and only works under the experimental framework-based Python.app).

    jackjansen commented 22 years ago

    Logged In: YES user_id=45365

    The mods to _tkinter.c and tkappinit.c are in the repository. What still needs to be done is a readme file explaining where to obtain the X11 headers, what to put into Setup.local and how to run your Tkinter scripts.

    jackjansen commented 22 years ago

    Logged In: YES user_id=45365

    This patch was applied 3 months ago, and noone seems to be willing to write a readme. Still, people seem succesful in getting this to work, so let's forget about the readme and close the patch.