<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">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 class="im">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 class="h5"> </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><meta http-equiv="content-type" content="text/html; charset=utf-8"></div>
<blockquote class="webkit-indent-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">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"></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><meta http-equiv="content-type" content="text/html; charset=utf-8"><span class="Apple-style-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 class="caps" 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 class="Apple-style-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; font: normal normal normal 12px/normal 'Bitstream Vera Sans Mono', Courier, monospace; 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); border-top-left-radius: 3px 3px; border-top-right-radius: 3px 3px; border-bottom-right-radius: 3px 3px; border-bottom-left-radius: 3px 3px; ">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>