[Mplayer-cvslog] CVS: main/DOCS/tech patches.txt,1.6,1.7

Diego Biurrun CVS diego at mplayerhq.hu
Tue Dec 24 19:41:23 CET 2002


Update of /cvsroot/mplayer/main/DOCS/tech
In directory mail:/var/tmp.root/cvs-serv21160/DOCS/tech

Modified Files:
	patches.txt 
Log Message:
Document things from the start.
Inspired by a patch from Thomas Jarosch <tomj at simonv.com>.


Index: patches.txt
===================================================================
RCS file: /cvsroot/mplayer/main/DOCS/tech/patches.txt,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- patches.txt	12 Dec 2002 03:02:03 -0000	1.6
+++ patches.txt	24 Dec 2002 18:41:20 -0000	1.7
@@ -20,8 +20,12 @@
 
 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 @@
    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.
    




More information about the MPlayer-cvslog mailing list