[FFmpeg-devel] [PATCH] doc/developer: Remove mentioning of sending pull requests
Ganesh Ajjanagadde
gajjanag at mit.edu
Sun Dec 6 15:54:56 CET 2015
On Sun, Dec 6, 2015 at 8:46 AM, wm4 <nfxjfg at googlemail.com> wrote:
> On Sat, 5 Dec 2015 21:49:55 -0800
> Timothy Gu <timothygu99 at gmail.com> wrote:
>
>> The _de facto_ policy on patch submission has always been "sending it to
>> the mailing list."
>> ---
>> doc/developer.texi | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/doc/developer.texi b/doc/developer.texi
>> index 9a901d8..9383b21 100644
>> --- a/doc/developer.texi
>> +++ b/doc/developer.texi
>> @@ -32,13 +32,11 @@ consult @url{https://ffmpeg.org/legal.html}.
>>
>> @section Contributing
>>
>> -There are 3 ways by which code gets into ffmpeg.
>> +There are two ways by which code gets into ffmpeg.
>> @itemize @bullet
>> @item Submitting Patches to the main developer mailing list
>> see @ref{Submitting patches} for details.
>> @item Directly committing changes to the main tree.
>> - at item Committing changes to a git clone, for example on github.com or
>> - gitorious.org. And asking us to merge these changes.
>
> Technically, doing this is ok. We just don't want to use github's pull
> request UI. Changes should be reviewed and discussed on the ML. But
> posting on the ML and linking to an existing git tree isn't too bad, I
> suppose.
But that is covered in point 1: patches get sent to the ML. This needs
to be done regardless. What a user does on top is really up to them,
e.g referencing a git tree in the patch notes.
>
>> @end itemize
>>
>> Whichever way, changes should be reviewed by the maintainer of the code
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
More information about the ffmpeg-devel
mailing list