Merge lp://staging/~ted/ubuntu-app-launch/drop-surface-flinger into lp://staging/ubuntu-app-launch/14.04

Proposed by Ted Gould
Status: Merged
Approved by: Charles Kerr
Approved revision: 147
Merged at revision: 145
Proposed branch: lp://staging/~ted/ubuntu-app-launch/drop-surface-flinger
Merge into: lp://staging/ubuntu-app-launch/14.04
Diff against target: 186 lines (+9/-47)
6 files modified
click-exec.c (+0/-7)
desktop-exec.c (+1/-3)
exec-line-exec.c (+0/-7)
tests/exec-util-test.cc (+2/-10)
upstart-jobs/application-click.conf.in (+2/-10)
upstart-jobs/application-legacy.conf.in (+4/-10)
To merge this branch: bzr merge lp://staging/~ted/ubuntu-app-launch/drop-surface-flinger
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+213326@code.staging.launchpad.net

Commit message

Drop Surface Flinger support

Description of the change

Free us from the flinger!

To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

Looks good.

One suggestion, it would be better to use g_get_user_data_dir() than to build it by hand.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

gah! I just asked you to fix code that's being removed. Clearly you should fix it, /then/ remove it.

review: Approve
148. By Ted Gould

Merge trunk

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches