[MPlayer-cvslog] r26969 - trunk/DOCS/tech/svn-howto.txt

[Diego Biurrun]

On Tue, Jun 03, 2008 at 12:10:20AM -0500, compn wrote:
> On Tue,  3 Jun 2008 03:37:44 +0200 (CEST), diego <subversion at mplayerhq.hu> wrote:
> >+++ trunk/DOCS/tech/svn-howto.txt	Tue Jun  3 03:37:43 2008
> >@@ -305,6 +305,7 @@ II. POLICY / RULES:
> >      - verbosity changes from default (info) level
> >      - changes to "historical" parts of docs and web pages
> >      - use of internal or external libraries
> >+     - reverting commits from other developers
> 
> + without discussion

Check out the whole text before commenting please.  This merely lists
which actions have proved to be controversial in the past:

  12. Always send a patch to the mplayer-dev-eng mailing list before committing
    if you suspect that the change is going to be controversial. Based on past
    experience, these changes are likely to be controversial:
     - feature removal, even if obsolete
     - changes to "special" output messages (like the "Core dumped ;)" message)
     - verbosity changes from default (info) level
     - changes to "historical" parts of docs and web pages
     - use of internal or external libraries
     - reverting commits from other developers

We obviously now have a precedent of a revert turning out to be
controversial.

Diego