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

Lets Tkinter work with MacOSX native Tk #35685

Closed
tonylownds mannequin opened this issue Dec 7, 2001 · 5 comments
Closed

Lets Tkinter work with MacOSX native Tk #35685

tonylownds mannequin opened this issue Dec 7, 2001 · 5 comments
Assignees
Labels

Comments

@tonylownds
Copy link
Mannequin

tonylownds mannequin commented Dec 7, 2001

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:

    assignee = 'https://github.com/jackjansen'
    closed_at = <Date 2002-03-17.21:51:34.000>
    created_at = <Date 2001-12-07.02:44:30.000>
    labels = ['OS-mac']
    title = 'Lets Tkinter work with MacOSX native Tk'
    updated_at = <Date 2002-03-17.21:51:34.000>
    user = 'https://bugs.python.org/tonylownds'

    bugs.python.org fields:

    activity = <Date 2002-03-17.21:51:34.000>
    actor = 'jackjansen'
    assignee = 'jackjansen'
    closed = True
    closed_date = None
    closer = None
    components = ['macOS']
    creation = <Date 2001-12-07.02:44:30.000>
    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']

    @tonylownds
    Copy link
    Mannequin Author

    tonylownds mannequin commented Dec 7, 2001

    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.

    @tonylownds tonylownds mannequin closed this as completed Dec 7, 2001
    @tonylownds tonylownds mannequin assigned jackjansen Dec 7, 2001
    @tonylownds tonylownds mannequin added the OS-mac label Dec 7, 2001
    @tonylownds tonylownds mannequin closed this as completed Dec 7, 2001
    @tonylownds tonylownds mannequin assigned jackjansen Dec 7, 2001
    @tonylownds tonylownds mannequin added the OS-mac label Dec 7, 2001
    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Dec 7, 2001

    Logged In: YES
    user_id=21627

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

    @jackjansen
    Copy link
    Member

    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
    Copy link
    Member

    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
    Copy link
    Member

    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.

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant