# Cygwin git and windows network shares 6 messages from 2006-06-16 to 2006-06-17. Participants: Niklas Frykholm, Juergen Ruehle, Christopher Faylor. Thread: https://gitlist.dev/t/4531 ## Niklas Frykholm, 2006-06-16 12:58 Subject: Cygwin git and windows network shares Message-ID: <4492AAFA.20807@grin.se> URL: https://gitlist.dev/e/4492AAFA.20807%40grin.se ``` I'm trying to use cygwin git (compiled from the 1.4.0 tarball) to create repository on a windows network share, but I get an error message. $ cd //computer/git/project $ git init-db defaulting to local storage area Could not rename the lock file? The repository seems to be left in an inconsistent state after this: $ git clone //computer/git/project/ fatal: no matching remote head fetch-pack from '//computer/git/project/.git' failed. When working only with local files, I do not get these errors. Does anyone know the cause of this error/any way around it? // Niklas ``` ## Juergen Ruehle, 2006-06-16 14:24 Subject: Re: Cygwin git and windows network shares Message-ID: <17554.48926.852000.679014@lapjr.intranet.kiel.bmiag.de> URL: https://gitlist.dev/e/17554.48926.852000.679014%40lapjr.intranet.kiel.bmiag.de In-Reply-To: <4492AAFA.20807@grin.se> ``` Niklas Frykholm writes: > I'm trying to use cygwin git (compiled from the 1.4.0 tarball) to create > repository on a windows network share, but I get an error message. > > $ cd //computer/git/project > $ git init-db > defaulting to local storage area > Could not rename the lock file? cygwin's rename seems to be capable of overwriting an existing target only on NTFS. The following hack is a workaround, but is probably not safe. diff --git a/lockfile.c b/lockfile.c index 2346e0e..5e78211 100644 --- a/lockfile.c +++ b/lockfile.c @@ -48,6 +48,7 @@ int commit_lock_file(struct lock_file *l strcpy(result_file, lk->filename); i = strlen(result_file) - 5; /* .lock */ result_file[i] = 0; + unlink(result_file); i = rename(lk->filename, result_file); lk->filename[0] = 0; return i; ``` ## Christopher Faylor, 2006-06-16 22:10 Subject: Re: Cygwin git and windows network shares Message-ID: <20060616221014.GA22181@trixie.casa.cgf.cx> URL: https://gitlist.dev/e/20060616221014.GA22181%40trixie.casa.cgf.cx In-Reply-To: <17554.48926.852000.679014@lapjr.intranet.kiel.bmiag.de> ``` On Fri, Jun 16, 2006 at 04:24:30PM +0200, Juergen Ruehle wrote: >Niklas Frykholm writes: > > I'm trying to use cygwin git (compiled from the 1.4.0 tarball) to create > > repository on a windows network share, but I get an error message. > > > > $ cd //computer/git/project > > $ git init-db > > defaulting to local storage area > > Could not rename the lock file? > >cygwin's rename seems to be capable of overwriting an existing target >only on NTFS. The following hack is a workaround, but is probably not >safe. Actually, Cygwin's rename has a specific check to make sure that the file is deleted. It tries very hard to do things the right way but if your samba server doesn't return the correct error code then it is possible that it could be confused. cgf ``` ## Christopher Faylor, 2006-06-16 22:30 Subject: Re: Cygwin git and windows network shares Message-ID: <20060616223006.GA23737@trixie.casa.cgf.cx> URL: https://gitlist.dev/e/20060616223006.GA23737%40trixie.casa.cgf.cx In-Reply-To: <20060616221014.GA22181@trixie.casa.cgf.cx> ``` On Fri, Jun 16, 2006 at 06:10:14PM -0400, Christopher Faylor wrote: >On Fri, Jun 16, 2006 at 04:24:30PM +0200, Juergen Ruehle wrote: >>Niklas Frykholm writes: >> > I'm trying to use cygwin git (compiled from the 1.4.0 tarball) to create >> > repository on a windows network share, but I get an error message. >> > >> > $ cd //computer/git/project >> > $ git init-db >> > defaulting to local storage area >> > Could not rename the lock file? >> >>cygwin's rename seems to be capable of overwriting an existing target >>only on NTFS. The following hack is a workaround, but is probably not >>safe. > >Actually, Cygwin's rename has a specific check to make sure that the >file is deleted. It tries very hard to do things the right way but if >your samba server doesn't return the correct error code then it is >possible that it could be confused. >diff --git a/lockfile.c b/lockfile.c >index 2346e0e..5e78211 100644 >--- a/lockfile.c >+++ b/lockfile.c >@@ -48,6 +48,7 @@ int commit_lock_file(struct lock_file *l > strcpy(result_file, lk->filename); > i = strlen(result_file) - 5; /* .lock */ > result_file[i] = 0; >+ unlink(result_file); > i = rename(lk->filename, result_file); > lk->filename[0] = 0; > return i; I also meant to ask if there was an i is nonzero after the call to the rename() above? If so, what was the errno? If not, it is a huge problem if rename is reporting that it is able to rename a file but is not actually doing it. cgf (cygwin maintainer) ``` ## Juergen Ruehle, 2006-06-17 01:02 Subject: Re: Cygwin git and windows network shares Message-ID: <17555.21646.398000.619804@lapjr.intranet.kiel.bmiag.de> URL: https://gitlist.dev/e/17555.21646.398000.619804%40lapjr.intranet.kiel.bmiag.de In-Reply-To: <17554.48926.852000.679014@lapjr.intranet.kiel.bmiag.de> ``` Juergen Ruehle writes: > The following hack is a workaround, but is probably not safe. Please just ignore it, it only helps on FAT32 partitions. Sorry for the confusion. jr ``` ## Christopher Faylor, 2006-06-17 01:23 Subject: Re: Cygwin git and windows network shares Message-ID: <20060617012338.GB23924@trixie.casa.cgf.cx> URL: https://gitlist.dev/e/20060617012338.GB23924%40trixie.casa.cgf.cx In-Reply-To: <17555.21858.292000.494454@lapjr.intranet.kiel.bmiag.de> ``` On Sat, Jun 17, 2006 at 03:05:38AM +0200, Juergen Ruehle wrote: >Christopher Faylor writes: > > I also meant to ask if there was an i is nonzero after the call to the > > rename() above? If so, what was the errno? If not, it is a huge > > problem if rename is reporting that it is able to rename a file but is > > not actually doing it. > >After some testing the conclusion is that it's not lying, but only >failing in interesting ways on my (and seemingly also Niklas') system: > > - rename on NTFS succeeds (and returns 0) > > - rename on FAT32 succeeds if target does not exist (and returns 0) > > rename on FAT32 fails if target exists with EACCESS > > (manual mv on commandline works) > > - rename on a network share always hangs for a while and then fails > with EBUSY (even if target does not exist) > > (share served by XP, tested both NTFS and FAT32) > > (manual mv still works) > >Various combinations of server, ntsec, and smbntsec didn't seem to >make a difference; /etc/{passwd,group} have been freshly created. Thanks. I really appreciate the details. I've passed them on to the last person to touch the rename code. cgf ```