Merge lp:~cjwatson/indicator-network-prompt/porting into lp:indicator-network-prompt

Proposed by Colin Watson
Status: Needs review
Proposed branch: lp:~cjwatson/indicator-network-prompt/porting
Merge into: lp:indicator-network-prompt
Diff against target: 12 lines (+1/-1)
1 file modified
debian/control (+1/-1)
To merge this branch: bzr merge lp:~cjwatson/indicator-network-prompt/porting
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Indicator Applet Developers Pending
Review via email: mp+210389@code.launchpad.net

Commit message

Only build-depend on valgrind on architectures that have it.

Description of the change

With Qt 5.2, indicator-network-prompt could build on all architectures, except that it build-depends on valgrind which isn't available everywhere. Since it doesn't actually use valgrind in the default build anyway (-DENABLE_MEMCHECK_OPTION=NO), it would be fine to restrict the build-dependency to architectures where it's available.

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

Unmerged revisions

21. By Colin Watson

Only build-depend on valgrind on architectures that have it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/control'
--- debian/control 2013-08-08 10:34:11 +0000
+++ debian/control 2014-03-11 12:53:26 +0000
@@ -14,7 +14,7 @@
14 pkg-config,14 pkg-config,
15 python3-dbusmock,15 python3-dbusmock,
16 qtbase5-dev,16 qtbase5-dev,
17 valgrind,17 valgrind [amd64 armhf i386 powerpc],
18Standards-Version: 3.9.418Standards-Version: 3.9.4
19Homepage: https://launchpad.net/indicator-network-prompt19Homepage: https://launchpad.net/indicator-network-prompt
20# If you aren't a member of ~unity-team but need to upload packaging changes,20# If you aren't a member of ~unity-team but need to upload packaging changes,

Subscribers

People subscribed via source and target branches