in the current build system, to configure we required both tp-salut and tp-gabble, in cases where we only need the gabble or salut plugins we should have
a configure time switch to enable or disable the plugins we don't need.
patch adds the options to choose the plugins we need to build.
What's this? Should this be in a separate commit? I don't see anything about win32 in the commit message?
I reckon we could get rid of the "telepathy-" prefix; the option is really long! This goes for the gabble one too.
Otherwise seems fine.
oops yeah.. those two lines have leaked from another WIP patch.
anyways..new patch ->
Siraj, you really should have one branch per fd.o bug, and all the patches in the branch belong to that bug. This is because review is done at the bug level; when a bug gets review+ you can then just merge the whole branch.
And again, keyword, not whiteboard for "patch", please.
+ AS_HELP_STRING([--enable-salut-plugin],[build telepathy-salut plugin default=yes]),
Doesn't look like the default would be yes, although the help string says so?
I think both gabble and salut plugin should default to yes for consistency. So consider this accepted after flipping the default for salut.
This seems to have been merged.