[SCM] libav/experimental: Patch review process

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 15:57:52 UTC 2013


The following commit has been merged in the experimental branch:
commit f5f11d7101aba3a6ed497f4cc805b842433f8192
Author: Michael Niedermayer <michaelni at gmx.at>
Date:   Thu Mar 15 11:59:50 2007 +0000

    Patch review process
    
    Originally committed as revision 8409 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/doc/ffmpeg-doc.texi b/doc/ffmpeg-doc.texi
index f582755..714a5cf 100644
--- a/doc/ffmpeg-doc.texi
+++ b/doc/ffmpeg-doc.texi
@@ -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
 and has no lrint()')
 
-We reply to all submitted patches and either apply or reject with some
-explanation why, but sometimes we are quite busy so it can take a week or two.
+ at section Patch review process
+
+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
 

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list