Merge lp:~jelmer/brz/merge-lca-regression into lp:brz

Proposed by Jelmer Vernooij on 2018-03-24
Status: Merged
Approved by: Jelmer Vernooij on 2018-03-24
Approved revision: 6924
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/merge-lca-regression
Merge into: lp:brz
Diff against target: 25 lines (+5/-3)
1 file modified
breezy/merge.py (+5/-3)
To merge this branch: bzr merge lp:~jelmer/brz/merge-lca-regression
Reviewer Review Type Date Requested Status
Martin Packman 2018-03-24 Approve on 2018-03-24
Review via email: mp+342049@code.launchpad.net

Commit message

Fix regression dealing with LCA merges.

Description of the change

Fix regression dealing with LCA merges.

paths is set to ((base_path, [lca_path1, lca_path2, ...], this_path, other_path),
whereas _dump_conflicts() takes a 3-tuple of paths.

To post a comment you must log in.
Martin Packman (gz) wrote :

Thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'breezy/merge.py'
2--- breezy/merge.py 2018-03-24 11:21:54 +0000
3+++ breezy/merge.py 2018-03-24 18:33:38 +0000
4@@ -1356,8 +1356,9 @@
5 self.tt.unversion_file(trans_id)
6 # This is a contents conflict, because none of the available
7 # functions could merge it.
8- file_group = self._dump_conflicts(name, paths, parent_id, file_id,
9- set_version=True)
10+ file_group = self._dump_conflicts(
11+ name, (base_path, other_path, this_path), parent_id,
12+ file_id, set_version=True)
13 self._raw_conflicts.append(('contents conflict', file_group))
14 elif hook_status == 'success':
15 self.tt.create_file(lines, trans_id)
16@@ -1368,7 +1369,8 @@
17 self._raw_conflicts.append(('text conflict', trans_id))
18 name = self.tt.final_name(trans_id)
19 parent_id = self.tt.final_parent(trans_id)
20- self._dump_conflicts(name, paths, parent_id, file_id)
21+ self._dump_conflicts(
22+ name, (base_path, other_path, this_path), parent_id, file_id)
23 elif hook_status == 'delete':
24 self.tt.unversion_file(trans_id)
25 result = "deleted"

Subscribers

People subscribed via source and target branches