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

Björn-Egil Dahlberg egil@REDACTED
Tue Jun 7 19:05:10 CEST 2011


On 2011-06-07 18:41, Raimo Niskanen wrote:
> 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...

+1, I would like that.

> 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.

I think Mikaels point is well taken. For small patches it is much easier 
to review a patch via mail then on github and discussions on various 
issues could easily be done in the mail.

Patch notifications should always carry a 'git fetch' entry at the top 
of the mail.

Regards,
Björn-Egil
Erlang/OTP



More information about the erlang-patches mailing list