[MPlayer-dev-eng] [PATCH] forceable software volume control

Torinthiel torinthiel at megapolis.pl
Fri Nov 5 17:54:12 CET 2004


On Fri, Nov 05, 2004 at 06:47:08PM +0200, Oded Shimon wrote:
> 
> On Friday 05 November 2004 18:23, The Wanderer wrote:
> > No, I've used the command I gave more than once in the past to compare
> > files against CVS (including for creation of patches), and it's worked
> > just fine. I've never had to specify a revision with MPlayer, and would
> > be considerably surprised if it were suddenly necessary now.
> I think you might still be a bit confused, or I am - CVS remembers when you 
> downloaded your file from CVS. And the server remembers all the files from 
> every time. So when you do a CVS 'diff', it doesn't compare with the most 
> recent MPlayer, it compares with the files you _originally_ downloaded, so 
> you can see ONLY the changes you made, and not the changes made in the CVS. 
> This is its behavior at least for me...

If you don't specify -r it compares against revision you have. But if you
did cvs up (and had no sticky flags) that revision is the HEAD revision.
So his procedure (diff right after update) was correct. And this doesn't
tell why the patch got rejected.
Torinthiel

-- 
 Waclaw "Torinthiel" Schiller       GG#: 542916, 3073512
   torinthiel(at)megapolis(dot)pl
   gpg: B06901F1 fpr: FAA3 559F CAE9 34DE CDC8  7346 2B6E 39F2 B069 01F1
 "No classmates may be used during this examination"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20041105/63f8bdf1/attachment.pgp>


More information about the MPlayer-dev-eng mailing list