Re: [PATCH 1/2] run_diff_{files,index}(): update calling convention.

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: [PATCH 1/2] run_diff_{files,index}(): update calling convention.

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:42:53

I suspect that you are dual booting and browsing a git
repository that is on a read-only mounted NTFS filesystem from
the Linux side, and the index was created on Cygwin git?

I (perhaps luckily) am fairly ignorant on the way things done in
Windows environment.  For one thing, I do not know if NTFS has
notion of i-number, file owner uid, and other information that
are used in the index (not that I want to know).

If NTFS does not support the information returned by lstat(2)
fully on disk, I would imagine Cygwin and NTFS filesystem driver
in the Linux kernel need to fake some fields that NTFS does not
natively store, and if the value faked by Cygwin and NTFS driver
in the Linux kernel disagree, then it is not at all surprising
to see if an unmodified path shows up as cache-dirty.

Re: [PATCH 1/2] run_diff_{files,index}(): update calling convention.

From: Marco Costalba <hidden>
Date: 2016-06-15 22:42:53

On 2/10/07, Junio C Hamano [off-list ref] wrote:
I suspect that you are dual booting and browsing a git
repository that is on a read-only mounted NTFS filesystem from
the Linux side, and the index was created on Cygwin git?
Yes. That's it.
I (perhaps luckily) am fairly ignorant on the way things done in
Windows environment.  For one thing, I do not know if NTFS has
notion of i-number, file owner uid, and other information that
are used in the index (not that I want to know).

If NTFS does not support the information returned by lstat(2)
fully on disk, I would imagine Cygwin and NTFS filesystem driver
in the Linux kernel need to fake some fields that NTFS does not
natively store, and if the value faked by Cygwin and NTFS driver
in the Linux kernel disagree, then it is not at all surprising
to see if an unmodified path shows up as cache-dirty.
So in this case your patch that introduce '--refresh' option in 'git
runstatus' is not just a shortcut for 'git update-index' + 'git
runstatus' but adds some real value.

One more reason for asking you to add it before 1.5 release ;-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help