Re: [Tracker] MP4 Extractor



Hi!

On Tue, Apr 6, 2010 at 3:02 PM, Florent Viard <fviard lacie com> wrote:
Hi,

I don't think that having a mp4 extractor conflict with having the tracker-extract-gstreamer module. For me the mp4 one is specific where there is always the gstreamer one as fallback for all audio / video not having an extractor (or by disabling the mp4 extractor for people preferring the meta datas extracted the mp4 extractor).
For me having the more extractors possible is the best even if I agree that it shouldn't be a priority to develop new ones. (I developed this one as it was needed by us as we don't have gstreamer on the device and require the faster indexation time with the lower cpu and memory usage. And for this, launching gstreamer wasn't the best solution :p

 The problem comes if we integrate it in the current master, and then we need to maintain it ;)

 A good solution can be to create your own project for that extractor. I did that for the OGG extractor for maemo:
http://gitorious.org/tracker-vorbis-extractor

 The is the distribution who choose what plugins to include (instead of making them compilation options in tracker).
  

FYI the beginning of the extractor looks "byte by byte" as it skip the file header but for the meta tag parsing it is not dependent of the byte position, so there shouldn't be problems in the future I think :) Moreover, I think that the meta tags system of mp4 is cleaner/simpler than one for mp3 and so there isn't a lot of different versions like for mp3. So i think that this code is not complex as is the one for mp3.

With "byte by byte" i meant the low level parsing  compared with gstreamer. The code doesn't look specially ugly :)
 

By the way, did you started looking at the nmm:ontology? because I had a look at how the ontologies are made and it looks easy :) So, I could try to submit a patch regarding my previous ontology change proposal even if it certainly will take me some days.

Not yet, not sure i can do it soon. Patches are welcome, but please take the comments in that thread into account.

Regards,

Ivan



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]