Download-Tools/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-02-12 04:44:25 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Patch review process

Browse Source 
Originally committed as revision 8409 to svn://svn.ffmpeg.org/ffmpeg/trunk
This commit is contained in:
Michael Niedermayer 2007-03-15 11:59:50 +00:00
parent 7ca6cc099c
commit f5f11d7101
1 changed files with 19 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
doc/ffmpeg-doc.texi
View File

@ -1594,8 +1594,25 @@ It also helps quite a bit if you tell us what the patch does (for example
'replaces lrint by lrintf'), and why (for example '*BSD isn't C99 compliant 'replaces lrint by lrintf'), and why (for example '*BSD isn't C99 compliant
and has no lrint()') and has no lrint()')
We reply to all submitted patches and either apply or reject with some @section Patch review process
explanation why, but sometimes we are quite busy so it can take a week or two.
All patches posted to ffmpeg-devel will be reviewed, unless they contain a
clear note that the patch is not for SVN.
Reviews and comments will be posted as replies to the patch on the
mailinglist. The patch submitter then has to take care of every comment,
that can be by resubmitting a changed patch or by disscussion. Resubmitted
patches will themselfs be reviewed like any other patch. If at some point
a patch passes review with no comments then it is approved, that can for
simple and small patches happen immedeatly while large patches will generally
have to be changed and reviewed many times before they are approved.
After a patch is approved it will be applied to ffmpeg svn
We will review all submitted patches, but sometimes we are quite busy so
especially for large patches this can take several weeks.
When resubmitting patches, please do not make any significant changes
unrelated to the comments such patches will be rejected, Instead submit
such significant changes or new features as seperate patches.
@section Regression tests @section Regression tests