mirror of https://git.ffmpeg.org/ffmpeg.git
doc/developer.texi: refine the "contributing code" section
Drop the reference to directly committing code, because - it is highly discouraged and very rarely done these days - there is no good reason NOT to submit patches for review Add a link to the development policy chapter.
This commit is contained in:
parent
66871647dc
commit
6a3e174ad1
|
@ -46,15 +46,10 @@ consult @url{https://ffmpeg.org/legal.html}.
|
||||||
|
|
||||||
@section Contributing code
|
@section Contributing code
|
||||||
|
|
||||||
There are 2 ways by which code gets into FFmpeg:
|
All proposed code changes should be submitted for review to
|
||||||
@itemize @bullet
|
@url{mailto:ffmpeg-devel@@ffmpeg.org, the development mailing list}, as
|
||||||
@item Submitting patches to the ffmpeg-devel mailing list.
|
described in more detail in the @ref{Submitting patches} chapter. The code
|
||||||
See @ref{Submitting patches} for details.
|
should comply with the @ref{Development Policy} and follow the @ref{Coding Rules}.
|
||||||
@item Directly committing changes to the main tree.
|
|
||||||
@end itemize
|
|
||||||
|
|
||||||
Whichever way, changes should be reviewed by the maintainer of the code
|
|
||||||
before they are committed. And they should follow the @ref{Coding Rules}.
|
|
||||||
The developer making the commit and the author are responsible for their changes
|
The developer making the commit and the author are responsible for their changes
|
||||||
and should try to fix issues their commit causes.
|
and should try to fix issues their commit causes.
|
||||||
|
|
||||||
|
@ -267,6 +262,7 @@ For Emacs, add these roughly equivalent lines to your @file{.emacs.d/init.el}:
|
||||||
(setq c-default-style "ffmpeg")
|
(setq c-default-style "ffmpeg")
|
||||||
@end lisp
|
@end lisp
|
||||||
|
|
||||||
|
@anchor{Development Policy}
|
||||||
@chapter Development Policy
|
@chapter Development Policy
|
||||||
|
|
||||||
@section Patches/Committing
|
@section Patches/Committing
|
||||||
|
|
Loading…
Reference in New Issue