Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
FORM_TYPE should be hidden, type should be text-single, and name and caps
should be text-multi.
This is significant because if the types of name and caps are not
explicitly specified in the message, Wocky will guess. If there is only
one value, it will guess text-single, and the GValue will contain
G_TYPE_STRING, not G_TYPE_STRV, causing a crash when we call
g_value_get_boxed.
Reviewed-by: Olli Salli <olli.salli@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=45236
|
|
Otherwise, the client side will have to guess it: text-multi if there
is more than one of something, otherwise text-single.
Reviewed-by: Olli Salli <olli.salli@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=45236
|
|
Otherwise we lose our interest in services' statuses whenever a new
Ytstenut client is added - even if it's just a temporary one for the
request-and-handle pattern.
Reviewed-by: Olli Salli <olli.salli@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=53167
|
|
Reviewed-by: Olli Salli <olli.salli@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=53160
|
|
Reviewed-by: Alvaro Soliverez <alvaro.soliverez@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=53162
|
|
Gabble's and Salut's pkg-config files changed the variable names of the
place to plug plugins to "pluginexecdir", so this is the thing we need
to be looking for now, over the old "plugindir".
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=48060
|
|
Reviewed-by: Olli Salli <olli.salli@collabora.co.uk>
|
|
|
|
|
|
|
|
is in salut core and not in the plugin code
|
|
So it builds as a DLL on windows
NOTE: this doesn't make the linker whine about undefined symbols on linux. This is a libtool policy
libtool -no-undefined is a no-op on Linux. So we need the buildbot windows target to verify we don't
end up using symbols which aren't really in the salut-plugins library.
|
|
|
|
This is needed because in some platforms (eg Android) the files
cannot be compiled twice. Therefore, we copy the files to salut's
directory and build there.
The copied files are added to .gitignore to prevent those files from
being added to the repository
|
|
This is needed because in some platforms (eg Android) the files
cannot be compiled twice. Therefore, we copy the files to gabble's
directory and build there.
The copied files are added to .gitignore to prevent those files from
being added to the repository
|
|
|
|
|
|
|
|
|
|
Johnny Lamb
|
|
|
|
Conflicts:
gabble/message-channel.c
gabble/status.c
salut/message-channel.c
salut/status.c
|
|
Commit 4f94ae6 typoed the implementation of --enable-salut-plugin, so
HAVE_TP_SALUT was always false.
|
|
Gabble and Salut Plugin API was updated to remove the dupliate
TpBaseconnection from create_channel_managers. This patch updates
the plugin-base to match the API change
Signed-off-by: Rob Staudinger <robsta@linux.intel.com>
|
|
Gabble and Salut Plugin API was updated to remove the dupliate
TpBaseconnection from create_channel_managers. This patch updates
the plugin-base to match the API change
|
|
|
|
telepathy-salut now uses a seperate library for plugins, similar to
mission control and gabble, This patch updates salut plugin to use
this new library, and the releated API changes which the new plugin API
introduces. (mainly the following)
- Use SalutPluginConnection instead of SalutConnection
- Assign ytstenut_plugin_create_sidecar_finish to create sidecar_finish
- Use salut_plugin_connection_get_name and salut_plugin_connection_get_session
fdo bug : https://bugs.freedesktop.org/show_bug.cgi?id=45804
|
|
it should be enable_salut_plugin not enable_salt_plugin.
So HAVE_TP_SALUT is never true when --enable-salut-plugin is issued
for configure script.
|
|
A few filenames in tests/twisted/tools were too long for default 'tar',
call AM_INIT_AUTOMAKE with 'tar-ustar' for remedy.
Also tests/twisted/Makefile.am had non-existant files in EXTRA_DIST,
which prevented the dist from finishing. Remove those.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=45773
|
|
https://bugs.freedesktop.org/show_bug.cgi?id=45704
|
|
Add -no-undefined option so that .dll files are created during
windows builds.
|
|
Address the review comments of fdo#44447. This patch fixes the
salut build, since the plugins API of gabble and salut is different
at this point.
|
|
|
|
Use GabblePluginConnection instead of GabbleConnection
|
|
--enable-salut-plugin builds the sault plugin and --enable-gabble-plugin
builds the gabble plugin
|
|
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
|
|
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
Signed-off-by: Rob Staudinger <robsta@linux.intel.com>
|
|
Signed-off-by: Rob Staudinger <robsta@linux.intel.com>
|
|
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
If we ensure the status sidecar way into a connection, Gabble might
already know about services, so the status needs to update its
properties.
Finally, now it can! I even added a test.
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
If a contact appears and displays that it has support for all these
services, we should remember this when the Status sidecar appears
later on in the CM process.
This works fine in salut. It will not work yet in gabble.
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
These are all copies of the salut tests, but it's really not worth
trying to make them work on both salut and gabble so here is some code
duplication. Sorry.
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
Previously, I'd done:
...
<items xmlns="the-node">
...
when actually it should be:
...
<items node="the-node">
...
Bad me.
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
This way it'll have an id stamped on.
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|
|
Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>
|