Code review comment for lp:~jelmer/bzr/get-known-graph-ancestry

Revision history for this message
Robert Collins (lifeless) wrote :

On Sat, 2010-04-10 at 01:26 +0000, Jelmer Vernooij wrote:
> Jelmer Vernooij has proposed merging lp:~jelmer/bzr/get-known-graph-ancestry into lp:bzr.

Just to say, I've got reservations about this - we want to shrink
Repository IMO, not expand it. Why can't bzr-svn just expose a VF object
- even if its slow if used for untuned things, it would work, wouldn't
it? And avoid the duplication that this adds.

 review: disapprove

review: Disapprove

« Back to merge proposal