<div dir="ltr">Hi Matt,<br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jul 11, 2015 at 1:25 AM, Matt Mackall <span dir="ltr"><<a href="mailto:mpm@selenic.com" target="_blank">mpm@selenic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, 2015-07-10 at 20:10 +0200, Erik Huelsmann wrote:<br>
> # HG changeset patch<br>
> # User Erik Huelsmann <<a href="mailto:ehuels@gmail.com">ehuels@gmail.com</a>><br>
> # Date 1435346089 -7200<br>
> #      Fri Jun 26 21:14:49 2015 +0200<br>
> # Node ID c8b0c9fb18ec813244b100e7ecf9ee5eb7b92f88<br>
> # Parent  ff5172c830022b64cc5bd1bae36b2276e9dc6e5d<br>
> merge: add support for 'union' merge strategy to internal merge tool.<br>
><br>
> 'union' merge is a merge strategy where the left and right side of a<br>
> conflicting merge are merged into the target without generating a conflict.<br>
><br>
> One use-case for this merge strategy is the Changelog file being changed<br>
> on multiple branches and conflicting when being merged back to the main<br>
> branch.<br>
><br>
> The idea for this merge strategy has been taken from Git.<br>
<br>
</span>This looks ok, but it's a bit much going on in one patch by our<br>
standards. Could I get you to split it up into the following pieces:<br>
<br>
- bits that add start/end_marker (but please, no _ in new names)<br>
- bits that union option to simplemerge<br>
- bits that split out filemerge helper function<br>
- bits that add union merge + test of same<br></blockquote><div><br></div><div>Ok. I can split it up that way, no problem. Just verifying before diving in: are you expecting it as a set of patches ('1 of 4'...)? Each with the same commit text as I provided above? Or are you expecting the last patch with the commit text as above, and the others with a shorter description like "In preparation of implementing union merge, rearrange ...."?</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, we may want the start/end marker business to be fully internal to<br>
simplemerge rather than passing in more args.<br></blockquote><div><br></div><div>Consider it done! :-)</div><div><br></div><div>I'm a bit spotty on time availability, but I'll get to resubmitting the patches; hopefully without too much delay.</div><div><br></div><div>Thanks for the prompt response!</div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Bye,<div><br></div><div>Erik.</div><div><br></div><div><a href="http://efficito.com/" target="_blank">http://efficito.com</a> -- Hosted accounting and ERP.</div><div>Robust and Flexible. No vendor lock-in.</div></div></div>
</div></div>