[Ffmpeg-devel] question on submitting changes ...

[Michael Niedermayer]

Hi

On Thu, Aug 11, 2005 at 10:11:57AM +0200, Michel Bardiaux wrote:
> Guillaume POIRIER wrote:
> >Hi,
> >
> >On 8/10/05, Brown, Mike <mikeb at vibephone.com> wrote:
> >
> >>I've made a small change to ffmpeg to allow ffmpeg to process Microsoft 
> >>WMV
> >>format files that contain script streams (aka command media).  This change
> >>allows the files to be processed but ignores the script stream.
> >>
> >>I'm new to ffmpeg and since the mplayer docs are offline I don't have
> >>information on the details of submitting changes back to ffmpeg.  If this
> >>functionality is of interest would someone let me know the procedure for
> >>submitting the changes?
> >
> >
> >Make a unified patch against current CVS:
> >
> >diff -u old.c new.c > patch.diff
> >will for example give you a patch to go from old.c to new.c
> >
> >Don't forget to re-read your patch so that you don't make some
> >whitespace changes.
> >
> >Guillaume
> The easiest way to make a diff against cvs is:
> 
> cd <ffmpeg top> [the one where configure is]
> cvs diff -ub <path>/<file>.c [e.g. libavcodec/avcodec.h] > patch.diff

do not use -b! it will trash indention, manually remove whitespace changes
if you have some in your code, which you should not in the first place

[...]
-- 
Michael