[MERGE] Conditionalize format warnings
Matthew D. Fuller
fullermd at over-yonder.net
Sat May 26 19:31:34 BST 2007
On Fri, May 25, 2007 at 09:49:37AM -0400 I heard the voice of
Aaron Bentley, and lo! it spake thus:
> >
> > Oh, also: it looks like this is an infinite loop waiting to happen:
> >
> > Branch.open => BzrDir.open_branch => BzrDir._check_supported => Branch.open
Bah.
> It's a bit different, but I wonder whether we should move the nag to
> (Tree|Branch|Repository).lock_write, after the lock succeeds?
Very probably. That would be a "we" that doesn't include me, though;
that requires competence with the codebase.
For now, I'll suggest that I cut it back to just checking the
locations/global config so we avoid multi-opens and loops and such,
and just merging that, so we have SOME way to shut it off, pending a
fuller and cleaner solution to the larger issues.
--
Matthew Fuller (MF4839) | fullermd at over-yonder.net
Systems/Network Administrator | http://www.over-yonder.net/~fullermd/
On the Internet, nobody can hear you scream.
More information about the bazaar
mailing list