Merge lp:~markgius/bzr-extmerge/git-mergetool-support into lp:~zindar/bzr-extmerge/trunk

Proposed by Mark Gius
Status: Needs review
Proposed branch: lp:~markgius/bzr-extmerge/git-mergetool-support
Merge into: lp:~zindar/bzr-extmerge/trunk
Diff against target: 15 lines (+4/-1)
1 file modified
extmerge.py (+4/-1)
To merge this branch: bzr merge lp:~markgius/bzr-extmerge/git-mergetool-support
Reviewer Review Type Date Requested Status
Fred McCann (community) Disapprove
Jelmer Vernooij (community) code Approve
Erik Bågfors Pending
Review via email: mp+63414@code.launchpad.net

Commit message

Adds accepted configuration for git-mergetool style merging.

Description of the change

Adds accepted configuration for git-mergetool style merging.

Left is local committed copy, middle is merged copy with markers, right is remote copy. This config works extremely well with vimdiff.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Looks reasonable to me (though I'm not a bzr-extmerge maintainer)

review: Approve (code)
Revision history for this message
Fred McCann (xred) wrote :

We've already accepted a proposal that covers this use case; ("%t", "%r", "%o") will be available in the next release.

review: Disapprove

Unmerged revisions

17. By Mark Gius

Add config for git-mergetool style merges

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'extmerge.py'
--- extmerge.py 2010-07-28 14:16:55 +0000
+++ extmerge.py 2011-06-03 18:01:45 +0000
@@ -35,7 +35,10 @@
35 # meld-style 3 way merge, one file (%T) being used for both in- and output35 # meld-style 3 way merge, one file (%T) being used for both in- and output
36 ('%b', '%T', '%o'),36 ('%b', '%T', '%o'),
37 # meld-style 2 way merge, one file (%T) being used for both in- and output37 # meld-style 2 way merge, one file (%T) being used for both in- and output
38 ('%T', '%o')38 ('%T', '%o'),
39 # git mergetool style merge
40 ('%t', '%r', '%o'),
41
39)42)
4043
41mergetools = [44mergetools = [

Subscribers

People subscribed via source and target branches

to all changes: