[PATCH] dirstate: rebuild should update dirstate properly

Yuya Nishihara yuya at tcha.org
Thu Sep 29 11:25:57 EDT 2016

On Tue, 27 Sep 2016 14:06:24 +0000, Mateusz Kwapich wrote:
> On rebuild all the files are set to match the files in the revision but the revision doesn’t provide
> us the correct mtimes to set. I’d argue it’s better to set the mtimes to value explicitly meaning
> “unknown” than to a prepared arbitrary value.
> Same for file mode: why do we even try to guess the file mode on disk?

No idea why. I've checked b304c2496f52, which introduced the test, and
a8f7791e3680, which introduced dirstate.rebuild(). What I can guess from
them is we would set file modes because it was easy at that time.

So I lean toward taking this patch. Is V2 the latest one?


More information about the Mercurial-devel mailing list