Merge lp://qastaging/~mterry/qtmir/ual-parse into lp://qastaging/qtmir
Proposed by
Michael Terry
| Status: | Work in progress |
|---|---|
| Proposed branch: | lp://qastaging/~mterry/qtmir/ual-parse |
| Merge into: | lp://qastaging/qtmir |
| Diff against target: |
45 lines (+8/-7) 2 files modified
debian/control (+1/-1) src/modules/Unity/Application/application_manager.cpp (+7/-6) |
| To merge this branch: | bzr merge lp://qastaging/~mterry/qtmir/ual-parse |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Mir development team | Pending | ||
|
Review via email:
|
|||
Commit message
Use ubuntu-app-launch to parse AppIDs, instead of doing it ourselves.
Description of the change
In an effort to allow UAL flexibility to change appid formats in the future, I'm trying to remove instances of appid parsing elsewhere, and filter it all through UAL.
This branch does cause UAL to bleed out of the upstart/ folder. But since UAL is the sole authority for what appids are and how to parse them, I'm not sure that's avoidable. Plus, I think that level of abstraction has been pushed down into UAL which also uses systemd these days.
To post a comment you must log in.
Unmerged revisions
- 609. By Michael Terry
-
Let UAL parse appids for us
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
