Matthias Reichl wrote:
> Hi Duncan!
> 
> On Fri, May 15, 2009 at 08:39:12PM +0200, Duncan Webb wrote:
>> Will you try the attached patch and report back if this works?
> 
> The patch fails if MPLAYER_TEMP_DIR is empty and if the current
> directory is not writable.
> 
> The outdir option in newer MPlayer versions is really a nice
> thing, but since the current _videothumb_parse needs to do
> a chdir anyways, it'd be easier to just do the chdir in
> _videothumb (before starting mplayer) and not use the outdir
> option at all (for now). I've attached a modified patch.

Hi Matthias,

Thanks for the report and patch, this is now in svn with a slight
modification, instead of '/tmp' using config.FREEVO_TEMPDIR.

The rational is that this should be configurable and who knows one day
we may have a other operating systems native versions.

I've also changed the rss feed downloader to behave the same.

Duncan

------------------------------------------------------------------------------
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

Reply via email to