[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[Patch] RE: Windows drive letter check fails on lower case current working drive

From: Bert Huijben <bert_at_qqmail.nl>
Date: Tue, 19 Jan 2010 12:19:28 +0100

        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:bert_at_qqmail.nl]
> Sent: woensdag 6 januari 2010 13:45
> To: 'William A. Rowe, Jr.'; dev_at_apr.apache.org
> Cc: 'Philip Martin'; 'Bert Huijben'; dev_at_subversion.tigris.org
> Subject: RE: Windows drive letter check fails on lower case cwd
>
>
>
> > -----Original Message-----
> > From: Bert Huijben [mailto:bert_at_qqmail.nl]
> > Sent: maandag 9 november 2009 10:05
> > To: 'William A. Rowe, Jr.'
> > Cc: 'Philip Martin'; 'Bert Huijben'; dev_at_apr.apache.org;
> > dev_at_subversion.tigris.org
> > Subject: RE: Windows drive letter check fails on lower case cwd
> >
> > > -----Original Message-----
> > > From: William A. Rowe, Jr. [mailto:wrowe_at_rowe-clan.net]
> > > Sent: maandag 9 november 2009 4:25
> > > To: Bert Huijben
> > > Cc: 'Philip Martin'; 'Bert Huijben'; dev_at_apr.apache.org;
> > > dev_at_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
Received on 2010-01-19 12:20:07 CET

This is an archived mail posted to the Subversion Dev mailing list.