winesetuptk

[Alex Pasadyn]

Vincent Béron wrote:
> Alex, I tried to make winesetuptk keep comments after a key (see first
> chunk of CBase.itcl), but I had some problems making sure all calls to
> DataAddKey had another arg named aComment. There was at least one call
> which would make it crash.
> I can revive my attempt at that, or we can just make winesetuptk eat
> those comments.

It shouldn't be too hard to preserve them.  I'd like the thing to be 
able to read my config file and write it back with only whitespace changes.

> Is an empty string {} or "" in TCL?

Well, the trouble with Tcl is that there are no real data types, so 
depending on the commands used, the contents of variables and constants 
can be interpreted different ways.  The {}, "", [list ], and undecorated 
constructions do very different things, but in some simple cases (e.g. 
null strings or strings containing only a-z,0-9), they give the same 
results.  This results in many people not bothering to learn the 
difference until they can't get their script to work (e.g. it fails only 
for certain input data).  :)  Sorry, but I've spent a lot of time 
helping people with things like that!  That said, I normally use "" to 
denote anything that I mean to be a literal string, always construct 
lists using [list ], etc, just so it's obvious what was intended even if 
I could save a few characters by doing it a different way.

> If you work on it, could you also look in how the [spooler] section is
> read/written, so we don't lose "FILE" in case we edit a file containing
> it?

Yeah, I just tried running the SourceForge version of it for the fist 
time, and I noticed the same thing with my WinMM section getting deleted.

> With both patches applied, the default config and registry look pretty
> much the same (except for some trivial things like which drive is home,
> tmp, etc.) to what wineinstall gives us. Which can lead to the question,
> what's the purpose of it if wineinstall can do it?

With the SourceForge version, I backed up my settings and let it create 
a new config file, and it was at least good enough to run some simple 
apps, and I think that's all people are looking for right now.  Now, my 
computer is Linux-only, so if there are any problems with the detection 
of Windows partitions, someone else will have to report about them.

Later tonight I'll make those changes and merge these patches from today 
together and send an update.

BTW I'm starting a new thread for some other discussion...

Alex