Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 159496 - media-plugins/vdr-vdrrip - vdrrip-qh isn't working (again)
Summary: media-plugins/vdr-vdrrip - vdrrip-qh isn't working (again)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo VDR Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-30 10:21 UTC by E. Scaglione
Modified: 2007-01-03 03:21 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
mencoderparam-2.diff (mencoderparam-2.diff,2.70 KB, patch)
2006-12-30 10:25 UTC, E. Scaglione
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description E. Scaglione 2006-12-30 10:21:46 UTC
a bug that was fixed in r4 reappeared misteriously after an emerge --sync :-)
Corrected and renamed the patch file to avoid future problems


Dec 30 18:48:59 [vdrrip-qh] an error occured (rc 1) while processing the command
: /usr/bin/mencoder /film/vdrrip/temp.vdr -o /film/vdrrip/@CHI_VUOL_ESSERE_MILIO
NARIO.avi  -ovc lavc -lavcopts vcodec=mpeg4:vhq:vbitrate=99999:vqmin=2:vqmax=15:
vpass=2::ildct:ilme -sws 2 -vop pp=ci, -oac copy -aid 0 
Dec 30 18:48:59 [vdrrip-qh] error message: Option lavcopts: Unknown suboption _E
rror parsing option on the command line: -lavcopts

was caused by a wrong if block (&& instad of -a and a spurious then)

  if [ "$ppvalues" && "${ppvalues:1:2}" != "hb" ]
     then
     local mintopts=""
     local xintopts=""
  else
     then
     local mintopts=":ildct:ilme"
     local xintopts=":interlacing"
  fi

should have been like this:

  if [ "$ppvalues" -a "${ppvalues:1:2}" != "hb" ]
     then
     local mintopts=""
     local xintopts=""
  else
     local mintopts=":ildct:ilme"
     local xintopts=":interlacing"
  fi
Comment 1 E. Scaglione 2006-12-30 10:25:02 UTC
Created attachment 104998 [details, diff]
mencoderparam-2.diff

should of course replace mencoderparam.diff in the ebuild too
Comment 2 Matthias Schwarzott gentoo-dev 2007-01-03 03:21:00 UTC
Fixed in vdr-vdrrip-0.3.0-r6