Merge lp:~pete-woods/unity-voice/valgrind-not-on-all-platforms into lp:unity-voice

Proposed by Pete Woods
Status: Merged
Approved by: Pete Woods
Approved revision: 27
Merged at revision: 27
Proposed branch: lp:~pete-woods/unity-voice/valgrind-not-on-all-platforms
Merge into: lp:unity-voice
Diff against target: 25 lines (+2/-2)
2 files modified
cmake/FindValgrind.cmake (+1/-1)
debian/control (+1/-1)
To merge this branch: bzr merge lp:~pete-woods/unity-voice/valgrind-not-on-all-platforms
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Unity Team Pending
Review via email: mp+200394@code.launchpad.net

Commit message

Valgrind is not available on all platforms

Description of the change

Valgrind is not available on all platforms

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cmake/FindValgrind.cmake'
2--- cmake/FindValgrind.cmake 2013-08-22 21:04:44 +0000
3+++ cmake/FindValgrind.cmake 2014-01-03 12:08:26 +0000
4@@ -26,7 +26,7 @@
5 )
6
7 function(add_valgrind_test NAME EXECUTABLE)
8- if(ENABLE_MEMCHECK_OPTION)
9+ if(ENABLE_MEMCHECK_OPTION AND VALGRIND_PROGRAM)
10 add_test(${NAME} ${VALGRIND_PROGRAM} ${VALGRIND_PROGRAM_OPTIONS} "${CMAKE_CURRENT_BINARY_DIR}/${EXECUTABLE}")
11 else()
12 add_test(${NAME} ${EXECUTABLE})
13
14=== modified file 'debian/control'
15--- debian/control 2013-11-28 16:10:19 +0000
16+++ debian/control 2014-01-03 12:08:26 +0000
17@@ -15,7 +15,7 @@
18 qtbase5-dev,
19 sphinx-voxforge-hmm-en,
20 sphinx-voxforge-lm-en,
21- valgrind,
22+ valgrind [!arm64 !ppc64el],
23 Standards-Version: 3.9.4
24 Homepage: https://launchpad.net/unity-voice
25 # if you don't have have commit access to this branch but would like to upload

Subscribers

People subscribed via source and target branches

to all changes: