Mercurial > mplayer.hg
changeset 2200:1333f6184ad3
some ifno for cvs write owners
author | arpi |
---|---|
date | Sun, 14 Oct 2001 19:18:02 +0000 |
parents | 94dbd246fbbe |
children | 1b684f452eb8 |
files | DOCS/tech/cvs-howto.txt |
diffstat | 1 files changed, 91 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/DOCS/tech/cvs-howto.txt Sun Oct 14 19:18:02 2001 +0000 @@ -0,0 +1,91 @@ + +About CVS write access: by A'rpi +~~~~~~~~~~~~~~~~~~~~~~~ + +I. TECH SIDE: +============= +1. Changing password: + + As you probably got a restricted CVS-only shell, it's not trivial: + + ssh LOGIN@mplayerhq.hu passwd + + (replace LOGIN with your loginname. leave 'passwd' unchanged, it's command) + + Note: if you need real shell for something, tell me. + +2. Checkout devel. source tree: + + export CVS_RSH=ssh + cvs -z3 -d:ext:LOGIN@mplayerhq.hu:/cvsroot/mplayer co main + +3. Commiting changes: + + cvs -z3 commit -m "comment - what and why did you change" filename(s) + +4. Adding new files/dirs: + + cvs add filename/dirname + +5. Removing files: + + rm filename + cvs remove filename + +6. Check changes: + + cvs -z3 diff -u filename(s) + +7. Check changelog: + + cvs -z3 log filename(s) + +8. Rename/move files: + + you can't do that. ask CVS server admin (A'rpi) to do it. + (do not remove & re-add file - it will kill changelog!) + +If you have any tech problems with cvs server, contact me: +A'rpi <arpi@thot.banki.hu> + + +II. POLICY / RULES: +=================== + +1. You shouldn't commit code which makes cvs broken! + (i mean unfinished but enabled code which break + compiling or compiles but does not work) + +2. You don't have over-test things. if it works for you, + and you think it should work for others too, then commit. + If it has problems (portability, exploits compiler bugs, + unusual environment etc) they will be reported, it's ok. + +3. You can commit unfinished stuff (for testing etc), but it + must be disabled (#ifdef etc) by default. + +4. Do not change behaviour of the program (renaming options etc) + without discussing first at the MPlayer-dev-eng list. + +5. Source indenting and other cosmetical changes are refused. + I'll remove those commits... + Every developer has its own indenting style, you shouldn't + change it. Of course if you (re)write something then you can + use your own style... + (Many projects forces a given indenting style - we don't) + +6. Always fill out the comment at commiting (-m switch of cvs, or + in the editor if you left -m). + It shouldn't be such lines: "fixed!" or "Changed it." + Describe in a few lines (usually 1 line is enough) what did + you changed and why did you that. You can refer mails if bugfix. + +7. If you apply patch by someone else, include his name and email + in the cvs comment! + +8. I've developed something called CVS-Backup. It archives CVS + repository after each commit - so I can reverse your commits + (without messing up changelog) if they are bad. + + +I think our rules aren't too hard. If you have comments, contact me.