Move GConf depends to a seperate binary

Bug #1550896 reported by Julien Lavergne
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pidgin (Debian)
Fix Released
Unknown
pidgin (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Currently, gconf on pidgin is only used to retrieve proxy settings from GNOME. It puts an hard depends on gconf, even if pidgin is used on a non-gnome environment. Also, it seems that this feature may be broken (see LP #878032 and https://developer.pidgin.im/ticket/13867).

The included debdiff move away the gconf files to a pidgin-gnome binary, so people not using GNOME can remove the gconf depends.

Tags: patch
Revision history for this message
Julien Lavergne (gilir) wrote :
tags: added: patch
Changed in pidgin (Ubuntu):
importance: Undecided → Wishlist
Changed in pidgin (Debian):
status: Unknown → New
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, I don't think it's worth splitting to another binary. The depends was useful at the time where the schemas registration was done on installation but isn't really needed nowadays with the triggers use

there doesn't seem to be an easy way to get dh_gconf to not add the depends though, some suggestions (sorted of by somewhat order of preference)

- create a new dh_gconf flag to not add gconf to up misc:Depends and get piding to use it

- hack around in debian/rules (e.g move the schemas away before dh_gconf is called and move it back before the deb is packed so the misc depends is not added)

- don't use misc:Depends in control (I think nothing else get added at the moment, but it feels wrong and might be of use for other things later)

- don't install the schemas (it's only used by programs using libgnome, which is deprecated for ages and only to handle custom url like <protocol>://<contact>, in practice very few users are likely to hit those case if ever)

I would be happy to sponsor one of those changes, let me know what you think

Revision history for this message
Julien Lavergne (gilir) wrote :

Thanks for the answer. Here is my comments for the different solutions :

- New dh_gconf : it's seem a bit overkill, especially if pidgin is the only one to use it. And I don't know dh enough to make this change in the near future :-)

- hack on debian/rules : probably a good idea, but I don't think I can make it clean with cdbs

- don't use misc:Depends : yes, it feels wrong. If something else use this feature in the future, it will not be obvious to re-enable it in the package (most of packages have misc:Depends).

- Considering it's seems broken, and used for a very limited use cases, I think it's safe to just remove it. I attached the updated debdiff.

Revision history for this message
Tim Lunn (darkxst) wrote :

Removing gconf seems the best option to me, the gconf url-handlers won't work on any modern DE/apps. The proxy code pulls settings from gsettings first, so that should be fine also. The other options mentioned would seem a waste of time for something that will rarely be used.

Uploaded (with one minor change). Thanks!

Changed in pidgin (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pidgin - 1:2.10.12-0ubuntu5

---------------
pidgin (1:2.10.12-0ubuntu5) xenial; urgency=medium

  [ Julien Lavergne ]
  * debian/pidgin*.install
   - Don't install gconf schemas to avoid depends on gconf (LP: #1550896).

  [ Tim Lunn ]
  * debian/control
    - Drop build-depend on gconf2 it's not need with the above change

 -- Tim Lunn <email address hidden> Thu, 10 Mar 2016 17:06:47 +1100

Changed in pidgin (Ubuntu):
status: Fix Committed → Fix Released
Changed in pidgin (Debian):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.