Bugs item #1581274, was opened at 2006-10-20 06:18 Message generated for change (Comment added) made by sf-robot You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=446895&aid=1581274&group_id=46652
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: players Group: 1.6.0 >Status: Closed Resolution: Accepted Priority: 5 Private: No Submitted By: Tanja (owigera) Assigned to: Nobody/Anonymous (nobody) Summary: empty vf option in local_conf.py Initial Comment: My freevo box is very slow, thus I do not want to use postprocessing when I use mplayer for watching videos. In my local_conf.py the two MPLAYER_VF options are empty for that reason. MPLAYER_VF_INTERLACED = '' MPLAYER_VF_PROGRESSIVE = '' The command line that freevo constructs out of this contains -vf without anything usefull behind it. main.log: ChildApp.__init__(), pid=1306, app=/usr/bin/mplayer -autosync 100 -nolirc -autoq 100 -screenw 800 -screenh 600 -fs -slave -ao alsa -v -vo xv -cache 5000 -vf //media/Tv/Zapping_International_-_Das_Fernsehen_der_Uruguayer_20-10_0930.ts, poll=-1 This causes mplayer to crash: mplayer-stderr.log Option vf: //media/Tv/Zapping_International_-_Das_Fernsehen_der_Uruguayer_20-10_0930.ts doesn't exist. Error parsing option on the command line: -vf I wrote a small patch which tests if those two options contain something before adding the -vf option to the mplayer command. ---------------------------------------------------------------------- >Comment By: SourceForge Robot (sf-robot) Date: 2006-11-03 19:20 Message: Logged In: YES user_id=1312539 This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 14 days (the time period specified by the administrator of this Tracker). ---------------------------------------------------------------------- Comment By: Duncan Webb (duncanwebb) Date: 2006-10-20 10:44 Message: Logged In: YES user_id=104395 A long day... The correct patch is now in svn rel-1-6 at r8436 ---------------------------------------------------------------------- Comment By: Tanja (owigera) Date: 2006-10-20 08:27 Message: Logged In: YES user_id=1620763 I justed tested your patch. Unfortunately you were a little to fast with this. There is a "not" to much in the "if" ;-) New patch attached. ---------------------------------------------------------------------- Comment By: Duncan Webb (duncanwebb) Date: 2006-10-20 06:56 Message: Logged In: YES user_id=104395 Thanks that is a nice little patch. It has been applied to rel-1-6 in r8434. I have modified it slightly so that you can either specify the empty string or: MPLAYER_VF_INTERLACED = None MPLAYER_VF_PROGRESSIVE = None ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=446895&aid=1581274&group_id=46652 ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Freevo-devel mailing list Freevo-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freevo-devel
