Re: kitgen with packages - KBS

On 12 Apr 2007, at 21:40, rene wrote:

> I tried with 8.5 and found:
>
> - thread is missing in 8.x directory after "make tars" -> I used the
> old one

Oops.  I've been pondering whether to include thread as static  
extension with tclkit whenever threads are enabled, or to include it  
as dynamic extension since it is not essential for tclkit use.  Am  
inclined to leave it out of the exe (and then add it optionally back  
into vfs as shared lib, this is a bit the same issue as with incrtcl).

Are there any compelling arguments either way?

> - tk does not compile because of missing generic/ttk directory -> I
> used the HEAD version from tcl and tk

Ah, yes the /pub/tk/tars/tk.tar.gz file is quite old - I reall should  
update them...

> - tk does not link because of missing "-lXft" -> added to Linux switch
> GUI_OPT=..

I suppose I can add this for all cases (even 8.4 which does not use  
it), or would I have to special-case for the 8.5 builds?

> Then I found a bug with german "Umlauts". Try the following script:
>
> package req Tk
> proc main {} {
>   button .b -text "abc 123 üöä ÜÖÄ #"
>   grid .b
> }
> main

Thanks.  Also fails in 8.4 on Mac OS X.  Weird - it works with the  
above as script, just not wrapped.  Must be an encoding issue, but I  
see no diff between how mkclvfs and mk4vfs use memchan (both fill it  
in binary mode and then read the data in auto mode).

The following change in mkclvfs.tcl seems to solve it, from:
     set fd [::vfs::memchan]
     fconfigure $fd -translation binary
     puts -nonewline $fd $contents
     fconfigure $fd -translation auto
     seek $fd 0
to:
     set fd [::vfs::memchan]
     fconfigure $fd -translation binary
     puts -nonewline $fd $contents
     fconfigure $fd -translation auto -encoding [encoding system]
     seek $fd 0

But if this does what I think it does, then I don't understand why  
things ever worked before...

Andreas, can you shed light on this?  Did anything change in  
fconfigure?  (note that this is the memchan emulation based on rchan,  
not your original code)

Anyway, I'll check in the patch - an updated vlerq.tgz will be on the  
equi4 site tomorrow.

-jcw