[MERGE] Implement Tree.plan_merge, deprecate RevisionTree.get_weave

John Arbash Meinel john at arbash-meinel.com
Thu Jul 19 21:16:13 BST 2007


John Arbash Meinel has voted +1 (conditional).
Status is now: Conditionally approved
Comment:
+1 modulo the discussion between you and Robert about how to get the 
parents correct.

I think you actually want to report the parent ids as all ids except for 
the one you are merging. I believe I understand your point that if you 
have the parent_id included, then merge has nothing to do, because OTHER 
is now part of THIS.

However, it would seem that annotation-based merge when you have 
multiple merges already, should include all parents (but the current one 
you are re-evaluating).

Then again, I thought 'bzr remerge' was disabled in the case of having 
 >1 merge parents.

For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C469F8927.1080705%40utoronto.ca%3E



More information about the bazaar mailing list