Graeme Geldenhuys schreef:
2009/6/4 Vincent Snijders <vsnijd...@vodafonevast.nl>:
Maybe we should create a GetAppConfigDir branche for Graeme and let him
develop all the needed changes.
My custom branch is available on GitHub, including the upstream
(untouched clean) branch. Note though, that I have not pushed my
custom changes to GitHub in a few weeks.
How can I extract the patch with diff that I can apply to create
snapshots for people to test and find bugs? After all I think this code
is too dangerous to keep it in trunk. A patch must be reverted quickly
if it turns out to break too many things.
moving primary config paths to paths with spaces and special (non-ascii)
characters.
And here I thought Lazarus with utf-8 support can handle something as
simple as that without blinking an eye. Was I wrong in that
assumption? If I was, then I would seriously start worrying if I was
a LCL user.
Lazarus yes, the win32 RTL unfortunately not so easily.
That is the less concise version of "if it isn't broken, then don't fix it".
Well it just amazes me that some poor developer went through all the
trouble in removing the hard-code "c:\lazarus" primary config
directory to use GetAppConfigDir(false), yet the other platforms are
still stuck with the old behavior. That might be an indicator to the
Lazarus core developers, that the right hand doesn't know what the
left hand is going, and that the various platforms might get even more
fragmented and fragile over time.
No, the poor programmer replaces ExtractFilePath(ParamStr(0)) with
GetAppConfigDir(false) and then found out, that that doesn't work
correctly. LazConf is used by Lazarus and LazBuild and StartLazarus, so
GetAppConfigDir doesn't really work out of the box. I bet you didn't
think about that when you just proposed to make things easier.
Anyway, I rather work on something more useful than reducing the code in
lazbaseconf.inc and debugging it.
Vincent
_______________________________________________
Lazarus mailing list
Lazarus@lists.lazarus.freepascal.org
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus