Merge lp:~xnox/unity-scope-click/systemd into lp:unity-scope-click

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: 509
Merged at revision: 507
Proposed branch: lp:~xnox/unity-scope-click/systemd
Merge into: lp:unity-scope-click
Diff against target: 49 lines (+14/-1)
4 files modified
data/CMakeLists.txt (+5/-0)
data/click-scope-departments-db.service.in (+8/-0)
debian/control (+0/-1)
debian/unity-scope-click-departmentsdb.install (+1/-0)
To merge this branch: bzr merge lp:~xnox/unity-scope-click/systemd
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Dimitri John Ledkov (community) Approve
Review via email: mp+313713@code.launchpad.net

Commit message

Add systemd unit for departments db.
Drop upstart dependency.

Description of the change

Add systemd unit for departments db.
Drop upstart dependency.

To post a comment you must log in.
Revision history for this message
Dimitri John Ledkov (xnox) :
review: Approve
lp:~xnox/unity-scope-click/systemd updated
509. By Dimitri John Ledkov

systemd doesn't substitude $HOME

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

verified that this works correctly from the bileto ppa, please approve.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Looks good. Was just wondering if removal of the upstart dependency shouldn't be changed to a dependency of systemd | upstart instead (for formality)? As we basically ship both upstart and systemd files right now. Just a thought.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/CMakeLists.txt'
--- data/CMakeLists.txt 2016-11-03 20:21:42 +0000
+++ data/CMakeLists.txt 2016-12-22 13:19:32 +0000
@@ -5,6 +5,11 @@
5configure_file(${DEPARTMENTS_UPSTART_JOB}.in ${DEPARTMENTS_UPSTART_JOB} @ONLY)5configure_file(${DEPARTMENTS_UPSTART_JOB}.in ${DEPARTMENTS_UPSTART_JOB} @ONLY)
6install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${DEPARTMENTS_UPSTART_JOB} DESTINATION ${CMAKE_INSTALL_DATADIR}/upstart/sessions)6install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${DEPARTMENTS_UPSTART_JOB} DESTINATION ${CMAKE_INSTALL_DATADIR}/upstart/sessions)
77
8# systemd unit for default departments db
9set(DEPARTMENTS_SYSTEMD_UNIT click-scope-departments-db.service)
10configure_file(${DEPARTMENTS_SYSTEMD_UNIT}.in ${DEPARTMENTS_SYSTEMD_UNIT} @ONLY)
11install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${DEPARTMENTS_SYSTEMD_UNIT} DESTINATION ${CMAKE_INSTALL_PREFIX}/lib/systemd/user)
12
8install(13install(
9 FILES clickscope-screenshot.jpg apps-scope.png ubuntu-logo.png ubuntu-store-scope.png store-scope-icon.svg14 FILES clickscope-screenshot.jpg apps-scope.png ubuntu-logo.png ubuntu-store-scope.png store-scope-icon.svg
10 DESTINATION "${APPS_DATA_DIR}"15 DESTINATION "${APPS_DATA_DIR}"
1116
=== added file 'data/click-scope-departments-db.service.in'
--- data/click-scope-departments-db.service.in 1970-01-01 00:00:00 +0000
+++ data/click-scope-departments-db.service.in 2016-12-22 13:19:32 +0000
@@ -0,0 +1,8 @@
1[Unit]
2Description=Default Click Departments Database
3PartOf=graphical-session.target
4
5[Service]
6Type=oneshot
7ExecStart=/bin/cp -n @APPS_DATA_DIR@departments.db .cache/click-departments.db
8ExecStart=@APPS_DATA_DIR@update_schema.sh .cache/click-departments.db
09
=== modified file 'debian/control'
--- debian/control 2016-10-24 21:28:14 +0000
+++ debian/control 2016-12-22 13:19:32 +0000
@@ -46,7 +46,6 @@
46 packagekit-tools,46 packagekit-tools,
47 ubuntu-app-launch-tools,47 ubuntu-app-launch-tools,
48 ubuntu-download-manager,48 ubuntu-download-manager,
49 upstart,
50 ${misc:Depends},49 ${misc:Depends},
51 ${shlibs:Depends},50 ${shlibs:Depends},
52Breaks: unity (<< 7.0),51Breaks: unity (<< 7.0),
5352
=== modified file 'debian/unity-scope-click-departmentsdb.install'
--- debian/unity-scope-click-departmentsdb.install 2014-08-05 09:11:12 +0000
+++ debian/unity-scope-click-departmentsdb.install 2016-12-22 13:19:32 +0000
@@ -1,3 +1,4 @@
1usr/share/unity/scopes/clickapps/departments.db1usr/share/unity/scopes/clickapps/departments.db
2usr/share/unity/scopes/clickapps/update_schema.sh2usr/share/unity/scopes/clickapps/update_schema.sh
3usr/share/upstart/sessions/*.conf3usr/share/upstart/sessions/*.conf
4usr/lib/systemd/user/*.service

Subscribers

People subscribed via source and target branches