Re: File owner/group and git

2015-11-05 Thread David Turner
On Wed, 2015-11-04 at 18:38 -0800, Junio C Hamano wrote: > David Turner writes: > > > In unpack-trees.c, in verify_uptodate_1, we check ie_match_stat. This > > returns OWNER_CHANGED if a file has changed ownership since the index > > was updated. Do we actually care about that particular case?

Re: File owner/group and git

2015-11-04 Thread Junio C Hamano
David Turner writes: > In unpack-trees.c, in verify_uptodate_1, we check ie_match_stat. This > returns OWNER_CHANGED if a file has changed ownership since the index > was updated. Do we actually care about that particular case? Or really > anything other than DATA_CHANGED? That's a 10-year ol

File owner/group and git

2015-11-04 Thread David Turner
In unpack-trees.c, in verify_uptodate_1, we check ie_match_stat. This returns OWNER_CHANGED if a file has changed ownership since the index was updated. Do we actually care about that particular case? Or really anything other than DATA_CHANGED? (We noticed this due to a bug in our watchman code