Ping,

This issue (and mailinglist thread) is now more than two months old. The latest 
patch adds a testcase showing the issue in the APR testsuite. (There is also an 
XFail test in the subversion testsuite for this issue)

The patch was written on the 1.4.x branch but I svn switch'ed it to trunk for 
easy application.

        Bert

> -----Original Message-----
> From: Bert Huijben [mailto:b...@qqmail.nl]
> Sent: woensdag 6 januari 2010 13:45
> To: 'William A. Rowe, Jr.'; d...@apr.apache.org
> Cc: 'Philip Martin'; 'Bert Huijben'; d...@subversion.tigris.org
> Subject: RE: Windows drive letter check fails on lower case cwd
> 
> 
> 
> > -----Original Message-----
> > From: Bert Huijben [mailto:b...@qqmail.nl]
> > Sent: maandag 9 november 2009 10:05
> > To: 'William A. Rowe, Jr.'
> > Cc: 'Philip Martin'; 'Bert Huijben'; d...@apr.apache.org;
> > d...@subversion.tigris.org
> > Subject: RE: Windows drive letter check fails on lower case cwd
> >
> > > -----Original Message-----
> > > From: William A. Rowe, Jr. [mailto:wr...@rowe-clan.net]
> > > Sent: maandag 9 november 2009 4:25
> > > To: Bert Huijben
> > > Cc: 'Philip Martin'; 'Bert Huijben'; d...@apr.apache.org;
> > > d...@subversion.tigris.org
> > > Subject: Re: Windows drive letter check fails on lower case cwd
> > >
> > > Bert Huijben wrote:
> > > >
> > > > The drive letters don't have locales; the rest of the paths have.
> > > There are
> > > > only 26 driveletters with the US-ASCII characters A-Z. (Internally
> > > always
> > > > represented by the upper case letters, but the current path can use a
> > > lower
> > > > case path, as that is only managed in userspace).
> > >
> > > That's an odd statement; all paths are Unicode ;-)  We don't strcasecmp
> > > paths.
> >
> > s/path/drive in its path/ for that last path.. thanks :)
> >
> > Paths are (of course) unicode, but use a some culture sensitive compare to
> > handle the case insensitivity. But there is no documented way to find which
> > locale it uses for each (part of a) drive.
> > (That information is stored in the system portion of NTFS at format time
> and
> > can vary over directories via junctions, etc.)
> >
> > Thanks for looking into this.
> 
> After some delay I created a new patch with the suggested changes and a
> testcase for inclusion in the apr testsuite. (Note that the test can falsely
> succeed if your test environment is on C:)
> 
> I tested this patch on the 1.4.x branch and 1.3.x, but created the patch based
> on trunk. (The visual studio projects don't include the expat build, but it is
> required for a successful compilation).
> 
> As noted earlier, this issue will be experienced by far more users once
> Subversion 1.7 is released, so it would be nice if it is backported to the
> relevant releases.
> 
>       Bert

Reply via email to