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 678394 - [Patch] Make GConf support optional
[Patch] Make GConf support optional
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
3.5.x
Other Linux
: Normal enhancement
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on:
Blocks: 622558
 
 
Reported: 2012-06-19 12:04 UTC by Marc-Antoine Perennou
Modified: 2012-08-06 20:53 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
make gconf support optional (5.96 KB, patch)
2012-06-19 12:04 UTC, Marc-Antoine Perennou 		committed Details | Review

Description Marc-Antoine Perennou 2012-06-19 12:04:50 UTC 
Created attachment 216735 [details] [review]
make gconf support optional

Gnome-sesion has fully been ported to using gsettings, but still depends on gconf for legacy reasons in autostarting applications.

Make this support optional in order to make the gconf dependency optional too.
Comment 1 Matthias Clasen 2012-06-26 17:19:46 UTC 
I think it would be better to use gconftool-2 and turn the build-time dep into a runtime one.
Comment 2 Marc-Antoine Perennou 2012-06-26 18:06:37 UTC 
Maybe we could combine both? Making it optional and using gcontool-2 if it's enabled ?
Comment 3 Matthias Clasen 2012-06-27 22:54:43 UTC 
If it is just a runtime dependency, there's no good reason to add ifdefs, imo
Comment 4 Marc-Antoine Perennou 2012-07-03 07:40:35 UTC 
I won't have much time in the next weeks, if anybody is willing to do it quick, feel free to do so.
Comment 5 Matthias Clasen 2012-08-06 18:58:45 UTC 
Looking more closely, it is not really possible to do this at runtime, because we need to monitor for key changes.
Comment 6 Ray Strode [halfline] 2012-08-06 20:53:27 UTC 
Attachment 216735 [details] pushed as 8baf35d - make gconf support optional

Okay let's it push it (with minor typo fixes)