Provide a way to listen to registry changes
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
Ted Gould | ||
| ubuntu-app-launch (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Bug Description
So unity8's launcher is going to start grabbing application/icon info from UAL. Which is great.
However, consider these two scenarios:
1) Application A in launcher is uninstalled -- icon should disappear from launcher.
2) Application B in launcher is updated and has a new icon -- icon should update in launcher.
The current way unity8 listens for this is not great. We install a session upstart job that listens to /usr/share/
Obviously this doesn't work for snaps. And is not super elegant. If UAL could tell us that something changed (it could even be at the granularity of "anything in the entire registry"), we could drop our filewatcher job and handle situations #1 and #2 above better.
Related branches
- Pete Woods (community): Approve
- Marcus Tomlinson (community): Approve
- unity-api-1-bot: Needs Fixing (continuous-integration)
-
Diff: 1009 lines (+694/-39)16 files modifiedlibubuntu-app-launch/app-store-legacy.cpp (+172/-0)
libubuntu-app-launch/app-store-legacy.h (+14/-0)
libubuntu-app-launch/info-watcher.h (+14/-0)
libubuntu-app-launch/registry-impl.cpp (+40/-14)
libubuntu-app-launch/registry-impl.h (+17/-2)
libubuntu-app-launch/registry.cpp (+10/-0)
libubuntu-app-launch/registry.h (+19/-0)
tests/CMakeLists.txt (+10/-0)
tests/app-store-legacy.cpp (+155/-0)
tests/eventually-fixture.h (+76/-0)
tests/jobs-systemd.cpp (+2/-2)
tests/libual-cpp-test.cc (+43/-8)
tests/libual-test.cc (+2/-8)
tests/registry-mock.h (+13/-0)
tests/test-directory.h (+94/-0)
tools/ubuntu-app-watch.cpp (+13/-5)
| description: | updated |
| Launchpad Janitor (janitor) wrote | #1 |
| Changed in ubuntu-app-launch (Ubuntu): | |
| status: | New → Confirmed |
| Changed in ubuntu-app-launch (Ubuntu): | |
| status: | Confirmed → Triaged |
| importance: | Undecided → High |
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → Ted Gould (ted) |
| importance: | Undecided → High |
| milestone: | none → p1 |
| status: | New → Confirmed |
| Changed in canonical-devices-system-image: | |
| milestone: | p1 → p2 |
| Changed in canonical-devices-system-image: | |
| milestone: | p2 → u8c-z |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → In Progress |
| Launchpad Janitor (janitor) wrote | #2 |
| Changed in ubuntu-app-launch (Ubuntu): | |
| status: | Triaged → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Released |
Status changed to 'Confirmed' because the bug affects multiple users.