[lazarus] Wrong note in windows: Lazarus source directory notset

Mattias Gaertner nc-gaertnma at netcologne.de
Sat Dec 20 06:23:26 EST 2003


On Sat, 20 Dec 2003 12:27:00 +0100  Vincent Snijders <vslist at zonnet.nl>
wrote:

> On Sat, 20 Dec 2003 12:00:20 +0100
> Mattias Gaertner <nc-gaertnma at netcologne.de> wrote:
> 
> > On Fri, 19 Dec 2003 22:51:45 +0100  Vincent Snijders
> > <vslist at zonnet.nl> wrote:
> > 
> > > Hi,
> > > 
> > > Lazarus gives a wrong note: Lazarus source directory not set, even
> > > if it is set correctly. The reason is a difference in behaviour of
> > > the FileExists function. 
> > > 
> > > Lazarus appends the directory seperator ('/' or '\' depending on the
> > > platform) to path given by the user. Under linux FileExists returns
> > > true for /my/lazarus/source/dir and /my/lazarus/source/dir/. Under
> > > windows c:\my\lazarus\source\dir returns true, but
> > > c:\my\lazarus\source\dir\ returns false.
> > > 
> > > IMO the most simple solution to solve this wrong not, would be
> > > storing the dir without the appended directory seperator, but I am
> > > not sure about the impact this would have for the rest of lazarus.
> > 
> > Probably none. The IDE trims and adds path delimiters as it needs. So,
> > deleting the path delimiter will neither break, nor fix it completely.
> > 
> > 
> > > There might be simplier solutions.
> > 
> > I think, the only solution is to check all fileexists if they should
> > be replaced by DirectoryExists. But there are a lot of places...
> 
> I 'll start replacing FileExists with DirectoryExists, since
> DirectoryExists doesn't get upset from a append directory separator.

Why that?
I added a ChompPathDelim to make sure.

BTW, I already replaced a few FileExists.


Mattias






More information about the Lazarus mailing list