threads / discuss / 23579

Case sensitivity on Windows for absolute paths.

Subject: Case sensitivity on Windows for absolute paths.

## tl;dr

5 messages between Apr 23, 2010 and Apr 25, 2010.

replies: 4people: 2as markdown or json

Ricky Clarkson· Apr 23, 2010, 14:12 UTC · lore
Hi,

I noticed that if I have a git repository at C:\foo, and there is a file in there named bar, git add bar works fine, git add C:\foo/bar works fine but git add c:\foo/bar does not. git reports that the file 'is outside repository'. Looking at setup.c it seems that string comparisons are being used to determine whether a file is within the repository.

For my purposes I can probably make a Windows build that does it while ignoring the case, but is there a more proper way that git should do it? I'm not familiar enough with the POSIX file routines to know.

A separate tool, maven, is what specifies it as lowercase, otherwise I would obviously just write C: instead of c: myself. I think I know how to fix that so it uses relative paths, and will look into that, but it would be good to have git do the right thing too.

Thanks, Ricky.

--
Ricky Clarkson
Java and Scala Programmer, AD Holdings
+44 1928 706373
Skype: ricky_clarkson
Google Talk: ricky.clarkson@gmail.com
Google Wave: ricky.clarkson@googlewave.com
Johannes Sixt· Apr 23, 2010, 19:46 UTC · re: Ricky Clarkson · lore

Re: Case sensitivity on Windows for absolute paths.

On Freitag, 23. April 2010, Ricky Clarkson wrote:
Show 6 quoted lines
> I noticed that if I have a git repository at C:\foo, and there is a
> file in there named bar, git add bar works fine, git add C:\foo/bar
> works fine but git add c:\foo/bar does not.  git reports that the file
> 'is outside repository'.  Looking at setup.c it seems that string
> comparisons are being used to determine whether a file is within the
> repository.

Yes, we should be a bit more liberal when drive letters are compared; they should be treated in a case-insensitive manner, but currently we are strict.

> For my purposes I can probably make a Windows build that does it while
> ignoring the case, but is there a more proper way that git should do
> it?  I'm not familiar enough with the POSIX file routines to know.
My guess is that it is sufficient that
1. the internal version of GIT_DIR is recorded with an uppercase drive letter;
2. normalize_path_copy() upper-cases the drive letter.
Then the comparison in prefix_path() should do the right thing.
-- Hannes
Ricky Clarkson· Apr 25, 2010, 07:33 UTC · re: Johannes Sixt · lore

Re: Case sensitivity on Windows for absolute paths.

Hi again,

It would be good if git automatically uppercased the drive letter, yes. However, I found the real root of my problem. Git Bash (msysgit) sets the working directory to c:\foo for programs launched if the working directory within it is /c/foo. Like having spaces in filenames, that just uncovers a whole myriad of small bugs and oddities in other programs.

For reference, Cygwin doesn't have that problem; cd /cygdrive/c/foo && cmd, cmd gets C:\foo as its working directory.

So to make what I wanted work, I just made sure everything msysgit installed be on the PATH, then ran everything from cmd.

I hope that this helps the next person who experiences this problem.

Thanks, Ricky.

--
Ricky Clarkson
Java and Scala Programmer, AD Holdings
+44 1928 706373
Skype: ricky_clarkson
Google Talk: ricky.clarkson@gmail.com
Google Wave: ricky.clarkson@googlewave.com
On 23 April 2010 20:46, Johannes Sixt <j6t@kdbg.org> wrote:
Show 25 quoted lines
> On Freitag, 23. April 2010, Ricky Clarkson wrote:
>> I noticed that if I have a git repository at C:\foo, and there is a
>> file in there named bar, git add bar works fine, git add C:\foo/bar
>> works fine but git add c:\foo/bar does not.  git reports that the file
>> 'is outside repository'.  Looking at setup.c it seems that string
>> comparisons are being used to determine whether a file is within the
>> repository.
>
> Yes, we should be a bit more liberal when drive letters are compared; they
> should be treated in a case-insensitive manner, but currently we are strict.
>
>> For my purposes I can probably make a Windows build that does it while
>> ignoring the case, but is there a more proper way that git should do
>> it?  I'm not familiar enough with the POSIX file routines to know.
>
> My guess is that it is sufficient that
>
> 1. the internal version of GIT_DIR is recorded with an uppercase drive letter;
>
> 2. normalize_path_copy() upper-cases the drive letter.
>
> Then the comparison in prefix_path() should do the right thing.
>
> -- Hannes
>
Johannes Sixt· Apr 25, 2010, 18:08 UTC · re: Ricky Clarkson · lore

Re: Case sensitivity on Windows for absolute paths.

On Sonntag, 25. April 2010, Ricky Clarkson wrote:
> It would be good if git automatically uppercased the drive letter,
> yes.  However, I found the real root of my problem.  Git Bash
> (msysgit) sets the working directory to c:\foo for programs launched
> if the working directory within it is /c/foo.

I don't understand what you mean. Git Bash cannot influence whether git sees the drive letter in the working directory in uppercase or lowercase. Unless by "working directory" you mean something else than what getcwd() returns.

-- Hannes
Ricky Clarkson· Apr 25, 2010, 18:47 UTC · re: Johannes Sixt · lore

Re: Case sensitivity on Windows for absolute paths.

Hi Johannes,
$ cd /c/uk-org-netvu/

Ricky@RICKY-XP /c/uk-org-netvu (trunk) $ cmd Microsoft Windows XP [Version 5.1.2600] (C) Copyright 1985-2001 Microsoft Corp.

c:\uk-org-netvu>
If I run cmd via Start -> Run, then the prompt shows C:\uk-org-netvu>

I also verified this using the Scala interpreter, printing out System.getProperty("user.dir"), which varied similarly in case depending on whether I started it from cmd or from Git Bash.

Thanks, Ricky.

--
Ricky Clarkson
Java and Scala Programmer, AD Holdings
+44 1928 706373
Skype: ricky_clarkson
Google Talk: ricky.clarkson@gmail.com
Google Wave: ricky.clarkson@googlewave.com
On 25 April 2010 19:08, Johannes Sixt <j6t@kdbg.org> wrote:
Show 12 quoted lines
> On Sonntag, 25. April 2010, Ricky Clarkson wrote:
>> It would be good if git automatically uppercased the drive letter,
>> yes.  However, I found the real root of my problem.  Git Bash
>> (msysgit) sets the working directory to c:\foo for programs launched
>> if the working directory within it is /c/foo.
>
> I don't understand what you mean. Git Bash cannot influence whether git sees
> the drive letter in the working directory in uppercase or lowercase. Unless
> by "working directory" you mean something else than what getcwd() returns.
>
> -- Hannes
>

← back to recent threads