diff options
author | Derek Buitenhuis <derek.buitenhuis@gmail.com> | 2013-01-06 13:10:27 -0500 |
---|---|---|
committer | Derek Buitenhuis <derek.buitenhuis@gmail.com> | 2013-01-07 14:55:48 -0500 |
commit | b5f9b9ac3681acb06d95530f34660ba9fe225305 (patch) | |
tree | 34f8889a968a9e4689c8b36848fd5676db7da35c | |
parent | 9a00374cb4512a58a1fee366b850dfa87c76e1f3 (diff) | |
download | ffmpeg-b5f9b9ac3681acb06d95530f34660ba9fe225305.tar.gz |
doc: Merge disjointed bits about emailing patches
We obviously prefer git-send-email(1), and the disjointed nature
of the two statements was misleading.
Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
-rw-r--r-- | doc/developer.texi | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/doc/developer.texi b/doc/developer.texi index cab4a96f01..f374614463 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -362,12 +362,6 @@ The tool is located in the tools directory. Run the @ref{Regression Tests} before submitting a patch in order to verify it does not cause unexpected problems. -Patches should be posted as base64 encoded attachments (or any other -encoding which ensures that the patch will not be trashed during -transmission) to the -@uref{https://lists.libav.org/mailman/listinfo/libav-devel, libav-devel} -mailing list. - 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()'). This kind of explanation should be the body of the @@ -376,8 +370,12 @@ commit message. Also please if you send several patches, send each patch as a separate mail, do not attach several unrelated patches to the same mail. -Use @code{git send-email} when possible since it will properly send patches -without requiring extra care. +Patches should be posted to the +@uref{https://lists.libav.org/mailman/listinfo/libav-devel, libav-devel} +mailing list. Use @code{git send-email} when possible since it will properly +send patches without requiring extra care. If you cannot, then send patches +as base64-encoded attachments, so your patch is not trashed during +transmission. Your patch will be reviewed on the mailing list. You will likely be asked to make some changes and are expected to send in an improved version that |