Opened 11 years ago

Closed 11 years ago

#2178 closed defect (fixed)

mencoder is throwing away valid, useful options

Reported by: edjb Owned by:
Priority: normal Component: mencoder
Version: HEAD Severity: major
Keywords: Cc:
Blocked By: Blocking:
Reproduced by developer: no Analyzed by developer: no

Description

This line


 if(!entry || (mp_opt->flags & M_OPT_GLOBAL)){

in

  parser-mecmd.c

is causing mencoder to silently throw away command line
arguments such as:

sid
slang
vobsubid

I don't understand the code or I'd try to put a patch
together. But if I simply change the "if" to:

if ( 1 )

then the above command line arguments do what they are
supposed to do.

Here's my command line that attempts to select vobsub index
"1" (english). Without the above patch it always selects
vobsub index=0 (which happens to be German).

./mencoder  "/media/NTFS_3TB/Opera/Wagner - Tristan und Isolde - Barenboim - Bayreuth/Tristan.und.Isolde.Act.2/Tristan.und.Isolde.Act.2.avi"  -vobsub "/media/NTFS_3TB/Opera/Wagner - Tristan und Isolde - Barenboim - Bayreuth/Tristan.und.Isolde.Act.2/Tristan.und.Isolde.Act.2"  -vf harddup -ofps 30000/1001 -oac lavc -ovc lavc -of mpeg -lavcopts abitrate=256:vcodec=mpeg2video:keyint=1:vbitrate=2000: vrc_maxrate=5000:vrc_buf_size=1835 -mpegopts muxrate=12000 -af lavcresample=44100 -font /usr/share/fonts/truetype/ttf-dejavu/DejaVuSans.ttf -slang en -vobsubid 1 -sid 1 -subpos 100 -v -ifo /media/NTFS_3TB/Opera/VTS_06_0.IFO -o /dev/null



Change History (1)

comment:1 by edjb, 11 years ago

Resolution: fixed
Status: newclosed

It sucks that mencoder doesn't complain about parameters that must appear either
before or after an input filename. -sublang must appear before any input filenames.
But this isn't documented and there's no error message.

This should be addressed at some point. If you google this kind of issue many
people just end up giving up in frustration.

Note: See TracTickets for help on using tickets.