Re: [Tracker] tracker-extract-xine + tracker-extract-gstreamer + cleanups

[Jamie McCracken <jamiemcc blueyonder co uk>]

Laurent Aguerreche wrote:
> Le vendredi 20 octobre 2006 Ã 03:58 +0200, Laurent Aguerreche a Ãcrit :
> > Hello,
> >
> > a patch to use Xine directly via libxine. So Tracker doesn't call Xine
> > unlike tracker-extract-{totem,mplayer}.
> > In future, I would like to do the same with Gstreamer but currently I
> > have some difficulties with Gstreamer programming practices.  :-D
> >
> > This patch also include some trivial cleanups in tracker-extract.c.
>
> My patch had some issues with conditional compilation... I fixed it.
> Now, "--enable-gstreamer" (default), "--enable-xine", "--enable-totem"
> and "--enable-mplayer" work.
> This new patch includes the previous one + a new extractor that directly
> uses GStreamer 0.10.
>
> But GStreamer never found video bitrate... Since Nautilus (which uses
> GStreamer) isn't able to do that too, I suppose there is a bug in
> GStreamer 0.10 on Debian Unstable. I would be happy to see someone get
> video bitrate info and to know its linux distribution.

its looking good

just a few changes required:

1) can we only have a conditional compilation option for the gstreamer 
stuff (the rest dont need one as they are not being compiled in!)

2) if gstreamer is not used then try spawning the remaining options 
(totem, mplayer and xine)

g_spawn_sync returns FALSE if we could not execute totem, mplayer or 
xine so it would be good to try them in that order until one of them 
returns TRUE.

sorry to throw this back at you but would you mind modifying the patch 
for the above?

thanks for your hard work here


--
Mr Jamie McCracken
http://jamiemcc.livejournal.com/