Merge lp:~ted/indicator-location/upstart-job into lp:indicator-location/13.10

Proposed by Ted Gould
Status: Superseded
Proposed branch: lp:~ted/indicator-location/upstart-job
Merge into: lp:indicator-location/13.10
Diff against target: 138 lines (+71/-23)
6 files modified
data/CMakeLists.txt (+38/-18)
data/com.canonical.indicator.location (+3/-0)
data/indicator-location.conf.in (+8/-0)
data/indicator-location.desktop.in (+9/-0)
data/indicator-location.service.in (+0/-3)
src/phone.cc (+13/-2)
To merge this branch: bzr merge lp:~ted/indicator-location/upstart-job
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Indicator Applet Developers Pending
Review via email: mp+182489@code.launchpad.net

This proposal has been superseded by a proposal from 2013-10-29.

Commit message

Covert to upstart job

Description of the change

No description needed

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
44. By Ted Gould

Grab trunk

45. By Ted Gould

XDG Autostart File

Unmerged revisions

45. By Ted Gould

XDG Autostart File

44. By Ted Gould

Grab trunk

43. By Ted Gould

Some fixes

42. By Ted Gould

Adding in an upstart job

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/CMakeLists.txt'
2--- data/CMakeLists.txt 2013-08-26 20:34:44 +0000
3+++ data/CMakeLists.txt 2013-10-29 16:42:11 +0000
4@@ -1,22 +1,42 @@
5 ##
6-## DBus Service File
7-##
8-
9-# where to install
10-set (DBUS_SERVICE_DIR "${CMAKE_INSTALL_FULL_DATADIR}/dbus-1/services")
11-message (STATUS "${DBUS_SERVICE_DIR} is the DBus Service File install dir")
12-
13-set (SERVICE_NAME "${CMAKE_PROJECT_NAME}.service")
14-set (SERVICE_FILE "${CMAKE_CURRENT_BINARY_DIR}/${SERVICE_NAME}")
15-set (SERVICE_FILE_IN "${CMAKE_CURRENT_SOURCE_DIR}/${SERVICE_NAME}.in")
16-
17-# build it
18-set (pkglibexecdir "${CMAKE_INSTALL_FULL_PKGLIBEXECDIR}")
19-configure_file ("${SERVICE_FILE_IN}" "${SERVICE_FILE}")
20-
21-# install it
22-install (FILES "${SERVICE_FILE}"
23- DESTINATION "${DBUS_SERVICE_DIR}")
24+## Upstart Job File
25+##
26+
27+# where to install
28+set (UPSTART_JOBS_DIR "${CMAKE_INSTALL_FULL_DATADIR}/upstart/sessions")
29+message (STATUS "${UPSTART_JOBS_DIR} is the Upstart Jobs File install dir")
30+
31+set (UPSTART_JOB_NAME "${CMAKE_PROJECT_NAME}.conf")
32+set (UPSTART_JOB_FILE "${CMAKE_CURRENT_BINARY_DIR}/${UPSTART_JOB_NAME}")
33+set (UPSTART_JOB_FILE_IN "${CMAKE_CURRENT_SOURCE_DIR}/${UPSTART_JOB_NAME}.in")
34+
35+# build it
36+set (pkglibexecdir "${CMAKE_INSTALL_FULL_PKGLIBEXECDIR}")
37+configure_file ("${UPSTART_JOB_FILE_IN}" "${UPSTART_JOB_FILE}")
38+
39+# install it
40+install (FILES "${UPSTART_JOB_FILE}"
41+ DESTINATION "${UPSTART_JOBS_DIR}")
42+
43+##
44+## XDG Autostart File
45+##
46+
47+# where to install
48+set (XDG_AUTOSTART_DIR "/etc/xdg/autostart")
49+message (STATUS "${XDG_AUTOSTART_DIR} is the XDG Autostart install dir")
50+
51+set (XDG_AUTOSTART_NAME "${CMAKE_PROJECT_NAME}.desktop")
52+set (XDG_AUTOSTART_FILE "${CMAKE_CURRENT_BINARY_DIR}/${XDG_AUTOSTART_NAME}")
53+set (XDG_AUTOSTART_FILE_IN "${CMAKE_CURRENT_SOURCE_DIR}/${XDG_AUTOSTART_NAME}.in")
54+
55+# build it
56+set (pkglibexecdir "${CMAKE_INSTALL_FULL_PKGLIBEXECDIR}")
57+configure_file ("${XDG_AUTOSTART_FILE_IN}" "${XDG_AUTOSTART_FILE}")
58+
59+# install it
60+install (FILES "${XDG_AUTOSTART_FILE}"
61+ DESTINATION "${XDG_AUTOSTART_DIR}")
62
63 ##
64 ## Unity Indicator File
65
66=== modified file 'data/com.canonical.indicator.location'
67--- data/com.canonical.indicator.location 2013-09-16 13:34:29 +0000
68+++ data/com.canonical.indicator.location 2013-10-29 16:42:11 +0000
69@@ -6,3 +6,6 @@
70 [phone]
71 ObjectPath=/com/canonical/indicator/location/phone
72
73+[phone_greeter]
74+ObjectPath=/com/canonical/indicator/location/phone
75+
76
77=== added file 'data/indicator-location.conf.in'
78--- data/indicator-location.conf.in 1970-01-01 00:00:00 +0000
79+++ data/indicator-location.conf.in 2013-10-29 16:42:11 +0000
80@@ -0,0 +1,8 @@
81+description "Indicator Location Backend"
82+
83+start on indicators-loaded or indicator-services-start
84+stop on desktop-end or indicator-services-end
85+
86+respawn
87+
88+exec @pkglibexecdir@/indicator-location-service
89
90=== added file 'data/indicator-location.desktop.in'
91--- data/indicator-location.desktop.in 1970-01-01 00:00:00 +0000
92+++ data/indicator-location.desktop.in 2013-10-29 16:42:11 +0000
93@@ -0,0 +1,9 @@
94+[Desktop Entry]
95+Type=Application
96+Name=Indicator Location
97+Exec=@pkglibexecdir@/indicator-location-service
98+NotShowIn=Unity;
99+NoDisplay=true
100+StartupNotify=false
101+Terminal=false
102+
103
104=== removed file 'data/indicator-location.service.in'
105--- data/indicator-location.service.in 2013-08-11 16:43:06 +0000
106+++ data/indicator-location.service.in 1970-01-01 00:00:00 +0000
107@@ -1,3 +0,0 @@
108-[D-BUS Service]
109-Name=com.canonical.indicator.location
110-Exec=@pkglibexecdir@/indicator-location-service
111
112=== modified file 'src/phone.cc'
113--- src/phone.cc 2013-10-08 18:05:58 +0000
114+++ src/phone.cc 2013-10-29 16:42:11 +0000
115@@ -246,11 +246,22 @@
116 GMenu * menu;
117 GMenu * submenu;
118 GMenuItem * header;
119+ GMenuItem * location;
120+ GMenuItem * gps;
121
122 /* create the submenu */
123 submenu = g_menu_new ();
124- g_menu_append (submenu, _("Location detection"), "indicator." LOCATION_ACTION_KEY);
125- g_menu_append (submenu, _("GPS"), "indicator." GPS_ACTION_KEY);
126+
127+ location = g_menu_item_new (_("Location detection"), "indicator." LOCATION_ACTION_KEY);
128+ g_menu_item_set_attribute (location, "x-canonical-type", "s", "com.canonical.indicator.switch");
129+ g_menu_append_item (submenu, location);
130+ g_object_unref (location);
131+
132+ gps = g_menu_item_new (_("GPS"), "indicator." GPS_ACTION_KEY);
133+ g_menu_item_set_attribute (gps, "x-canonical-type", "s", "com.canonical.indicator.switch");
134+ g_menu_append_item (submenu, gps);
135+ g_object_unref (gps);
136+
137 // disabled for 13.04 -- the location settings panel isn't complete
138 // g_menu_append (submenu, _("Location settingsā€¦"), "indicator." SETTINGS_ACTION_KEY "::location");
139

Subscribers

People subscribed via source and target branches