<div class="gmail_quote">On Fri, Jun 3, 2011 at 12:21 PM, Mikael Pettersson <span dir="ltr"><<a href="mailto:mikpe@it.uu.se" target="_blank">mikpe@it.uu.se</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>p.s. Thank you for inlining the patch and providing a rationale, most git</div>
users on this list don't do that which makes peer review difficult.<br>
<div><div></div><div> </div></div></blockquote><div><br></div><div>If inline patches are preferred, perhaps the following recommendations should be revised to say so more clearly:</div><div><br></div></div>
<blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div class="gmail_quote"><div><a href="https://github.com/erlang/otp/wiki/submitting-patches" target="_blank">https://github.com/erlang/otp/wiki/submitting-patches</a></div>

</div></blockquote><div class="gmail_quote"><div><a href="https://github.com/erlang/otp/wiki/submitting-patches" target="_blank"></a><br></div><div>Right now, the recommendations on that page begin with the strong suggestion that the Right Thing is to email a reference to the Git branch that represents the proposed patches:</div>

<div><br></div><div><span style="font-family:helvetica, arial, freesans, clean, sans-serif;line-height:23px"><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204, 204, 204);border-left-style:solid;padding-left:1ex">

The recommended way of sending patches is described on this page. Basically, you push your changes to a git repository and send us an email with a reference to your public git repository and branch.... If you send us a patch in other ways than described on this page, it will mean more work for us and we cannot guarantee anything. It is up to the maintainer for that part of <span style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px;line-height:1.4em">OTP</span> to decide whether (s)he will pick up the patch.</blockquote>

<div style="font-size:14px"><br></div></span></div><div>It is only much later in the document, in the "Sending the patch" section, that readers learn that inline patches are acceptable at all, but even then it's only via a parenthetical comment:</div>

<div><br></div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204, 204, 204);border-left-style:solid;padding-left:1ex">

<span style="font-family:helvetica, arial, freesans, clean, sans-serif;line-height:19px">(We also accept <strong style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px;line-height:1.4em">inline</strong> patches compatible with format generated by <code style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px;line-height:1.4em;background-color:rgb(248, 248, 248);border-top-width:1px;border-right-width:1px;border-bottom-width:1px;border-left-width:1px;border-top-style:solid;border-right-style:solid;border-bottom-style:solid;border-left-style:solid;border-top-color:rgb(222, 222, 222);border-right-color:rgb(222, 222, 222);border-bottom-color:rgb(222, 222, 222);border-left-color:rgb(222, 222, 222)">git format-patch</code>, but please make sure that your email client has not garbled the message.)</span></blockquote>

<div><br></div><div><br></div><div>What <i>is</i> the preferred method of submitting patches to the list? Inline? Git fetch command? Some combination of the two?</div><div><br></div><div>Cheers,</div><div>Tom</div><div><br>

</div></div>