[erlang-patches] What is the preferred method of submitting patches to the list? [was Re: [PATCH] ei: integer overflow in string/atom encoding]

[Raimo Niskanen]

On Tue, Jun 07, 2011 at 12:11:01PM -0400, Tom Moertel wrote:
> On Fri, Jun 3, 2011 at 12:21 PM, Mikael Pettersson <mikpe@REDACTED> wrote:
> 
> > p.s. Thank you for inlining the patch and providing a rationale, most git
> > users on this list don't do that which makes peer review difficult.
> >
> >
> 
> If inline patches are preferred, perhaps the following recommendations
> should be revised to say so more clearly:

That Mikael likes inline patches is not our official policy.
He is in this context a peer himself, although a dear one.

We find patches in the form of Git branch references less error prone.
Also, one can rather easily navigate to the contributor's github repository
and view the commits, not as easy as having the patch in the mail, though.

Maybe we should wish for contributors to besides the git fetch command
supply a http link to the branch for easier online viewing...

Those who do inline patches correct are welcome to do so, but
recommending that as the preferred method would render us lots
of corrupt patches due to malicious mail clients and whatnot.

/ Raimo Niskanen, Erlang/OTP, Ericsson AB

> 
> https://github.com/erlang/otp/wiki/submitting-patches
> 
> <https://github.com/erlang/otp/wiki/submitting-patches>
> 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:
> 
> 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 OTP to decide whether (s)he will pick up the patch.
> 
> 
> 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:
> 
> (We also accept *inline* patches compatible with format generated by git
> > format-patch, but please make sure that your email client has not garbled
> > the message.)
> 
> 
> 
> What *is* the preferred method of submitting patches to the list? Inline?
> Git fetch command? Some combination of the two?
> 
> Cheers,
> Tom

> _______________________________________________
> erlang-patches mailing list
> erlang-patches@REDACTED
> http://erlang.org/mailman/listinfo/erlang-patches


-- 

/ Raimo Niskanen, Erlang/OTP, Ericsson AB