Comment 24 for bug 415508

Revision history for this message
Martin Pool (mbp) wrote :

The doc update is in https://code.edge.launchpad.net/~mbp/bzr/doc/+merge/10638

The dirstate file does promise to hold the size of the working tree copy, presumably the size of the canonical form. However, it doesn't actually do so :-( there are multiple places where it compares it to or assigns it from the stat value size. So there's no reliable way to know the size of the canonical form other than by filtering it.

(Even if we fixed all the callers people with existing trees might keep having problems with it.)

What I'm going to do overall is: change path_content_summary so that it doesn't return the size unless it's known to be correct; in fact don't return it at all for formats that support content filtering. The change commit so that it doesn't look at the size. This should keep performance similar to what we have at present.