Merge lp:~mterry/unity-lens-shopping/modern-build into lp:unity-lens-shopping

Proposed by Michael Terry
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 34
Merged at revision: 34
Proposed branch: lp:~mterry/unity-lens-shopping/modern-build
Merge into: lp:unity-lens-shopping
Diff against target: 61 lines (+8/-3)
5 files modified
data/Makefile.am (+1/-1)
data/unity-lens-shopping.service.in (+1/-1)
debian/changelog (+2/-0)
debian/rules (+3/-0)
src/Makefile.am (+1/-1)
To merge this branch: bzr merge lp:~mterry/unity-lens-shopping/modern-build
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+133580@code.launchpad.net

Commit message

Use --fail-missing and install daemon into pkglibexecdir.

Description of the change

A couple small changes to standardize the build:
* Use --fail-missing, despite it not really mattering since this is a single binary package. Still, future-proofing.
* Install the daemon into pkglibexecdir, instead of libexecdir.

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Looks excellently good :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/Makefile.am'
--- data/Makefile.am 2012-07-25 08:13:14 +0000
+++ data/Makefile.am 2012-11-08 22:13:23 +0000
@@ -6,7 +6,7 @@
6dbus_services_DATA = $(service_in_files:.service.in=.service)6dbus_services_DATA = $(service_in_files:.service.in=.service)
77
8%.service: %.service.in8%.service: %.service.in
9 $(AM_V_GEN)sed -e "s|\@libexecdir\@|$(libexecdir)|" $< > $@9 $(AM_V_GEN)sed -e "s|\@pkglibexecdir\@|$(pkglibexecdir)|" $< > $@
1010
11EXTRA_DIST = \11EXTRA_DIST = \
12 $(service_in_files)12 $(service_in_files)
1313
=== modified file 'data/unity-lens-shopping.service.in'
--- data/unity-lens-shopping.service.in 2012-07-25 08:13:14 +0000
+++ data/unity-lens-shopping.service.in 2012-11-08 22:13:23 +0000
@@ -1,3 +1,3 @@
1[D-BUS Service]1[D-BUS Service]
2Name=com.canonical.Unity.Lens.Shopping2Name=com.canonical.Unity.Lens.Shopping
3Exec=@libexecdir@/unity-shopping-daemon3Exec=@pkglibexecdir@/unity-shopping-daemon
44
=== modified file 'debian/changelog'
--- debian/changelog 2012-11-06 17:29:28 +0000
+++ debian/changelog 2012-11-08 22:13:23 +0000
@@ -1,10 +1,12 @@
1unity-lens-shopping (6.8.0-0ubuntu2) UNRELEASED; urgency=low1unity-lens-shopping (6.8.0-0ubuntu2) UNRELEASED; urgency=low
22
3 * Install daemon into pkglibexecdir, not libexecdir
3 * debian/control:4 * debian/control:
4 - Update Vcs-Bzr5 - Update Vcs-Bzr
5 - Build-Depend on dh-autoreconf, gnome-common6 - Build-Depend on dh-autoreconf, gnome-common
6 * debian/rules:7 * debian/rules:
7 - Use dh_autoreconf to call ./autogen.sh8 - Use dh_autoreconf to call ./autogen.sh
9 - Use --fail-missing
810
9 -- Michael Terry <mterry@ubuntu.com> Tue, 06 Nov 2012 12:28:18 -050011 -- Michael Terry <mterry@ubuntu.com> Tue, 06 Nov 2012 12:28:18 -0500
1012
1113
=== modified file 'debian/rules'
--- debian/rules 2012-11-06 17:29:28 +0000
+++ debian/rules 2012-11-08 22:13:23 +0000
@@ -11,3 +11,6 @@
1111
12override_dh_autoreconf:12override_dh_autoreconf:
13 NOCONFIGURE=1 dh_autoreconf ./autogen.sh13 NOCONFIGURE=1 dh_autoreconf ./autogen.sh
14
15override_dh_install:
16 dh_install --fail-missing
1417
=== modified file 'src/Makefile.am'
--- src/Makefile.am 2012-10-17 14:47:27 +0000
+++ src/Makefile.am 2012-11-08 22:13:23 +0000
@@ -5,7 +5,7 @@
55
6DATADIR = $(datadir)6DATADIR = $(datadir)
77
8libexec_PROGRAMS = \8pkglibexec_PROGRAMS = \
9 unity-shopping-daemon \9 unity-shopping-daemon \
10 $(NULL)10 $(NULL)
1111

Subscribers

People subscribed via source and target branches

to all changes: