Re: [PATCH] Add a new lstat implementation based on Win32 API, and make stat use that implementation too.

From: Robin Rosenberg <robin.rosenberg.lists@dewire.com>
Date: Sep 2, 2007, 21:42 UTC
Message-ID: <200709022342.42733.robin.rosenberg.lists@dewire.com>
In-Reply-To: <Pine.LNX.4.64.0709022133190.28586@racer.site>

söndag 02 september 2007 skrev Johannes Schindelin:

Show 10 quoted lines

> Hi,
> 
> On Sun, 2 Sep 2007, Robin Rosenberg wrote:
> 
> > s?ndag 02 september 2007 skrev Marius Storm-Olsen:
> > > (Also, since Windows doesn't really handle symlinks, it's fine that stat just uses lstat)
> > 
> > It does now: See http://msdn2.microsoft.com/en-us/library/aa363866.aspx
> 
> Oh?  *goes and tries to create one on a USB stick* No.  Besides, IIRC you

Set core.symlinks = false for working on a broken file system

> cannot even create symlinks to another partition.  Copying a symlink will

That is not the normal use for symlinks. It is a case where it breaks, but symbolic links in a git repo that points outside the repo is probably not a good idea, especially if it is a cross platform project. It is far less broken than today anyway.

> copy the _linked_ file.  So to call this "symlink" is a little... uhm... 
> preposterous.

$ ln -s Makefile x $ cp x y $ ls -ld x y lrwxrwxrwx 1 me me 8 sep 2 23:36 x -> Makefile -rw-r--r-- 1 me me 32164 sep 2 23:36 y

Same behaviour as on linux.

> Plus, on a page linked from the link you posted, it says that it is 
> only supported from Vista onwards.  So you must be kidding me.

core.symlinks = false if ithey aren't supported. You actually need admin privileges too, but I don't know any windows developer who hasn't got that.

-- robin