[FFmpeg-user] Problem with -vf crop with current git

[Steve Kenton]

 >>>Is this a regression (did it work with older versions)?
 >>
 >> Well, Ubuntu 12.04 ships with libav and the "frozen" ffmpeg included 
there
 >> did the cropping, but I don't know how it relates to "real" ffmpeg 
1.0+ I
 >> build from git.
 >
 >You could try some older ffmpeg versions to see if they fail (if they
 >don't, then the ideal would be using git-bisect to detect the exact
 >offending commit...)
 >-r

I checked some old versions and behaviour changed between tags n0.8 and n0.9
So, as requested I did a git bisect and it points to the following merge:

411cc5c46a89ce0a79832c37d0eab1ed884e5423 is the first bad commit
Merge: 5a6f4a1 6cfed11
Author: Michael Niedermayer <michaelni at gmx.at>
Date:   Sun Sep 11 23:35:35 2011 +0200

     Merge remote-tracking branch 'qatar/master'

     * qatar/master: (31 commits)
       audioconvert: add explanatory comments to channel_names array
       audioconvert: K&R whitespace cosmetics
       avconv: use correct index when selecting metadata to write to.
       avconv: fix inverted variable
       doc/avconv: document option types (input/output/per-stream/...)
       doc/avtools-common-opts: write a section about stream specifiers.
       doc/avconv: remove two pointless paragraphs.
       doc/avconv: document that global options should be specified first.
       doc/avconv: remove entries for nonexistent options
       doc/avconv: remove documentation for removed 'timestamp' option
       doc: cosmetics, rename fftools-common-opts to avtools-....
       avconv: move streamid_map to options context.
       avconv: extend -vf syntax
       avconv: move top_field_first to options context.
       avconv: move inter/intra matrix to options context.
       avconv: remove -psnr option.
       avconv: remove me_threshold option.
       avconv: move video_rc_override_string to options context.
       avconv: move frame pixel format to the options context.
       avconv: move frame aspect ratio to the options context.

I've about exhausted my git-foo and don't know how to subdivide the merge

I don't really object to the current behaviour of requiring filters to be
after all the inputs, it's the silently ignoring them behaviour change that
was introduced by the commit that caused me trouble. An error would be nice

Steve Kenton


BTW - the truncated history resulting from:
"git clone --depth 1 git://source.ffmpeg.org/ffmpeg"
caused me several very puzzled minutes when I first tried to bisect :-)