Merge lp:~alecu/click/rename-app-launch-in-clickhook into lp:click

Proposed by Alejandro J. Cura on 2014-06-05
Status: Rejected
Rejected by: Colin Watson on 2014-06-05
Proposed branch: lp:~alecu/click/rename-app-launch-in-clickhook
Merge into: lp:click
Diff against target: 9 lines (+1/-1)
1 file modified
debian/ (+1/-1)
To merge this branch: bzr merge lp:~alecu/click/rename-app-launch-in-clickhook
Reviewer Review Type Date Requested Status
Colin Watson 2014-06-05 Disapprove on 2014-06-05
Review via email:

Commit message

Rename upstart-app-launch to ubuntu-app-launch in the desktop hook

To post a comment you must log in.
Colin Watson (cjwatson) wrote :

I prefer the approach in; that's consistent with how I've handled this elsewhere.

review: Disapprove

Unmerged revisions

430. By Alejandro J. Cura on 2014-06-05

Rename upstart-app-launch to ubuntu-app-launch in the desktop hook

429. By PS Jenkins bot on 2014-06-04

Releasing 0.4.24

428. By Colin Watson on 2014-06-04

Click 0.4.24: Add 14.10 framework to "click chroot"; fix getpwnam caching; handle renaming of upstart-app-launch to ubuntu-app-launch. Fixes: 1323998

427. By PS Jenkins bot on 2014-05-20


426. By Michael Vogt on 2014-05-20

Click 0.4.23: handle framework removal, x86 chroot support, and various other small fixes. Fixes: 1271944, 1294659, 1296820, 1319153, 1320786

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/'
--- debian/ 2013-09-04 13:25:45 +0000
+++ debian/ 2014-06-05 16:11:35 +0000
@@ -1,4 +1,4 @@
1User-Level: yes1User-Level: yes
2Pattern: ${home}/.local/share/click/hooks/desktop/${id}.desktop2Pattern: ${home}/.local/share/click/hooks/desktop/${id}.desktop
3Exec: [ -e /usr/share/click/hooks/upstart-app-launch-desktop.hook ] || click desktophook3Exec: [ -e /usr/share/click/hooks/ubuntu-app-launch-desktop.hook ] || click desktophook
4Hook-Name: desktop4Hook-Name: desktop


People subscribed via source and target branches