Merge lp:~mterry/unity-asset-pool/fail-missing into lp:unity-asset-pool

Proposed by Michael Terry
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 62
Merged at revision: 62
Proposed branch: lp:~mterry/unity-asset-pool/fail-missing
Merge into: lp:unity-asset-pool
Diff against target: 48 lines (+6/-3)
4 files modified
debian/changelog (+3/-0)
debian/compat (+1/-1)
debian/control (+1/-1)
debian/rules (+1/-1)
To merge this branch: bzr merge lp:~mterry/unity-asset-pool/fail-missing
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+133399@code.launchpad.net

Commit message

Use dh9 and --fail-missing

Description of the change

Use modern build standards like dh9 and --fail-missing.

Note that the chmod changes are because dh9 will try to run executable debhelper files and use their output. So while the executable state of debian/* files didn't use to matter, it does now.

To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

(Note that while --fail-missing doesn't really do anything for this package, since it doesn't have an upstream build system, it might as well be put in place for consistency and in case a build system does get used in the future.)

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

I'm all for consistency!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog' (properties changed: +x to -x)
2--- debian/changelog 2012-11-05 22:06:56 +0000
3+++ debian/changelog 2012-11-08 03:50:24 +0000
4@@ -2,6 +2,9 @@
5
6 * debian/control:
7 - Update Vcs-Bzr
8+ - Require dh9
9+ * debian/rules:
10+ - Use --fail-missing
11 * debian/source/format:
12 - Switch to format 1.0, which works better with bzr-builddeb's
13 split mode. We don't have patches anyway.
14
15=== modified file 'debian/compat' (properties changed: +x to -x)
16--- debian/compat 2010-02-15 12:26:28 +0000
17+++ debian/compat 2012-11-08 03:50:24 +0000
18@@ -1,1 +1,1 @@
19-7
20+9
21
22=== modified file 'debian/control' (properties changed: +x to -x)
23--- debian/control 2012-11-05 22:06:56 +0000
24+++ debian/control 2012-11-08 03:50:24 +0000
25@@ -2,7 +2,7 @@
26 Section: gnome
27 Priority: optional
28 Maintainer: Kenneth Wimer <kwwii@ubuntu.com>
29-Build-Depends: debhelper (>= 7.0.50~),
30+Build-Depends: debhelper (>= 9),
31 python-scour,
32 icon-naming-utils
33 Standards-Version: 3.9.1
34
35=== modified file 'debian/copyright' (properties changed: +x to -x)
36=== modified file 'debian/install' (properties changed: +x to -x)
37=== modified file 'debian/rules'
38--- debian/rules 2010-12-13 16:19:38 +0000
39+++ debian/rules 2012-11-08 03:50:24 +0000
40@@ -4,7 +4,7 @@
41 dh $@ --with scour
42
43 override_dh_install:
44- dh_install
45+ dh_install --fail-missing
46 for d in $$(find $(CURDIR)/debian/unity-icon-theme/usr/share/icons -mindepth 2 -maxdepth 2 -type d) ; do \
47 (cd $$d; for c in *; do /usr/lib/icon-naming-utils/icon-name-mapping -c $$c; done); \
48 done

Subscribers

People subscribed via source and target branches

to all changes: