Re: [Tracker] Low hanging fruit from the catacombs of Jolla

[Martyn Russell <martyn lanedo com>]

On 14/12/12 07:41, Philip Van Hoof wrote:
> Ahaa, ok that explains it. Thanks for double checking this Aleksander.
>
> On Fri, 2012-12-14 at 07:57 +0100, Aleksander Morgado wrote:
> > > https://build.pub.meego.com/package/files?package=tracker&project=CE%
> > > 3AMW%3AShared
> > >
> > > So eh, let's get it upstream.
> > >
> > > So can somebody review this for master? Martyn?
> > >
> > > ps. I can afaik still commit and push to git-gnome.
> >
> > That MeeGo patch was still under discussion:
> > https://bugzilla.gnome.org/show_bug.cgi?id=607548

* tracker-0.10.37-fix-linking-with-newer-toolchain.patch
* tracker-0.10.37-fix-linking-with-newer-glib.patch

  These two shouldn't be needed - looks like a fix added to circumvent
  breaks elsewhere. Not something we should be applying upstream AFAICS.

* tracker-0.10.37-add-userguides-service-for-maemo.patch

  This seems unnecessary with >= 0.14.

* 0005-Fix-missing-gobject-introspection-checks.patch

  This is to disable introspection. So not what we want upstream. There
  is a --disable-introspection switch already for configure. Perhaps
  this wasn't around a while ago?

* 0003-tracker-0.9.26-create-tests-aegis.patch

  This is really not for the public version of Tracker I would say.

* 0002-Tracker-extract-Parse-the-video-filename-to-obtain-e.patch       

  Reviewed, but needs resolution as Aleksander says. Still applies too.

* 0001-Tracker-0.7.23-desktop-files.patch

  Pushed.

--

Thanks for bringing these up Philip.

--
Regards,
Martyn

Founder and CEO of Lanedo GmbH.