Opened 11 years ago

Closed 11 years ago

#5923 closed defect (fixed)

-flags trell not valid in current ffmpeg

Reported by: Joel <mythtv@…> Owned by: xris
Priority: minor Milestone: unknown
Component: perl / nuvexport Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

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.

Change History (3)

comment:1 Changed 11 years ago by Dibblah

Status: newassigned

comment:2 Changed 11 years ago by 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'

comment:3 Changed 11 years ago by xris

Resolution: fixed
Status: assignedclosed

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

Note: See TracTickets for help on using tickets.