Mounter docklet fills gconf with hueshift keys

Reported by Rico Tzschichholz on 2010-06-30
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Docky
Low
Chris S.
docky (Ubuntu)
Undecided
Unassigned
Lucid
Undecided
Unassigned
Maverick
Undecided
Unassigned

Bug Description

The Mounter items inherit from ColoredDockItem and creates for every mounted device a new hueshift-gconf-key which isn't even used.

Robert Dyer (psybers) on 2010-06-30
Changed in docky:
importance: Undecided → Wishlist
assignee: nobody → Chris S. (cszikszoy)
Chris S. (cszikszoy) on 2010-06-30
Changed in docky:
status: New → Confirmed
importance: Wishlist → Low
milestone: none → 2.0.5
Robert Dyer (psybers) wrote :

Rev 1565 partially fixes this, by giving a better (stable) unique id.

The question is do we want them to be colorable? If so it has to have a stable id! My fix allows this for anything with a Root. I don't really know what mounts dont have a Root, but those will still fail to hold their colors.

Changed in docky:
milestone: 2.0.5 → 2.0.6
Rico Tzschichholz (ricotz) wrote :

temporary fix with bzr1571

Changed in docky:
milestone: 2.0.6 → 2.0.7

Accepted docky into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in docky (Ubuntu Lucid):
status: New → Fix Committed
tags: added: verification-needed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package docky - 2.0.6-2

---------------
docky (2.0.6-2) unstable; urgency=low

  * Upload to unstable.

docky (2.0.6-1) experimental; urgency=low

  * New upstream release 2.0.6 "Just lovin' it",
    changes include:
    + fix check for existing instance
    + hitting ESC should cancel drags (LP: #460071)
    + make sure plugins enabled by default are added to the dock on first run
      and enable Trash docklet by default
    + update badge drawing so the text is consistently sized
    + fix leak of ThemedIcon/FileIcon in IconFromGIcon
    + fix non-square items arent properly darken (LP: #584094)
    + remove MenuList.Dispose which is not really needed, solves (LP: #609170)
    + shrink config dialog (LP: #591448)

    + RecentDocs: fix crash (LP: #580723)
    + Mounter: use a better MountItem.UniqueID and don't fill up gconf with
      useless keys (temporary fix for LP: #600311)
    + Weather: add weather icon for Smoke (LP: #612437)
    + CPUMonitor: fix CPU utilization percentage (LP: #604677)
    + GMail:
      - add debuging messages to gmail and log web response status when we
        have a web exception
      - fix crash and leak in Gmail
  * debian/patches:
    + drop 01_use_cli_for_wrapper.patch
  * patch build-system and bump dependency for new gio-sharp >= 2.22.2
    + update debian/patches/use-system-gio-sharp.patch
 -- Iain Lane <email address hidden> Tue, 07 Sep 2010 09:03:13 +0100

Changed in docky (Ubuntu Maverick):
status: New → Fix Released
Changed in docky:
milestone: 2.0.7 → none
Jean-Baptiste Lallement (jibel) wrote :

SRU verification for Lucid:
I have reproduced the problem with docky 2.0.5-0ubuntu1 in lucid-updates and have verified that the version of docky 2.0.6-0ubuntu1 in -proposed fixes the issue.

Marking as verification-done

tags: added: verification-done
removed: verification-needed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package docky - 2.0.6-0ubuntu1

---------------
docky (2.0.6-0ubuntu1) lucid-proposed; urgency=low

  * New upstream release 2.0.6 "Just lovin' it",
    changes include:
    + fix check for existing instance
    + hitting ESC should cancel drags (LP: #460071)
    + make sure plugins enabled by default are added to the dock on first run
      and enable Trash docklet by default
    + update badge drawing so the text is consistently sized
    + fix leak of ThemedIcon/FileIcon in IconFromGIcon
    + fix non-square items arent properly darken (LP: #584094)
    + remove MenuList.Dispose which is not really needed, solves (LP: #609170)
    + shrink config dialog (LP: #591448)

    + RecentDocs: fix crash (LP: #580723)
    + Mounter: use a better MountItem.UniqueID and don't fill up gconf with
      useless keys (temporary fix for LP: #600311)
    + Weather: add weather icon for Smoke (LP: #612437)
    + CPUMonitor: fix CPU utilization percentage (LP: #604677)
    + GMail:
      - add debuging messages to gmail and log web response status when we
        have a web exception
      - fix crash and leak in Gmail
 -- Rico Tzschichholz <email address hidden> Wed, 01 Sep 2010 10:20:15 +0200

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

Other bug subscribers