Closed Bug 1543531 Opened 5 years ago Closed 5 years ago

Port |Bug 1433685 - Remove nsGConfService|

Categories

(MailNews Core :: General, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 68.0

People

(Reporter: jorgk-bmo, Assigned: darktrojan)

Details

Attachments

(1 file)

https://searchfox.org/comm-central/search?q=nsIGConfService&case=false&regexp=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.

Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(geoff)
Flags: needinfo?(acelists)

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/dda5855e718c
Port bug 1433685 - Remove nsGConfService; rs=bustage-fix

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Assignee: nobody → geoff
Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(geoff)
Flags: needinfo?(acelists)
Attachment #9057408 - Flags: review?(jorgk)
Target Milestone: --- → Thunderbird 68.0
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.
Attachment #9057408 - Flags: review?(jorgk) → review+

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.
Attachment #9057408 - Flags: review+
Type: defect → task
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: