[MPlayer-DOCS] CVS: main/help help_mp-en.h,1.214,1.215

The Wanderer inverseparadox at comcast.net
Thu Dec 15 21:50:17 CET 2005

Reynaldo H. Verdejo Pinochet wrote:

> On Thu, Dec 15, 2005 at 06:57:21PM +0100, The Wanderer CVS wrote:
>> CVS change done by The Wanderer CVS
>> Update of /cvsroot/mplayer/main/help
>> In directory mail:/var2/tmp/cvs-serv2139/help
> Nice! thanks for taking the time.

Not a problem; I should have gotten to it earlier, actually.

>> Modified Files:
>> 	help_mp-en.h 
>> Log Message:
>> make -o mandatory and add a warning when the extension does not
>> match the container format, patch by Reynaldo Pinochet
> Well, just a few things:
> if you want to shorten my name, use Reynaldo Verdejo ;)

Acknowledged - I wasn't sure, but I'll try to remember this in future.
(Avoiding the shorter forms would come more easily if I could be sure
enough of my editor to write the commit message somewhere other than the
command line, and so be able to split it into multiple lines..)

Is it important enough for me to change the commit message in the log?

> that patch was originally written by Oded, i just modified it

I knew that he made the original proposal of that change, but my
understanding was that your patch was not just a retooling of his
previous one... and in any case, I don't seem to recall many (any?)
times when multiple people who contributed revisions along the way to a
final patch were credited in the commit message.

       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

Secrecy is the beginning of tyranny.

More information about the MPlayer-DOCS mailing list