Re: [Tracker] PATCH: Allow disabling all extractors

[Martyn Russell <martyn lanedo com>]

On 11/07/13 12:20, Jonatan Pålsson wrote:
> On 11 July 2013 12:56, Philip Van Hoof <philip codeminded be> wrote:
> > For example
> > git checkout -b optional_extractors tracker-ivi/optional_extractors
> > git branch optional_extractors_upstream
> > git rebase -i optional_extractors
> > #Comment out the Tracker-IVI specific commits' pick lines
> > git push tracker-ivi optional_extractors_upstream
> >
> > And you publish (just) optional_extractors_upstream to us.
>
> Ah, excellent. I will do this from now on, thanks for the instructions.

Great patches Jonatan, thanks! :)

> > I also recommend not using autogen.sh for your Tracker-IVI specific
> > configure switches. You should use your package system's features for
> > this purpose instead. Consider taking a look at Tracker's gitorious
> > repository which hosts the branches and tags for MeeGo Harmattan's
> > debian packaging and adaptations that are specific for the N9.
> >
> > Just do the exact same thing.
>
> This is indeed much nicer. I will have a look at using the package
> systems instead of autogen, thanks for the pointer!

Ultimately adding configure flags add maintenance burden at some stage. 
We really only want configure switches for commonly disabled or enabled 
features. But the ones you sent patches for are fine - actually, I think 
the extractors are the one area people would highly want to configure. 
Philip has a good point about packaging... If you have configure 
switches you can certainly be more lazy about what you package :) e.g. 
*.so. Though, that's not generally recommended.

--
Regards,
Martyn

Founder and CEO of Lanedo GmbH.