Reece Dunn wrote: > On 02/09/07, Brian Gernhardt wrote: >> On Sep 2, 2007, at 12:33 PM, Reece Dunn wrote: >>> You pull a repository that contains executable scripts that are >>> required to work in order to build the system. You then make some >>> modifications to the local repository and run the 'git add .' >>> command. Since this patch is reporting executable bits >>> differently, the mode change is stored as well as the local >>> modifications. Now the changes are pushed upstream (along with >>> the file mode changes). >> >> We already have a way to tell git that the "executable bit" is >> worthless, and any Windows port should use it. > > Ok, so as the executable bit is worthless, there doesn't need to be > any special casing in this patch to deal with it. Right, this is true. And I was debating it with myself, and just added it for completion; at least for the first revision of the patch. It doesn't really affect the performance all that much anyways. The conversion of the FileTime to unix time_t is far more heavy. (Which is why I'm debating to just ignore the access time) If we could somehow rather use the FileTimes directly in the index, instead of having to convert them, we could have even better performance when stat'ing on Windows. (However, it would result in an incompatible index, so everyone would have to 'git update-index --refresh' on all repositories before they can use the new version.) -- .marius