After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 706803 - Disable GSettings in tp-aw
Disable GSettings in tp-aw
Status: RESOLVED FIXED
Product: gnome-online-accounts
Classification: Core
Component: Telepathy
3.9.x
Other Linux
: Normal major
: ---
Assigned To: Marco Barisione
GNOME Online Accounts maintainer(s)
Depends on: 706801
Blocks:
 
 
Reported: 2013-08-26 14:14 UTC by Marco Barisione
Modified: 2013-08-28 16:12 UTC
See Also:
GNOME target: ---
GNOME version: 3.9/3.10

Attachments
configure.ac: disable tp-aw's GSettings (1.32 KB, patch)
2013-08-27 14:22 UTC, Marco Barisione 		committed Details | Review

Description Marco Barisione 2013-08-26 14:14:46 UTC 
+++ This bug was initially created as a clone of Bug #706801 +++

tp-aw's configure has a few arguments to install stuff in separate directories, so GOA and Empathy don't ship the same files to the same paths.
This doesn't work for the GSettings schemas as the rules are generated automatically. It would not make sense anyway to ships 2 schemas for the same path.

GSettings is used by tp-aw only to store the path where the last avatar was. It's useful, but I suggest to just drop this feature from tp-aw completely or just from GOA. It's not going to be a real problem and we should get a proper fix when the fix for bug #706317 is merged.
Comment 1 Marco Barisione 2013-08-27 14:22:22 UTC 
Created attachment 253257 [details] [review]
configure.ac: disable tp-aw's GSettings

This is needed to avoid multiple components shipping the same schema.
It's just a temporary workaround that won't be needed when
https://bugzilla.gnome.org/show_bug.cgi?id=706317 will be fixed.
Comment 2 Marco Barisione 2013-08-27 14:23:13 UTC 
Before merging this we need to update tp-aw to a version that contain the (not yet merged) fix for bug #706801.
Comment 3 Debarshi Ray 2013-08-28 12:41:52 UTC 
Review of attachment 253257 [details] [review]:

Looks good, but you also need to update the tpaw submodule for this. Right?
Comment 4 Marco Barisione 2013-08-28 14:25:43 UTC 
Pushed to master.