Implementing convert setrevmap (for p4.py)

[Patrick Mézard]

Frank A. Kingswood a écrit :
> Hallo!
> 
> Tom pointed out that convert/p4.py does not implement setrevmap and 
> therefore is not easily used incrementally.

Sources do not need to implement setrevmap() to work incrementally. The revision map is automatically filled with the revisions returned by the source during graph discovery. It is serialized so the revision graph could be updated incrementally. Two sources do additional work with the revision map:
- filemap rebuilds some state to avoid rediscovering the converted subgraph details every time
- subversion uses it to support shallow conversions (from a starting revisions)

So, if the p4 source has issues with incremental conversion, the cause is likely elsewhere. Perhaps the stored identifiers are not stable?

--
Patrick Mézard