comparison tests/test-wireproto-exchangev2.t @ 40924:08cfa77d7288

wireprotov2: unify file revision collection and linknode derivation

The old mechanism for choosing which file revisions to send in the
haveparents=True case was buggy in multiple ways - the most severe
of which being that file revisions were excluded when they shouldn't
have been.

This commit unifies the logic for deriving the filenodes that will
be sent by the "filesdata" command. We now consistently read files
data from manifests. The "haveparents" argument now controls whether
we iterate ctx.files() or use the full manifest to derive relevant
files.

The logic here is still woefully lacking to fully support shallow
clones. It will require an API break to fully address. This commit
should at least make the server APIs emit proper data, which is
strictly better than before.

Differential Revision: https://phab.mercurial-scm.org/D5406

40923:3ed77780f4a6

  new changesets 639c8990d6a5
  (run 'hg update' to get a working copy)

#if reporevlogstore
  $ hg -R client-linknode-2 debugrevlogindex dupe-file
  abort: revlog 'dupe-file' not found
  [255]
#endif

  $ hg -R client-linknode-2 verify
  checking changesets
  checking manifests
  crosschecking files in changesets and manifests
  checking files
   warning: revlog 'data/dupe-file.i' not in fncache!
   2: empty or missing dupe-file
   dupe-file@2: manifest refers to unknown revision 2ed2a3912a0b
  checked 3 changesets with 2 changes to 3 files
  1 warnings encountered!
  hint: run "hg debugrebuildfncache" to recover from corrupt fncache
  2 integrity errors encountered!
  (first damaged changeset appears to be 2)
  [1]

40924:08cfa77d7288

  new changesets 639c8990d6a5
  (run 'hg update' to get a working copy)

#if reporevlogstore
  $ hg -R client-linknode-2 debugrevlogindex dupe-file
     rev linkrev nodeid       p1           p2
       0       2 2ed2a3912a0b 000000000000 000000000000
#endif