Port |Bug 1433685 - Remove nsGConfService|
Categories
(MailNews Core :: General, task)
Tracking
(Not tracked)
People
(Reporter: jorgk-bmo, Assigned: darktrojan)
Details
Attachments
(1 file)
4.25 KB,
patch
|
jorgk-bmo
:
review+
aceman
:
review+
|
Details | Diff | Splinter Review |
https://searchfox.org/comm-central/search?q=nsIGConfService&case=false®exp=false&path=
Something for the Linux heads. Maybe just remove the stuff since much of it has if (gconf)
, so it will work without it, mostly.
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/dda5855e718c
Port bug 1433685 - Remove nsGConfService; rs=bustage-fix
Assignee | ||
Comment 2•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Reporter | ||
Comment 3•5 years ago
|
||
Comment on attachment 9057408 [details] [diff] [review] 1543531-remove-gconf-service.diff Straight removal, thanks. Bug 1433685 comment #0 says: gconf, being a long deprecated GNOME 2 component, should not be dependend upon. So it's most likely OK to remove it completely.
OK, we can remove this, but we need to see if we have the replacement Gsettings backend used. In the patch I only see gIOService being used in addition. Is that the alternative to GConf?
Also, Seamonkey is probably broken too after the m-c patch.
(In reply to :aceman from comment #4)
OK, we can remove this, but we need to see if we have the replacement Gsettings backend used. In the patch I only see gIOService being used in addition. Is that the alternative to GConf?
Also, Seamonkey is probably broken too after the m-c patch.
Yes, we have a patch in progress at the moment.
Comment on attachment 9057408 [details] [diff] [review] 1543531-remove-gconf-service.diff Review of attachment 9057408 [details] [diff] [review]: ----------------------------------------------------------------- I tested this on Linux (not GNOME but KDE) and in the System integration dialog I could set Daily as default for Newsgroups and this setting did stick across restarts of TB. So it seems there is still an OS backend that we call and it stores the setting.
Comment hidden (Intermittent Failures Robot) |
Updated•5 years ago
|
Description
•