On Mon, Jun 21, 2010 at 05:16:27PM -0400, Chris Ball wrote: > Hi Trevor, > > >> I'm surprised there still isn't an upstream bzr fix. We might > >> have to think up a workaround > > > My favorite workaround is: > > 1) disable recursive remove test > > 2) hope nothing else triggers bzr race > > Already tried that. :) Standard (non-recursive) ticket deletion hit > it too. > > .. sure enough, a simple "be remove <id>" hangs, so it's not a > testsuite artifact. Suck.
Drat. I'll look into what I changed in that portion of code... > Oh, also, I see the following output post-merge: > > % ./be list > Multiple paths for eff20807-07f0-444d-8992-f69ab3f526c5: > .be/bea86499-824e-4e77-b085-2d581fa9ccab/bugs/597a7386-643f-4559-8dc4-68719 24229b6/comments/eff20807-07f0-444d-8992-f69ab3f526c5 > ... > bea/12c:uw: Bug aggregation. Multi-repo meta-BE? > ... > > The multiple paths warning is new That's because the Storage backend flattens everything to a single level. The UUIDs should be unique without having to be nested. The .be directory, however, is not flat (it should be though, maybe once I switch to btrfs ;), and Storage gives collision warnings when its doing the flattening. They'll only show up on the first run when you on a 'Bugs Everywhere Directory v1.4' that previously existed as a 'Bugs Everywhere Directory v1.3'. The duplicates are the result of the old "copy UUID" algorithm used by be merge The new implementation generates new UUIDs for the copied comments, storing the old UUID in .alt_id if its empty. If its full, we just drop the old UUID from the copied comment. > as is the "bea/" prefix. That's me moving towards multi-repo integration. To allow cross-referenceing between repositories (along the lines of Sphinx' intersphinx http://sphinx.pocoo.org/ext/intersphinx.html ). See bea// for details. Note that you don't need to write out the full id, just enough to avoid ambiguity. So if BE only knows about the local repo, bea86499-824e-4e77-b085-2d581fa9ccab/597a7386-643f-4559-8dc4-6871924229b6/eff20807-07f0-444d-8992-f69ab3f526c5 bea/597a/eff /597a/eff /597a/e are all equivalent. Note that you can't abbreviate 597a further, because there is also a /597b. See libbe.util.id docstrings for more details on the ID format. -- This email may be signed or encrypted with GPG (http://www.gnupg.org). The GPG signature (if present) will be attached as 'signature.asc'. For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy My public key is at http://www.physics.drexel.edu/~wking/pubkey.txt
pgpRVGTyD35Zy.pgp
Description: PGP signature
_______________________________________________ Be-devel mailing list [email protected] http://void.printf.net/cgi-bin/mailman/listinfo/be-devel
