[Monotone-commits-diffs] Revision bb0cfea82580c21c64a139a5fe2136496a6f78b2

[monotone]

-----------------------------------------------------------------
Revision: bb0cfea82580c21c64a139a5fe2136496a6f78b2
Ancestor: 21a6b294b1e07de2a98ae917f0a9443284e92d8e
Author: address@hidden
Date: 2008-04-29T20:16:33
Branch: net.venge.monotone.cvsimport-branch-reconstruction

Added files:
        tests/importing_cvs_cycle_splitter3/__driver__.lua
        tests/importing_cvs_cycle_splitter3/cvs-repository/test/file1,v
        tests/importing_cvs_cycle_splitter3/cvs-repository/test/file2,v
Added directories:
        tests/importing_cvs_cycle_splitter3
        tests/importing_cvs_cycle_splitter3/cvs-repository
        tests/importing_cvs_cycle_splitter3/cvs-repository/test
Modified files:
        rcs_import.cc

ChangeLog: 

 * rcs_import.cc: prevent tag or branch_end blob types from sneaking into
   dependency cycles. No other events ever depend on any of those, so they
   should clearly never be part of a cycle.

   Remember symbol blobs, if they occur in a dependency cycle, however, at
   the moment we just emit a warning. We should favor breaking the cycle
   at the symbol, since those are the least reliable in most CVS repos.

   Prevent the cycle splitter from wanting to split blobs of size 1. Those
   cannot be split anymore for obvious reasons.

   Turned a warning back into an invariant: every blob in the cycle must
   have at least *one* dependency on at least *one* other blob in the
   cycle, otherwise we wouldn't have a cycle at the first place.

 * tests/importing_cvs_cycle_splitter3/*: added another cycle splitting
   test, which tests breaking cycles which include branch starts.

txtzCKUMX_xC9.txt
Description: Diff [21a6b294b1e07de2a98ae917f0a9443284e92d8e] -> [bb0cfea82580c21c64a139a5fe2136496a6f78b2]