Merge lp:~compiz-team/compiz/compiz.fix_1120009.2 into lp:compiz/0.9.9
- compiz.fix_1120009.2
- Merge into 0.9.9
Status: | Superseded |
---|---|
Proposed branch: | lp:~compiz-team/compiz/compiz.fix_1120009.2 |
Merge into: | lp:compiz/0.9.9 |
Diff against target: |
422 lines (+213/-120) 10 files modified
CMakeLists.txt (+35/-38) cmake/CompizCommon.cmake (+1/-0) cmake/FindGoogleTest.cmake (+46/-0) cmake/FindXorgGTest.cmake (+43/-0) cmake/GoogleTest.cmake (+20/-0) cmake/XorgGTest.cmake (+14/-0) debian/control (+3/-0) tests/system/xorg-gtest/CMakeLists.txt (+3/-31) tests/xorg-gtest/CMakeLists.txt (+48/-0) tests/xorg-gtest/src/CMakeLists.txt (+0/-51) |
To merge this branch: | bzr merge lp:~compiz-team/compiz/compiz.fix_1120009.2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Needs Fixing | |
MC Return | Pending | ||
Brandon Schaefer | Pending | ||
Review via email: mp+148156@code.launchpad.net |
This proposal supersedes a proposal from 2013-02-12.
This proposal has been superseded by a proposal from 2013-02-13.
Commit message
Enable xorg-gtest tests by default and build in CI.
This change also refactors a number of parts of the code to find and build the tests. It puts the logic to find Google Test and Xorg GTest in their own cmake files, and also splits the find logic with the build logic. Finally, it makes the xorg-gtest variables available to all subdirectories and not just the ones under tests/
(LP: #1120009)
Description of the change
Don't merge yet.
Enable xorg-gtest tests by default and build in CI.
This change also refactors a number of parts of the code to find and build the tests. It puts the logic to find Google Test and Xorg GTest in their own cmake files, and also splits the find logic with the build logic. Finally, it makes the xorg-gtest variables available to all subdirectories and not just the ones under tests/
(LP: #1120009)
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal | # |
FAILED: Continuous integration, rev:3555
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal | # |
FAILED: Continuous integration, rev:3557
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal | # |
FAILED: Continuous integration, rev:3558
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
Brandon Schaefer (brandontschaefer) wrote : Posted in a previous version of this proposal | # |
Cool, looks good to me. Yay for fixing that mis leading warning message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal | # |
FAILED: Continuous integration, rev:3559
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal | # |
Argh. I thought we upgraded xorg-gtest in the distro to 0.8.
I'll have to leave this as WIP until that happens.
Sam Spilsbury (smspillaz) wrote : | # |
Okay, I've been informed that we have 0.7 in the distro, and that is only in raring. I've bumped the required version and mmrazik has bumped the distro in CI.
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:3560
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
Sam Spilsbury (smspillaz) wrote : | # |
Seems like the tests are failing because the xserver isn't sending back the events we expect. Debugging this now.
Unmerged revisions
Preview Diff
1 | === modified file 'CMakeLists.txt' |
2 | --- CMakeLists.txt 2012-11-05 20:50:59 +0000 |
3 | +++ CMakeLists.txt 2013-02-13 11:40:49 +0000 |
4 | @@ -22,6 +22,10 @@ |
5 | # GSettings schema recompilation |
6 | include (CompizGSettings) |
7 | |
8 | +# Google Test and Xorg-GTest |
9 | +include (GoogleTest) |
10 | +include (XorgGTest) |
11 | + |
12 | include (cmake/base.cmake) |
13 | # Check if specific function is present in library. |
14 | include (CheckFunctionExists) |
15 | @@ -137,45 +141,38 @@ |
16 | |
17 | if (COMPIZ_BUILD_TESTING) |
18 | |
19 | - # Build Google Test and make its headers known |
20 | - find_package (GTest) |
21 | - |
22 | - if (NOT GTEST_FOUND) |
23 | - |
24 | - # Check for google test and build it locally |
25 | - set (GTEST_ROOT_DIR |
26 | - "/usr/src/gtest" # Default value, adjustable by user with e.g., ccmake |
27 | - CACHE |
28 | - PATH |
29 | - "Path to Google Test srcs" |
30 | - FORCE |
31 | - ) |
32 | - |
33 | - find_path (GTEST_INCLUDE_DIR gtest/gtest.h) |
34 | - |
35 | - if (GTEST_INCLUDE_DIR) |
36 | - add_subdirectory (${GTEST_ROOT_DIR} |
37 | - gtest) |
38 | - endif (GTEST_INCLUDE_DIR) |
39 | - |
40 | - set (GTEST_BOTH_LIBRARIES gtest gtest_main) |
41 | - set (GTEST_FOUND TRUE) |
42 | - |
43 | - endif (NOT GTEST_FOUND) |
44 | - |
45 | - find_library (GMOCK_LIBRARY gmock) |
46 | - find_library (GMOCK_MAIN_LIBRARY gmock_main) |
47 | - |
48 | - if (NOT GMOCK_LIBRARY OR NOT GMOCK_MAIN_LIBRARY OR NOT GTEST_FOUND) |
49 | - |
50 | - message ("Google Mock and Google Test not found - cannot build tests!") |
51 | + find_package (GoogleTest) |
52 | + |
53 | + if (GOOGLE_TEST_AND_MOCK_FOUND) |
54 | + |
55 | + build_google_test_locally (${CMAKE_BINARY_DIR}/__gtest) |
56 | + |
57 | + if (BUILD_XORG_GTEST) |
58 | + |
59 | + find_package (XorgGTest) |
60 | + |
61 | + if (XORG_GTEST_FOUND) |
62 | + |
63 | + build_xorg_gtest_locally (${CMAKE_BINARY_DIR}/__xorg_gtest) |
64 | + |
65 | + else (XORG_GTEST_FOUND) |
66 | + |
67 | + set (BUILD_XORG_GTEST FALSE) |
68 | + |
69 | + endif (XORG_GTEST_FOUND) |
70 | + |
71 | + endif (BUILD_XORG_GTEST) |
72 | + |
73 | + else (GOOGLE_TEST_AND_MOCK_FOUND) |
74 | + |
75 | set (COMPIZ_BUILD_TESTING OFF CACHE BOOL "" FORCE) |
76 | - |
77 | - else (NOT GMOCK_LIBRARY OR NOT GMOCK_MAIN_LIBRARY OR NOT GTEST_FOUND) |
78 | - |
79 | - include_directories (${GTEST_INCLUDE_DIRS}) |
80 | - |
81 | - endif (NOT GMOCK_LIBRARY OR NOT GMOCK_MAIN_LIBRARY OR NOT GTEST_FOUND) |
82 | + set (BUILD_XORG_GTEST OFF CACHE BOOL "" FORCE) |
83 | + |
84 | + endif (GOOGLE_TEST_AND_MOCK_FOUND) |
85 | + |
86 | +else (COMPIZ_BUILD_TESTING) |
87 | + |
88 | + set (BUILD_XORG_GTEST OFF CACHE BOOL "" FORCE) |
89 | |
90 | endif (COMPIZ_BUILD_TESTING) |
91 | |
92 | |
93 | === modified file 'cmake/CompizCommon.cmake' |
94 | --- cmake/CompizCommon.cmake 2013-01-04 04:46:01 +0000 |
95 | +++ cmake/CompizCommon.cmake 2013-02-13 11:40:49 +0000 |
96 | @@ -31,6 +31,7 @@ |
97 | option (COMPIZ_RUN_LDCONFIG "Leave OFF unless you need to run ldconfig after install") |
98 | option (COMPIZ_PACKAGING_ENABLED "Enable to manually set prefix, exec_prefix, libdir, includedir, datadir" OFF) |
99 | option (COMPIZ_BUILD_TESTING "Build Unit Tests" ON) |
100 | +option (BUILD_XORG_GTEST "Build Xorg GTest integration tests" ON) |
101 | |
102 | set (COMPIZ_DATADIR ${CMAKE_INSTALL_PREFIX}/share) |
103 | set (COMPIZ_METADATADIR ${CMAKE_INSTALL_PREFIX}/share/compiz) |
104 | |
105 | === added file 'cmake/FindGoogleTest.cmake' |
106 | --- cmake/FindGoogleTest.cmake 1970-01-01 00:00:00 +0000 |
107 | +++ cmake/FindGoogleTest.cmake 2013-02-13 11:40:49 +0000 |
108 | @@ -0,0 +1,46 @@ |
109 | +# Find Google Test and Google Mock, either with pkg-config or by |
110 | +# searching the system install paths. This sets: |
111 | +# |
112 | +# GOOGLE_TEST_AND_MOCK_FOUND |
113 | +# GTEST_INCLUDE_DIR |
114 | +# GTEST_ROOT_DIR |
115 | +# GMOCK_LIBRARY |
116 | +# GMOCK_MAIN_LIBRARY |
117 | + |
118 | +find_package (GTest) |
119 | + |
120 | +if (NOT GTEST_FOUND) |
121 | + |
122 | + # Check for google test and build it locally |
123 | + set (GTEST_ROOT_DIR |
124 | + "/usr/src/gtest" # Default value, adjustable by user with e.g., ccmake |
125 | + CACHE |
126 | + PATH |
127 | + "Path to Google Test srcs" |
128 | + FORCE) |
129 | + |
130 | + find_path (GTEST_INCLUDE_DIR gtest/gtest.h) |
131 | + |
132 | + set (GTEST_BOTH_LIBRARIES gtest gtest_main) |
133 | + set (GTEST_FOUND TRUE) |
134 | + set (GTEST_LOCAL_BUILD_REQUIRED TRUE) |
135 | + |
136 | +else (NOT GTEST_FOUND) |
137 | + |
138 | + set (GTEST_LOCAL_BUILD_REQUIRED FALSE) |
139 | + |
140 | +endif (NOT GTEST_FOUND) |
141 | + |
142 | +find_library (GMOCK_LIBRARY gmock) |
143 | +find_library (GMOCK_MAIN_LIBRARY gmock_main) |
144 | + |
145 | +if (NOT GMOCK_LIBRARY OR NOT GMOCK_MAIN_LIBRARY OR NOT GTEST_FOUND) |
146 | + |
147 | + message ("Google Mock and Google Test not found - cannot build tests!") |
148 | + set (GOOGLE_TEST_AND_MOCK_FOUND OFF CACHE BOOL "" FORCE) |
149 | + |
150 | +else (NOT GMOCK_LIBRARY OR NOT GMOCK_MAIN_LIBRARY OR NOT GTEST_FOUND) |
151 | + |
152 | + set (GOOGLE_TEST_AND_MOCK_FOUND ON CACHE BOOL "" FORCE) |
153 | + |
154 | +endif (NOT GMOCK_LIBRARY OR NOT GMOCK_MAIN_LIBRARY OR NOT GTEST_FOUND) |
155 | |
156 | === added file 'cmake/FindXorgGTest.cmake' |
157 | --- cmake/FindXorgGTest.cmake 1970-01-01 00:00:00 +0000 |
158 | +++ cmake/FindXorgGTest.cmake 2013-02-13 11:40:49 +0000 |
159 | @@ -0,0 +1,43 @@ |
160 | +# Find xorg-gtest,with pkg-config. This sets: |
161 | +# |
162 | +# XORG_GTEST_FOUND |
163 | +# XORG_SERVER_INCLUDE_XORG_GTEST |
164 | +# XORG_SERVER_GTEST_SRC |
165 | + |
166 | +# xorg-gtest |
167 | +pkg_check_modules (XORG_SERVER |
168 | + xorg-gtest>=0.7.0 |
169 | + xorg-server |
170 | + x11 |
171 | + xi) |
172 | + |
173 | +if (XORG_SERVER_FOUND) |
174 | + |
175 | + execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=prefix xorg-gtest OUTPUT_VARIABLE _xorg_gtest_prefix) |
176 | + execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=includedir xorg-gtest OUTPUT_VARIABLE _xorg_gtest_include_dir) |
177 | + execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=sourcedir xorg-gtest OUTPUT_VARIABLE _xorg_gtest_source_dir) |
178 | + execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=CPPflags xorg-gtest OUTPUT_VARIABLE _xorg_gtest_cflags) |
179 | + |
180 | + string (STRIP ${_xorg_gtest_prefix} _xorg_gtest_prefix) |
181 | + string (STRIP ${_xorg_gtest_include_dir} _xorg_gtest_include_dir) |
182 | + string (STRIP ${_xorg_gtest_source_dir} _xorg_gtest_source_dir) |
183 | + string (STRIP ${_xorg_gtest_cflags} _xorg_gtest_cflags) |
184 | + |
185 | + set (XORG_SERVER_GTEST_INCLUDES ${XORG_SERVER_INCLUDE_DIRS}) |
186 | + set (XORG_SERVER_GTEST_LIBRARY_DIRS ${XORG_SERVER_LIBRARIES}) |
187 | + set (XORG_SERVER_GTEST_LIBRARIES} ${XORG_SERVER_LIBRARIES}) |
188 | + set (XORG_SERVER_INCLUDE_XORG_GTEST ${_xorg_gtest_include_dir} CACHE PATH "Path to Xorg GTest Headers") |
189 | + set (XORG_SERVER_GTEST_SRC ${_xorg_gtest_source_dir} CACHE PATH "Path to Xorg GTest Sources") |
190 | + set (XORG_SERVER_GTEST_CFLAGS ${_xorg_gtest_cflags}) |
191 | + set (XORG_SERVER_GTEST_ROOT ${CMAKE_SOURCE_DIR}/tests/xorg-gtest CACHE PATH "Path to Xorg GTest CMake sources") |
192 | + set (COMPIZ_XORG_SYSTEM_TEST_INCLUDE_DIR ${CMAKE_SOURCE_DIR}/tests/xorg-gtest/include CACHE PATH "Path to Compiz Xorg GTest Headers") |
193 | + |
194 | + message (STATUS "Found xorg-gtest sources at " ${XORG_SERVER_GTEST_SRC}) |
195 | + set (XORG_GTEST_FOUND ON) |
196 | + |
197 | +else (XORG_SERVER_FOUND) |
198 | + |
199 | + message (WARNING "Could not found xorg-gtest, can't build xserver tests") |
200 | + set (XORG_GTEST_FOUND OFF) |
201 | + |
202 | +endif (XORG_SERVER_FOUND) |
203 | |
204 | === added file 'cmake/GoogleTest.cmake' |
205 | --- cmake/GoogleTest.cmake 1970-01-01 00:00:00 +0000 |
206 | +++ cmake/GoogleTest.cmake 2013-02-13 11:40:49 +0000 |
207 | @@ -0,0 +1,20 @@ |
208 | +# GoogleTest.cmake |
209 | +# build_google_test_locally (dir) takes a subdirectory |
210 | +# and builds Google Test in there locally if required |
211 | + |
212 | +function (build_google_test_locally build_directory) |
213 | + |
214 | + if (GOOGLE_TEST_AND_MOCK_FOUND) |
215 | + |
216 | + if (GTEST_LOCAL_BUILD_REQUIRED) |
217 | + |
218 | + add_subdirectory (${GTEST_ROOT_DIR} |
219 | + ${build_directory}) |
220 | + |
221 | + endif (GTEST_LOCAL_BUILD_REQUIRED) |
222 | + |
223 | + include_directories (${GTEST_INCLUDE_DIRS}) |
224 | + |
225 | + endif (GOOGLE_TEST_AND_MOCK_FOUND) |
226 | + |
227 | +endfunction () |
228 | |
229 | === added file 'cmake/XorgGTest.cmake' |
230 | --- cmake/XorgGTest.cmake 1970-01-01 00:00:00 +0000 |
231 | +++ cmake/XorgGTest.cmake 2013-02-13 11:40:49 +0000 |
232 | @@ -0,0 +1,14 @@ |
233 | +# XorgGTest.cmake |
234 | +# |
235 | +# build_xorg_gtest_locally (dir) specifies a subdirectory to |
236 | +# build xorg-gtest in locally |
237 | + |
238 | +function (build_xorg_gtest_locally build_directory) |
239 | + |
240 | + if (XORG_GTEST_FOUND) |
241 | + |
242 | + add_subdirectory (${XORG_SERVER_GTEST_ROOT} ${build_directory}) |
243 | + |
244 | + endif (XORG_GTEST_FOUND) |
245 | + |
246 | +endfunction () |
247 | |
248 | === modified file 'debian/control' |
249 | --- debian/control 2012-11-05 21:35:41 +0000 |
250 | +++ debian/control 2013-02-13 11:40:49 +0000 |
251 | @@ -34,8 +34,10 @@ |
252 | libxext-dev, |
253 | libxdamage-dev (>=1:1.0.3), |
254 | libxfixes-dev (>= 1:4.0.1), |
255 | + libxi-dev, |
256 | libxinerama-dev, |
257 | libxml-parser-perl, |
258 | + libxorg-gtest-dev, |
259 | libxrandr-dev (>= 2:1.1.0.2), |
260 | libxrender-dev (>= 1:0.9.1), |
261 | libxcursor-dev, |
262 | @@ -54,6 +56,7 @@ |
263 | python-pyrex, |
264 | gnome-control-center-dev, |
265 | gsettings-desktop-schemas-dev, |
266 | + xserver-xorg-dev (>= 0.7.0), |
267 | Standards-Version: 3.9.3 |
268 | # If you aren't a member of ~compiz-team but need to upload packaging changes, |
269 | # just go ahead. ~compiz-team will notice and sync up the code again. |
270 | |
271 | === modified file 'tests/system/xorg-gtest/CMakeLists.txt' |
272 | --- tests/system/xorg-gtest/CMakeLists.txt 2012-11-28 14:33:20 +0000 |
273 | +++ tests/system/xorg-gtest/CMakeLists.txt 2013-02-13 11:40:49 +0000 |
274 | @@ -1,33 +1,5 @@ |
275 | -include (FindPkgConfig) |
276 | - |
277 | -pkg_check_modules (XORG_SERVER xorg-gtest xorg-server x11) |
278 | - |
279 | -option (BUILD_XORG_GTEST "Build Xorg GTest integration tests" OFF) |
280 | - |
281 | -if (XORG_SERVER_FOUND AND BUILD_XORG_GTEST) |
282 | - |
283 | - execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=prefix xorg-gtest OUTPUT_VARIABLE _xorg_gtest_prefix) |
284 | - execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=includedir xorg-gtest OUTPUT_VARIABLE _xorg_gtest_include_dir) |
285 | - execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=sourcedir xorg-gtest OUTPUT_VARIABLE _xorg_gtest_source_dir) |
286 | - execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} --variable=CPPflags xorg-gtest OUTPUT_VARIABLE _xorg_gtest_cflags) |
287 | - |
288 | - string (STRIP ${_xorg_gtest_prefix} _xorg_gtest_prefix) |
289 | - string (STRIP ${_xorg_gtest_include_dir} _xorg_gtest_include_dir) |
290 | - string (STRIP ${_xorg_gtest_source_dir} _xorg_gtest_source_dir) |
291 | - string (STRIP ${_xorg_gtest_cflags} _xorg_gtest_cflags) |
292 | - |
293 | - set (XORG_SERVER_INCLUDE_XORG_GTEST ${_xorg_gtest_include_dir} CACHE PATH "Path to Xorg GTest Headers") |
294 | - set (XORG_SERVER_GTEST_SRC ${_xorg_gtest_source_dir} CACHE PATH "Path to Xorg GTest Sources") |
295 | - set (COMPIZ_XORG_SYSTEM_TEST_INCLUDE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/include CACHE PATH "Path to Compiz Xorg GTest Headers") |
296 | - |
297 | - message (STATUS "Found xorg-gtest sources at " ${XORG_SERVER_GTEST_SRC}) |
298 | - |
299 | - add_subdirectory (src) |
300 | +if (BUILD_XORG_GTEST) |
301 | + |
302 | add_subdirectory (tests) |
303 | |
304 | -else (XORG_SERVER_FOUND AND BUILD_XORG_GTEST) |
305 | - |
306 | - message (WARNING "Could not found xorg-gtest, not building system tests") |
307 | - set (BUILD_XORG_GTEST OFF) |
308 | - |
309 | -endif (XORG_SERVER_FOUND AND BUILD_XORG_GTEST) |
310 | +endif (BUILD_XORG_GTEST) |
311 | |
312 | === added directory 'tests/xorg-gtest' |
313 | === added file 'tests/xorg-gtest/CMakeLists.txt' |
314 | --- tests/xorg-gtest/CMakeLists.txt 1970-01-01 00:00:00 +0000 |
315 | +++ tests/xorg-gtest/CMakeLists.txt 2013-02-13 11:40:49 +0000 |
316 | @@ -0,0 +1,48 @@ |
317 | +set (COMPIZ_LD_LIBRARY_PATH ${CMAKE_BINARY_DIR}/src CACHE STRING "" FORCE) |
318 | +set (COMPIZ_BINARY ${CMAKE_BINARY_DIR}/src/compiz CACHE STRING "" FORCE) |
319 | + |
320 | +configure_file (${CMAKE_CURRENT_SOURCE_DIR}/src/compiz-xorg-gtest-config.h.in |
321 | + ${CMAKE_CURRENT_BINARY_DIR}/src/compiz-xorg-gtest-config.h |
322 | + @ONLY) |
323 | + |
324 | +include_directories (${XORG_SERVER_INCLUDE_XORG_GTEST} |
325 | + ${XORG_SERVER_GTEST_INCLUDES} |
326 | + ${COMPIZ_XORG_SYSTEM_TEST_INCLUDE_DIR} |
327 | + ${CMAKE_CURRENT_BINARY_DIR}/src) |
328 | + |
329 | +link_directories (${XORG_SERVER_LIBRARY_DIRS}) |
330 | + |
331 | +add_definitions (${XORG_SERVER_GTEST_CFLAGS}) |
332 | + |
333 | +set (_xorg_gtest_all_srcs |
334 | + ${XORG_SERVER_GTEST_SRC}/src/xorg-gtest-all.cpp) |
335 | + |
336 | +set (_xorg_gtest_main_srcs |
337 | + ${XORG_SERVER_GTEST_SRC}/src/xorg-gtest_main.cpp) |
338 | + |
339 | +add_library (xorg_gtest_all STATIC |
340 | + ${_xorg_gtest_all_srcs}) |
341 | + |
342 | +add_library (xorg_gtest_main STATIC |
343 | + ${_xorg_gtest_main_srcs}) |
344 | + |
345 | +add_library (compiz_xorg_gtest_system_test STATIC |
346 | + ${CMAKE_CURRENT_SOURCE_DIR}/src/compiz-xorg-gtest.cpp) |
347 | + |
348 | +target_link_libraries (xorg_gtest_all |
349 | + ${GTEST_BOTH_LIBRARIES} |
350 | + ${CMAKE_THREAD_LIBS_INIT} |
351 | + ${XORG_SERVER_GTEST_LIBRARIES}) |
352 | + |
353 | +target_link_libraries (xorg_gtest_main |
354 | + ${GTEST_BOTH_LIBRARIES} |
355 | + ${CMAKE_THREAD_LIBS_INIT} |
356 | + ${XORG_SERVER_GTEST_LIBRARIES}) |
357 | + |
358 | +target_link_libraries (compiz_xorg_gtest_system_test |
359 | + xorg_gtest_all |
360 | + xorg_gtest_main |
361 | + ${GTEST_BOTH_LIBRARIES} |
362 | + ${CMAKE_THREAD_LIBS_INIT} |
363 | + ${XORG_SERVER_LIBRARIES} |
364 | + ${X11_XI_LIBRARIES}) |
365 | |
366 | === renamed directory 'tests/system/xorg-gtest/include' => 'tests/xorg-gtest/include' |
367 | === renamed directory 'tests/system/xorg-gtest/src' => 'tests/xorg-gtest/src' |
368 | === modified file 'tests/xorg-gtest/src/CMakeLists.txt' |
369 | --- tests/system/xorg-gtest/src/CMakeLists.txt 2012-09-18 13:17:09 +0000 |
370 | +++ tests/xorg-gtest/src/CMakeLists.txt 2013-02-13 11:40:49 +0000 |
371 | @@ -1,51 +0,0 @@ |
372 | -set (COMPIZ_LD_LIBRARY_PATH ${CMAKE_BINARY_DIR}/src CACHE STRING "" FORCE) |
373 | -set (COMPIZ_BINARY ${CMAKE_BINARY_DIR}/src/compiz CACHE STRING "" FORCE) |
374 | - |
375 | -configure_file (${CMAKE_CURRENT_SOURCE_DIR}/compiz-xorg-gtest-config.h.in |
376 | - ${CMAKE_CURRENT_BINARY_DIR}/compiz-xorg-gtest-config.h |
377 | - @ONLY) |
378 | - |
379 | -add_definitions (${_xorg_gtest_cflags}) |
380 | - |
381 | -include_directories (${COMPIZ_XORG_SYSTEM_TEST_INCLUDE_DIR} |
382 | - ${XORG_SERVER_INCLUDE_DIRS} |
383 | - ${GTEST_INCLUDE_DIRS} |
384 | - ${XORG_SERVER_INCLUDE_XORG_GTEST} |
385 | - ${XORG_SERVER_GTEST_SRC} |
386 | - ${CMAKE_CURRENT_BINARY_DIR}) |
387 | - |
388 | -link_directories (${XORG_SERVER_LIBRARY_DIRS}) |
389 | - |
390 | -# This actually includes xorg-gtest-all and the defines |
391 | -set (_xorg_gtest_all_srcs |
392 | - ${XORG_SERVER_GTEST_SRC}/src/xorg-gtest-all.cpp) |
393 | - |
394 | -set (_xorg_gtest_main_srcs |
395 | - ${XORG_SERVER_GTEST_SRC}/src/xorg-gtest_main.cpp) |
396 | - |
397 | -add_library (xorg_gtest_all STATIC |
398 | - ${_xorg_gtest_all_srcs}) |
399 | - |
400 | -add_library (xorg_gtest_main STATIC |
401 | - ${_xorg_gtest_main_srcs}) |
402 | - |
403 | -add_library (compiz_xorg_gtest_system_test STATIC |
404 | - ${CMAKE_CURRENT_SOURCE_DIR}/compiz-xorg-gtest.cpp) |
405 | - |
406 | -target_link_libraries (xorg_gtest_all |
407 | - ${GTEST_BOTH_LIBRARIES} |
408 | - ${CMAKE_THREAD_LIBS_INIT} |
409 | - ${XORG_SERVER_LIBRARIES}) |
410 | - |
411 | -target_link_libraries (xorg_gtest_main |
412 | - ${GTEST_BOTH_LIBRARIES} |
413 | - ${CMAKE_THREAD_LIBS_INIT} |
414 | - ${XORG_SERVER_LIBRARIES}) |
415 | - |
416 | -target_link_libraries (compiz_xorg_gtest_system_test |
417 | - xorg_gtest_all |
418 | - xorg_gtest_main |
419 | - ${GTEST_BOTH_LIBRARIES} |
420 | - ${CMAKE_THREAD_LIBS_INIT} |
421 | - ${XORG_SERVER_LIBRARIES} |
422 | - ${X11_XI_LIBRARIES}) |
More useful tests can only be good.