Bug 44329 - Add a configure time switch to enable/disable the plugins required
Summary: Add a configure time switch to enable/disable the plugins required
Alias: None
Product: Ytstenut
Classification: Unclassified
Component: plugins (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Siraj Razick
QA Contact:
URL: http://cgit.collabora.com/git/user/si...
Whiteboard: review+ after a small change
Keywords: patch
Depends on:
Reported: 2011-12-30 12:54 UTC by Siraj Razick
Modified: 2012-08-06 12:43 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Description Siraj Razick 2011-12-30 12:54:06 UTC
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.
Comment 1 Siraj Razick 2012-01-03 12:39:40 UTC
patch adds the options to choose the plugins we need to build.
Comment 2 Jonny Lamb 2012-01-20 10:33:22 UTC
+LT_INIT([disable-static win32-dll])
+m4_ifdef([LT_OUTPUT], [LT_OUTPUT])

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.
Comment 3 Siraj Razick 2012-01-20 11:00:34 UTC
oops yeah.. those two lines have leaked from another WIP patch.

anyways..new patch ->
Comment 4 Olli Salli 2012-01-27 06:34:19 UTC
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.
Comment 5 Olli Salli 2012-01-27 06:52:00 UTC
And again, keyword, not whiteboard for "patch", please.

+    AS_HELP_STRING([--enable-salut-plugin],[build telepathy-salut plugin default=yes]),
+                   enable_salut_plugin=$enableval,enable_salut_plugin=no)

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.
Comment 6 Simon McVittie 2012-08-06 12:43:10 UTC
This seems to have been merged.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.