[SCM] libav/experimental: Add 'make checkheaders' to developer guidelines and patch checklist.
siretart at users.alioth.debian.org
siretart at users.alioth.debian.org
Sun Jun 30 16:31:55 UTC 2013
The following commit has been merged in the experimental branch:
commit 2cdba8bfc43b818ca8037421c9a136fa89327c66
Author: Diego Biurrun <diego at biurrun.de>
Date: Wed Oct 29 06:48:59 2008 +0000
Add 'make checkheaders' to developer guidelines and patch checklist.
Originally committed as revision 15741 to svn://svn.ffmpeg.org/ffmpeg/trunk
diff --git a/doc/general.texi b/doc/general.texi
index 78372f6..9a823d4 100644
--- a/doc/general.texi
+++ b/doc/general.texi
@@ -951,6 +951,9 @@ should also be avoided if they don't make the code easier to understand.
@item
If you add a new file, give it a proper license header. Do not copy and
paste it from a random place, use an existing file as template.
+ at item
+ If you add a new header or change an existing one, make sure it passes
+ @code{make checkheaders}.
@end enumerate
We think our rules are not too hard. If you have comments, contact us.
@@ -1026,6 +1029,8 @@ do not attach several unrelated patches to the same mail.
@item
Do the regression tests pass with the patch applied?
@item
+ Does @code{make checkheaders} pass with the patch applied?
+ at item
Is the patch a unified diff?
@item
Is the patch against latest FFmpeg SVN?
--
Libav/FFmpeg packaging
More information about the pkg-multimedia-commits
mailing list