Mir

Merge lp:~vanvugt/mir/fix-1388539 into lp:mir

Proposed by Daniel van Vugt
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 2022
Proposed branch: lp:~vanvugt/mir/fix-1388539
Merge into: lp:mir
Diff against target: 59 lines (+13/-11)
3 files modified
benchmarks/CMakeLists.txt (+8/-10)
benchmarks/android-input/CMakeLists.txt (+2/-0)
benchmarks/frame-uniformity/CMakeLists.txt (+3/-1)
To merge this branch: bzr merge lp:~vanvugt/mir/fix-1388539
Reviewer Review Type Date Requested Status
Alan Griffiths Approve
Alexandros Frantzis (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+240374@code.launchpad.net

Commit message

Fix FTBFS in frame-uniformity when MIR_ENABLE_TESTS=OFF (LP: #1388539)
Also tidy up.

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

OK.

review: Approve
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

OK

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'benchmarks/CMakeLists.txt'
2--- benchmarks/CMakeLists.txt 2014-10-30 05:33:07 +0000
3+++ benchmarks/CMakeLists.txt 2014-11-02 11:47:32 +0000
4@@ -1,14 +1,12 @@
5-include_directories(${MIR_3RD_PARTY_INCLUDE_DIRECTORIES})
6-include_directories(${MIR_ANDROID_INCLUDE_DIRECTORIES})
7-add_custom_target(
8- benchmarks
9-)
10+add_custom_target(benchmarks) # XXX Is this used by anything?
11+
12+add_subdirectory(cpu)
13+add_subdirectory(memory)
14+add_dependencies(benchmarks cpu_benchmarks memory_benchmarks)
15
16 if (MIR_ENABLE_TESTS)
17+ # Shouldn't tests dependent things be in tests/?
18 add_subdirectory(android-input)
19+ add_subdirectory(frame-uniformity)
20+ add_dependencies(benchmarks frame_uniformity_test_client)
21 endif ()
22-add_subdirectory(cpu)
23-add_subdirectory(memory)
24-add_subdirectory(frame-uniformity)
25-
26-add_dependencies(benchmarks cpu_benchmarks memory_benchmarks frame_uniformity_test_client)
27
28=== modified file 'benchmarks/android-input/CMakeLists.txt'
29--- benchmarks/android-input/CMakeLists.txt 2014-10-01 06:25:56 +0000
30+++ benchmarks/android-input/CMakeLists.txt 2014-11-02 11:47:32 +0000
31@@ -7,6 +7,8 @@
32 include_directories(
33 ${Mir_SOURCE_DIR}/tests/include
34 ${Mir_SOURCE_DIR}/include/platform
35+ ${MIR_ANDROID_INCLUDE_DIRECTORIES}
36+ ${MIR_3RD_PARTY_INCLUDE_DIRECTORIES}
37 )
38
39 add_executable(input-reader-perf ${SOURCES})
40
41=== modified file 'benchmarks/frame-uniformity/CMakeLists.txt'
42--- benchmarks/frame-uniformity/CMakeLists.txt 2014-10-22 15:55:37 +0000
43+++ benchmarks/frame-uniformity/CMakeLists.txt 2014-11-02 11:47:32 +0000
44@@ -7,6 +7,8 @@
45
46 ${PROJECT_SOURCE_DIR}/src/include/platform
47 ${PROJECT_SOURCE_DIR}
48+ ${MIR_ANDROID_INCLUDE_DIRECTORIES}
49+ ${MIR_3RD_PARTY_INCLUDE_DIRECTORIES}
50 )
51
52 add_executable(frame_uniformity_test_client
53@@ -40,4 +42,4 @@
54 ${CMAKE_THREAD_LIBS_INIT} # Link in pthread.
55 )
56
57-uses_android_input(frame_uniformity_test_client)
58\ No newline at end of file
59+uses_android_input(frame_uniformity_test_client)

Subscribers

People subscribed via source and target branches