Add timeout before removing launcher

Bug #1119801 reported by Andrew Gunnerson
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Unity
Fix Released
Undecided
Unassigned
Unity for Arch Project
Fix Released
Undecided
Andrew Gunnerson
unity (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Please add a timeout to the G_FILE_MONITOR_EVENT_DELETED event before actually removing the icon from the launcher. This is necessary for some package managers, whose upgrade processes work differently from dpkg.

dpkg upgrade process:
********************************
Install /usr/share/applications/something.desktop.dpkg-new
Move /usr/share/applications/something.desktop.dpkg-new to /usr/share/applications/something.desktop
********************************

Arch's pacman upgrade progress:
********************************
Remove /usr/share/applications/something.desktop
Install /usr/share/applications/something.desktop
********************************

This causes launcher icons to disappear whenever packages are updated. A timeout would help alleviate this issue.

Btw, the discussion took place on the unity-dev mailing list here: https://lists.launchpad.net/unity-dev/msg00594.html

I'd also like to point out that this patch was not written by me. It was written by Marco Trevisan :)

Related branches

Andrew Gunnerson (cxl)
description: updated
Changed in unity:
status: New → Confirmed
Andrew Gunnerson (cxl)
Changed in unity-for-arch:
status: New → Fix Released
assignee: nobody → Xiao-Long Chen (cxl)
Changed in unity:
status: Confirmed → Fix Committed
Andrea Azzarone (azzar1)
Changed in unity:
milestone: none → 7.0.0
Changed in unity (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity - 6.12.0daily13.02.26-0ubuntu1

---------------
unity (6.12.0daily13.02.26-0ubuntu1) raring; urgency=low

  [ Andrea Azzarone ]
  * Multi-monitor - Panel and launcher visible on top of multimonitor
    non-focused fullscreen apps (LP: #748539)
  * unity.tests.launcher.test_icon_behavior.LauncherIconsTests.* fails
    randomly (LP: #1131679)

  [ Xiao-Long Chen ]
  * Add timeout before removing launcher (LP: #1119801)

  [ MC Return ]
  * cppcheck reports true positives (LP: #1131152)

  [ Automatic PS uploader ]
  * Automatic snapshot from revision 3171
 -- Automatic PS uploader <email address hidden> Tue, 26 Feb 2013 04:02:40 +0000

Changed in unity (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Stephen M. Webb (bregma) wrote :

Fix Released in Unity Unity 7.0.0 "R series".

Changed in unity:
status: Fix Committed → 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.