From e26921ed5470e784287dfb49369e07559986feb7 Mon Sep 17 00:00:00 2001 From: diego Date: Tue, 24 Dec 2002 18:41:20 +0000 Subject: Document things from the start. Inspired by a patch from Thomas Jarosch . git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@8557 b3059339-0415-0410-9bf9-f77b7e298cf2 --- DOCS/tech/patches.txt | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'DOCS/tech/patches.txt') diff --git a/DOCS/tech/patches.txt b/DOCS/tech/patches.txt index 65b050cd1f..f3db528415 100644 --- a/DOCS/tech/patches.txt +++ b/DOCS/tech/patches.txt @@ -20,8 +20,12 @@ outdated patches. 5. Comment parts that really need it (tricky side-effects etc). Commenting trivial code not required. Comments must be English! + +6. If you implement new features, add or change command line switches or modify + the behavior of existing features, please do not forget to also update the + documentation. The documentation maintainers will assist you in doing this. -6. Send your patch to the mplayer-dev-eng mailing list as a base64-encoded +7. Send your patch to the mplayer-dev-eng mailing list as a base64-encoded attachment (use gzip or bzip2 *only* if it's bigger than 80k or if you know that your mailer messes up (reformats) text attachments) with the subject line: '[PATCH] very short description of the patch'. @@ -37,14 +41,14 @@ outdated patches. worry, mplayer-dev-eng is not high traffic and you can subscribe with the nomail option if you do not wish to receive all the mails. -7. Give us a few days to react. We try to review patches as fast as possible, +8. Give us a few days to react. We try to review patches as fast as possible, but unfortunately we are constantly overloaded with work, be it MPlayer related or from our day to day lives. If your patch seems to be ignored, please resend it and mention that you got ignored. We are interested in your work and will eventually either accept it or reject it with an explanation what and why we disliked about your patch. -8. Do not immediately ask for CVS write access. If you contributed one or more +9. Do not immediately ask for CVS write access. If you contributed one or more nice, acceptable patches and they need maintaining or you want to be an MPlayer developer, you'll get CVS write access. -- cgit v1.2.3