-flags trell not valid in current ffmpeg

[Joel <mythtv@…>]

nuvexport uses the option -flags +trell when converting to xvid. This option is no longer valid in the latest ffmpeg. It must now be specified as "-trellis 1" I don't know at what point it changed, so it might take some work to determine the ffmpeg version and provide the appropriate option based on ffmpeg version, as I imagine -trellis 1 won't work on a number of existing installs.

ffmpeg is still exiting with: av_interleaved_write_frame(): Error while opening file But, the above error is something I know needs to be fixed to work with current ffmpeg. I haven't figured out how to fix this part yet.

[disafan@…]

The below patch should work for newer FFMPEGs.

diff -Naur nuvexport/export/ffmpeg/MP4.pm nuvexport-new/export/ffmpeg/MP4.pm --- nuvexport/export/ffmpeg/MP4.pm 2008-04-11 07:46:31.000000000 -0300 +++ nuvexport-new/export/ffmpeg/MP4.pm 2008-12-12 22:09:57.000000000 -0200 @@ -237,7 +237,8 @@

;

} else {

• $ffmpeg_xtra .= ' -flags +mv4+trell+loop'

+ $ffmpeg_xtra .= ' -flags +mv4+loop' + .' -trellis 1'

.' -aic 1' .' -mbd 1' .' -cmp 2 -subcmp 2'

diff -Naur nuvexport/export/ffmpeg/XviD.pm nuvexport-new/export/ffmpeg/XviD.pm --- nuvexport/export/ffmpeg/XviD.pm 2008-04-11 07:46:31.000000000 -0300 +++ nuvexport-new/export/ffmpeg/XviD.pm 2008-12-12 22:08:48.000000000 -0200 @@ -150,7 +150,8 @@

. $self->param('bit_rate_tolerance', 32) . ' -bufsize 65535'

: )

• .' -flags +mv4+trell+loop'

+ .' -flags +mv4+loop' + .' -trellis 1'

.' -aic 1' .' -mbd 1' .' -cmp 2 -subcmp 2'

[xris]

(In [20325]) first start of support for ffmpeg 0.5.. includes fixing trellis parameter to close #5923