Matthias Reichl wrote:
> On Thu, May 14, 2009 at 04:53:09PM -0500, Jonathan Isom wrote:
>> On Thu, May 14, 2009 at 4:44 PM, Matthias Reichl <h...@horus.com> wrote:
>>> I guess it would be best if freevo wouldn't require this option
>>> until it's in an official mplayer release. Using mplayer SVN
>>> snapshots can really be a PITA...
>> Only problem with waiting for a release is currently it is recommended
>> by the developer that users and distributions use svn of mplayer. 1.0rc2
>> is rather old by the way. They have no plans to make new "releases".
>
> Well, there were plans for a 1.0rc3 release, but development seems
> to have stalled... 1.0rc2 might be rather old, but at least it's working
> and several distributions still ship this version (with security patches,
> of course).
>
> I've been compiling MPlayer source releases from 0.50 (or so) up to
> 1.0rc2 on my box, but then switched to using the binary packages
> from debian-multimedia.org for a very simple reason: last year I
> tried using MPlayer SVN source a few times and always ended up with
> a (partly) non-working mplayer. Before sending out bug reports I
> searched the MPlayer mailinglists just to find out that my bugs were
> known issues. Finding a working SVN version can take quite some time,
> and being an MPlayer user (not developer), I really want to avoid these
> troubles. IOW: no, I don't like their development model, it might
> be OK for distributors, but definitely not for users.
>
> Back to the original issue: Freevo 1.8.3 didn't use the "outdir"
> option (in util/videothumb.py) and worked fine. It would be really
> nice if freevo 1.8.4 wouldn't need it, too, as other people might
> experience the same issues as me.
Will you try the attached patch and report back if this works?
Duncan
Index: freevo_config.py
===================================================================
--- freevo_config.py (revision 11510)
+++ freevo_config.py (working copy)
@@ -396,6 +396,7 @@
Renamed IMAGEVIEWER_ASPECT to OSD_PIXEL_ASPECT as this affects not just images
Added AUTOSHUTDOWN_WAKEUP_TIME_PAD to control how much time to allow for
system boot to complete when waking up from an AUTOSHUTDOWN.
+ Added MPLAYER_TEMP_DIR for temporary files from mplayer
"""),
]
@@ -1693,6 +1694,13 @@
#
MPLAYER_OLDTVCHANNELCHANGE = False
+#
+# Pre mplayer revision 27650 has no outdir option for video thumbnails so use
+# MPLAYER_TEMP_DIR=''
+#
+MPLAYER_TEMP_DIR = '/tmp'
+
+
# ======================================================================
# Xine settings:
# ======================================================================
Index: local_conf.py.example
===================================================================
--- local_conf.py.example (revision 11510)
+++ local_conf.py.example (working copy)
@@ -1185,6 +1185,9 @@
# (if this feature present in mplayer)
# MPLAYER_HAS_FIELD_DOMINANCE = 1
#
+# Pre mplayer revision 27650 has no outdir option for video thumbnails so use
+# MPLAYER_TEMP_DIR = ''
+
# ======================================================================
# Xine settings:
# ======================================================================
Index: src/encodingcore.py
===================================================================
--- src/encodingcore.py (revision 11510)
+++ src/encodingcore.py (working copy)
@@ -466,7 +466,13 @@
self.length = 0
position = str(int(self.length / 2.0))
- arguments = [ '-vo', 'png:z=1:outdir=/tmp', '-ao', 'null', '-frames', '8', '-ss', position, '-zoom' ]
+ if config.MPLAYER_TEMP_DIR:
+ arguments = [ '-vo', 'png:z=1:outdir=%s' % config.MPLAYER_TEMP_DIR ]
+ else:
+ arguments = [ '-vo', 'png:z=1' ]
+
+ arguments += [ '-ao', 'null', '-frames', '8', '-ss', position, '-zoom' ]
+
if self.titlenum:
arguments += [ '-dvd-device', self.source, 'dvd://%s' % self.titlenum ]
else:
@@ -478,7 +484,8 @@
def _videothumb_parse(self, lines, data):
from util import vfs
# chdir to tmp so we have write access
- os.chdir('/tmp')
+ if config.MPLAYER_TEMP_DIR:
+ os.chdir(config.MPLAYER_TEMP_DIR)
for line in lines:
if line:
_debug_(line)
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables
unlimited royalty-free distribution of the report engine
for externally facing server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel
