Merge lp://staging/~ted/ubuntu-app-launch/lp1365408-symbolic-icon into lp://staging/ubuntu-app-launch/14.10
Proposed by
Ted Gould
| Status: | Merged |
|---|---|
| Approved by: | Charles Kerr |
| Approved revision: | 175 |
| Merged at revision: | 171 |
| Proposed branch: | lp://staging/~ted/ubuntu-app-launch/lp1365408-symbolic-icon |
| Merge into: | lp://staging/ubuntu-app-launch/14.10 |
| Diff against target: |
144 lines (+61/-25) 1 file modified
desktop-hook.c (+61/-25) |
| To merge this branch: | bzr merge lp://staging/~ted/ubuntu-app-launch/lp1365408-symbolic-icon |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Charles Kerr (community) | Approve | ||
| PS Jenkins bot (community) | continuous-integration | Approve | |
|
Review via email:
|
|||
Commit message
Add handling for X-Ubuntu-
Description of the change
The feature is that we're handling the symbolic icon just like we do the icon field. Also did some cleanups in making the fields all defines and making it so that we can specify the icon error in the recoverable error.
To post a comment you must log in.
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | # |
review:
Approve
(continuous-integration)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
PASSED: Continuous integration, rev:175
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://