[MPlayer-dev-eng] [PATCH] printf -> mp_msg conversion (etc), third patches

[Ivo]

Sorry to step in, but I think I have something useful to say about this :-)

On Monday 13 September 2004 15:04, The Wanderer wrote:
> Actually I was doing this intentionally, because A) I've seen
> several/many others do it the same way, B) that seems most of the time
> to be how they're committed (multiple CVS-log messages with the same
> description but one file per message, rather than many files and just
> one message),

The reason this occurs is if a developper does:

cvs -z3 commit somedir/file1 someotherdir/file2 somedir/file3

an editor (usualy vi) pops up and the cvslog message is written. After that, 
the cvs server applies the patches to all files specified and for each file 
an e-mail is sent with the same cvslog message.

> and C) it makes it much easier for me to re-create the
> patch if some part of it (say, just one file's worth) needs to be
> changed; rather than having to
>
> diff -u main/file1.c main.working/file1.c > patch.diff && diff -u
> main/file2.c main.working/file2.c >> patch.diff && (...)
>
> I'd just need to re-diff the single file.

You could also use cvs diff -u on your working directory. Like:

cd main.working
cvs diff -u file1.c file2.c file3.c > ../cvsdiff-u.to.be.sent.to.dev-eng
cd ..

It doesn't matter whether you changed only one file or more, it is always a 
complete up-to-date patch.

--Ivo