[Patches] [ python-Patches-410231 ] make setup.py do dynamic Tk extensions

noreply@sourceforge.net noreply@sourceforge.net
Tue, 20 Mar 2001 21:02:33 -0800


Patches item #410231, was updated on 2001-03-20 21:02
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=410231&group_id=5470

Category: Tkinter
Group: None
Status: Open
Priority: 5
Submitted By: Internet Discovery (idiscovery)
Assigned to: Nobody/Anonymous (nobody)
Summary: make setup.py do dynamic Tk extensions

Initial Comment:
In patch [#409043] Make _tkinter.c Tix aware, Guido
wrote

 Now that I've seen the flurry of patches related to 
 Tix, I prefer to do this after 2.1 is safe and well
 released.  We have 2 days until the code freeze for
 the 2.1b2 release, and that's too short to make
 sure this stuff all works correctly.  Certainly *I*
 don't have time to review it at all.

Sorry for the flurry, but it helped me seperate the
different issues with Martin Loewis's comments. He
argued that Tix, and all Tk extensions, should be
loaded dynamically, using tcl's [package require].
I agree, and this is fundamentally consistent with
the approach of setup.py. I have unpdated Tix.py
accordingly, and it is now standalone and dynamic.

The problem is that setup.py is not internally
consistent, because even when it is building Tcl/Tk
dynamically, it then tries to add WITH_* to build
the extensions *statically*.

It may be *vital* to have Tix included in 2.1b2 to act
as a testbed for building Tk extensions dynamically.
With setup.py you're trying to build dynamically, but
with Modules/Setup you're trying to build statically,
and Tools/freeze *requires* static build, etc. There
are a number of areas where the documentation (/README
/Tools/freeze/README) and code comments (Modules/Setup,
tclappinit.c, _tkinter.c, Tools/freeze/freeze.py) need
improving to explain static vs. dynamic issues, and
having Tix in 2.1b2 will help flush these issues out. 

I'll close out the flurry of other patches - this file
replaces them all. You may have to delete the files
associated with my other patches as I don't have
permission. No matter what *please* update all
tix4.1.8.0 to tix8.1.8.2 in 2.0.1 and 2.1 as tix4.1.8.0
was never an official release, and most Linux releases 
are now distributing tix8.1.8.x. 



----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=410231&group_id=5470