Merge lp:~alan-griffiths/mir/fix-libmircommon into lp:mir
- fix-libmircommon
- Merge into development-branch
Status: | Merged |
---|---|
Approved by: | Alan Griffiths |
Approved revision: | no longer in the source branch. |
Merged at revision: | 1795 |
Proposed branch: | lp:~alan-griffiths/mir/fix-libmircommon |
Merge into: | lp:mir |
Diff against target: |
554 lines (+92/-164) 24 files modified
debian/control (+0/-28) debian/libmirprotobuf-dev.install (+0/-2) debian/libmirprotobuf0.install (+0/-2) include/shared/mir/udev/wrapper.h (+0/-1) server-ABI-sha1sums (+1/-1) src/client/CMakeLists.txt (+4/-8) src/client/lttng/CMakeLists.txt (+2/-0) src/client/rpc/CMakeLists.txt (+1/-8) src/platform/CMakeLists.txt (+0/-3) src/platform/graphics/mesa/display.cpp (+1/-0) src/server/frontend/CMakeLists.txt (+1/-1) src/shared/CMakeLists.txt (+11/-8) src/shared/env/CMakeLists.txt (+4/-4) src/shared/fd/CMakeLists.txt (+4/-4) src/shared/geometry/CMakeLists.txt (+4/-6) src/shared/input/CMakeLists.txt (+18/-16) src/shared/input/android/CMakeLists.txt (+1/-5) src/shared/logging/CMakeLists.txt (+4/-4) src/shared/protobuf/CMakeLists.txt (+14/-37) src/shared/report/lttng/CMakeLists.txt (+4/-5) src/shared/sharedlibrary/CMakeLists.txt (+9/-11) src/shared/thread/CMakeLists.txt (+6/-10) tests/mir_test/CMakeLists.txt (+2/-0) tests/unit-tests/graphics/mesa/test_platform.cpp (+1/-0) |
To merge this branch: | bzr merge lp:~alan-griffiths/mir/fix-libmircommon |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Alexandros Frantzis (community) | Approve | ||
Kevin DuBois (community) | Approve | ||
Review via email: mp+227901@code.launchpad.net |
Commit message
shared, platform: link the shared objects into the mircommon shared library and not the libraries that link against it. (Partial fix for lp:1347522)
Description of the change
shared, platform: link the shared objects into the mircommon shared library and not the libraries that link against it
This still ends up with the android-input archive in libmirserver.so which seems a bit odd as that satisfies shared/
PS Jenkins bot (ps-jenkins) wrote : | # |
Alan Griffiths (alan-griffiths) wrote : | # |
This isn't a fix for everything in the linked bug, but does (mostly) get the .../shared/... code into libmircommon instead of a mixture of other libraries.
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1794
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1795
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Alexandros Frantzis (afrantzis) wrote : | # |
Looks good. Daniel mentioned an alternative (add_library(name OBJECT)), which perhaps is worth looking into it before landing this.
Also on my system MIR_COMMON_
Alan Griffiths (alan-griffiths) wrote : | # |
> Looks good. Daniel mentioned an alternative (add_library(name OBJECT)), which
> perhaps is worth looking into it before landing this.
Yes, trying it now.
> Also on my system MIR_COMMON_
> empty. Do we need it any more?
Not right now, but I'm not yet sure how to untangle the dependencies around android-input - I promise to remove it in a later MP if it doesn't get used.
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1798
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1800
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
FAILURE: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Alan Griffiths (alan-griffiths) wrote : | # |
-c 1792 probably fixes the error - re-approving
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1801
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Preview Diff
1 | === modified file 'debian/control' |
2 | --- debian/control 2014-07-24 14:18:00 +0000 |
3 | +++ debian/control 2014-07-25 13:59:43 +0000 |
4 | @@ -48,34 +48,6 @@ |
5 | # just go ahead. ~mir-team will notice and sync up the code again. |
6 | Vcs-Bzr: https://code.launchpad.net/~mir-team/mir/trunk |
7 | |
8 | -Package: libmirprotobuf0 |
9 | -Section: libs |
10 | -Architecture: i386 amd64 armhf arm64 |
11 | -Multi-Arch: same |
12 | -Pre-Depends: ${misc:Pre-Depends} |
13 | -Depends: ${misc:Depends}, |
14 | - ${shlibs:Depends}, |
15 | -Description: Display server for Ubuntu - protocol implementation |
16 | - Mir is a display server running on linux systems, with a focus on efficiency, |
17 | - robust operation and a well-defined driver model. |
18 | - . |
19 | - Contains the protocol's shared library. |
20 | - |
21 | -Package: libmirprotobuf-dev |
22 | -Section: libdevel |
23 | -Architecture: i386 amd64 armhf arm64 |
24 | -Multi-Arch: same |
25 | -Pre-Depends: ${misc:Pre-Depends} |
26 | -Depends: libmirprotobuf0 (= ${binary:Version}), |
27 | - libprotobuf-dev, |
28 | - ${misc:Depends}, |
29 | - ${shlibs:Depends}, |
30 | -Description: Display server for Ubuntu - protocol definition |
31 | - Mir is a display server running on linux systems, with a focus on efficiency, |
32 | - robust operation and a well-defined driver model. |
33 | - . |
34 | - Contains the protocol's definition files. |
35 | - |
36 | Package: libmirserver24 |
37 | Section: libs |
38 | Architecture: i386 amd64 armhf arm64 |
39 | |
40 | === removed file 'debian/libmirprotobuf-dev.install' |
41 | --- debian/libmirprotobuf-dev.install 2013-06-24 12:15:28 +0000 |
42 | +++ debian/libmirprotobuf-dev.install 1970-01-01 00:00:00 +0000 |
43 | @@ -1,2 +0,0 @@ |
44 | -usr/lib/*/libmirprotobuf.so |
45 | -usr/share/mir/protocol |
46 | |
47 | === removed file 'debian/libmirprotobuf0.install' |
48 | --- debian/libmirprotobuf0.install 2013-08-28 03:41:48 +0000 |
49 | +++ debian/libmirprotobuf0.install 1970-01-01 00:00:00 +0000 |
50 | @@ -1,2 +0,0 @@ |
51 | -usr/lib/*/libmirprotobuf.so.0 |
52 | - |
53 | |
54 | === added directory 'include/shared/mir/udev' |
55 | === renamed file 'include/platform/mir/udev/wrapper.h' => 'include/shared/mir/udev/wrapper.h' |
56 | --- include/platform/mir/udev/wrapper.h 2014-03-06 06:05:17 +0000 |
57 | +++ include/shared/mir/udev/wrapper.h 2014-07-25 13:59:43 +0000 |
58 | @@ -21,7 +21,6 @@ |
59 | |
60 | #include <memory> |
61 | #include <libudev.h> |
62 | -#include <mir/graphics/event_handler_register.h> |
63 | |
64 | namespace mir |
65 | { |
66 | |
67 | === modified file 'server-ABI-sha1sums' |
68 | --- server-ABI-sha1sums 2014-07-24 11:44:53 +0000 |
69 | +++ server-ABI-sha1sums 2014-07-25 13:59:43 +0000 |
70 | @@ -185,7 +185,7 @@ |
71 | 69a0f4d11e14ef67d0cdcca3a243b4ba4a918360 include/platform/mir/graphics/gl_primitive.h |
72 | f62525771ca2612959ff88498883630458513235 include/platform/mir/graphics/gl_program.h |
73 | 871e609c0fed0d566ddbaaa8ac2d7cd5c06dd09a include/platform/mir/abnormal_exit.h |
74 | -c0508e73dc0eda00276c252e2758a6d016b82aa6 include/platform/mir/udev/wrapper.h |
75 | +9f646fa12eaca5fc3b7a8fde6208673c071c8ef3 include/shared/mir/udev/wrapper.h |
76 | b45f14082c4f8b29efaa1b13de795dcb29deb738 include/platform/mir/options/option.h |
77 | f42afacda33e598f9c29f2f5b72932f75dd7d27c include/platform/mir/options/program_option.h |
78 | 5879da8056f9a4ff8960f586d27117c87b110b2a include/platform/mir/options/default_configuration.h |
79 | |
80 | === modified file 'src/client/CMakeLists.txt' |
81 | --- src/client/CMakeLists.txt 2014-07-22 20:42:40 +0000 |
82 | +++ src/client/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
83 | @@ -70,17 +70,13 @@ |
84 | |
85 | set( |
86 | MIR_CLIENT_LIBRARIES |
87 | + mirclientrpc |
88 | + mirclientlttngstatic |
89 | + mircommon |
90 | + |
91 | ${Boost_LIBRARIES} |
92 | ${CMAKE_THREAD_LIBS_INIT} |
93 | - ${PROTOBUF_LIBRARIES} |
94 | ${XKBCOMMON_LIBRARIES} |
95 | - |
96 | - mirprotobuf |
97 | - mirclientrpc |
98 | - mirclientlttngstatic |
99 | - mircommon |
100 | - |
101 | - 3rd_party |
102 | ) |
103 | |
104 | target_link_libraries( |
105 | |
106 | === modified file 'src/client/lttng/CMakeLists.txt' |
107 | --- src/client/lttng/CMakeLists.txt 2014-07-14 10:57:28 +0000 |
108 | +++ src/client/lttng/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
109 | @@ -27,6 +27,8 @@ |
110 | -ldl |
111 | ) |
112 | |
113 | +add_dependencies(mirclientlttngstatic mircommonprotobuf) |
114 | + |
115 | target_link_libraries( |
116 | mirclientlttng |
117 | |
118 | |
119 | === modified file 'src/client/rpc/CMakeLists.txt' |
120 | --- src/client/rpc/CMakeLists.txt 2014-07-22 20:42:40 +0000 |
121 | +++ src/client/rpc/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
122 | @@ -1,8 +1,3 @@ |
123 | -include_directories( |
124 | - ${PROTOBUF_INCLUDE_DIRS} |
125 | - ${CMAKE_CURRENT_BINARY_DIR} |
126 | -) |
127 | - |
128 | add_library( |
129 | mirclientrpc STATIC |
130 | |
131 | @@ -11,11 +6,9 @@ |
132 | mir_protobuf_rpc_channel.cpp |
133 | make_socket_rpc_channel.cpp |
134 | stream_socket_transport.cpp |
135 | - ${PROTO_SRCS} |
136 | ) |
137 | |
138 | target_link_libraries( |
139 | mirclientrpc |
140 | - |
141 | - ${PROTOBUF_LIBRARIES} |
142 | + mircommon |
143 | ) |
144 | |
145 | === modified file 'src/platform/CMakeLists.txt' |
146 | --- src/platform/CMakeLists.txt 2014-07-14 15:05:00 +0000 |
147 | +++ src/platform/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
148 | @@ -4,15 +4,12 @@ |
149 | add_library( |
150 | mirplatform SHARED |
151 | shared_library_loader.cpp |
152 | - udev_wrapper.cpp |
153 | ) |
154 | |
155 | target_link_libraries( |
156 | mirplatform |
157 | mirplatformgraphicscommon |
158 | mircommon |
159 | - |
160 | - ${UDEV_LDFLAGS} ${UDEV_LIBRARIES} |
161 | ) |
162 | |
163 | install(TARGETS mirplatform LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}) |
164 | |
165 | === modified file 'src/platform/graphics/mesa/display.cpp' |
166 | --- src/platform/graphics/mesa/display.cpp 2014-06-23 16:29:17 +0000 |
167 | +++ src/platform/graphics/mesa/display.cpp 2014-07-25 13:59:43 +0000 |
168 | @@ -25,6 +25,7 @@ |
169 | #include "kms_page_flipper.h" |
170 | #include "virtual_terminal.h" |
171 | #include "mir/graphics/overlapping_output_grouping.h" |
172 | +#include "mir/graphics/event_handler_register.h" |
173 | |
174 | #include "mir/graphics/display_report.h" |
175 | #include "mir/graphics/gl_context.h" |
176 | |
177 | === modified file 'src/server/frontend/CMakeLists.txt' |
178 | --- src/server/frontend/CMakeLists.txt 2014-07-14 15:05:00 +0000 |
179 | +++ src/server/frontend/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
180 | @@ -32,7 +32,7 @@ |
181 | ) |
182 | |
183 | # Fix build race condition - ensure the protobuf header is built before us. |
184 | -add_dependencies(mirfrontend mircommon) |
185 | +add_dependencies(mirfrontend mircommonprotobuf) |
186 | |
187 | target_link_libraries( |
188 | mirfrontend |
189 | |
190 | === modified file 'src/shared/CMakeLists.txt' |
191 | --- src/shared/CMakeLists.txt 2014-07-24 11:44:53 +0000 |
192 | +++ src/shared/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
193 | @@ -1,5 +1,6 @@ |
194 | set(MIR_GENERATED_INCLUDE_DIRECTORIES) |
195 | -set(MIR_COMMON_LIBRARIES) |
196 | +set(MIR_COMMON_SOURCES) |
197 | +set(MIR_COMMON_REFERENCES) |
198 | add_subdirectory(protobuf/) |
199 | |
200 | if (MIR_BUILD_PLATFORM_ANDROID) |
201 | @@ -31,20 +32,22 @@ |
202 | PARENT_SCOPE) |
203 | |
204 | |
205 | -add_library( |
206 | - mircommon SHARED |
207 | +MESSAGE( STATUS "MIR_COMMON_SOURCES. . . : ${MIR_COMMON_SOURCES}" ) |
208 | +MESSAGE( STATUS "MIR_COMMON_REFERENCES . : ${MIR_COMMON_REFERENCES}" ) |
209 | + |
210 | +add_library(mircommon |
211 | + SHARED |
212 | + ${MIR_COMMON_SOURCES} |
213 | ) |
214 | |
215 | -target_link_libraries( |
216 | - mircommon |
217 | - ${MIR_COMMON_LIBRARIES} |
218 | +target_link_libraries(mircommon |
219 | + ${MIR_COMMON_REFERENCES} |
220 | ) |
221 | |
222 | # TODO we need a place to manage ABI and related versioning but use this as placeholder |
223 | set(MIR_SHARED_ABI 1) |
224 | |
225 | -set_target_properties( |
226 | - mircommon |
227 | +set_target_properties(mircommon |
228 | |
229 | PROPERTIES |
230 | SOVERSION ${MIR_SHARED_ABI} |
231 | |
232 | === modified file 'src/shared/env/CMakeLists.txt' |
233 | --- src/shared/env/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
234 | +++ src/shared/env/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
235 | @@ -14,12 +14,12 @@ |
236 | # |
237 | # Authored by: Daniel van Vugt <daniel.van.vugt@canonical.com> |
238 | |
239 | -add_library(mirsharedenv STATIC |
240 | +add_library(mirsharedenv OBJECT |
241 | default_configuration.cpp |
242 | ) |
243 | |
244 | -list(APPEND MIR_COMMON_LIBRARIES |
245 | - mirsharedenv |
246 | +list(APPEND MIR_COMMON_SOURCES |
247 | + $<TARGET_OBJECTS:mirsharedenv> |
248 | ) |
249 | |
250 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
251 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
252 | |
253 | === modified file 'src/shared/fd/CMakeLists.txt' |
254 | --- src/shared/fd/CMakeLists.txt 2014-07-23 16:53:13 +0000 |
255 | +++ src/shared/fd/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
256 | @@ -14,12 +14,12 @@ |
257 | # |
258 | # Authored by: Kevin DuBois <kevin.dubois@canonical.com> |
259 | |
260 | -add_library(mirsharedfd STATIC |
261 | +add_library(mirsharedfd OBJECT |
262 | fd.cpp |
263 | ) |
264 | |
265 | -list(APPEND MIR_COMMON_LIBRARIES |
266 | - mirsharedfd |
267 | +list(APPEND MIR_COMMON_SOURCES |
268 | + $<TARGET_OBJECTS:mirsharedfd> |
269 | ) |
270 | |
271 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
272 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
273 | \ No newline at end of file |
274 | |
275 | === modified file 'src/shared/geometry/CMakeLists.txt' |
276 | --- src/shared/geometry/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
277 | +++ src/shared/geometry/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
278 | @@ -14,16 +14,14 @@ |
279 | # |
280 | # Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
281 | |
282 | -add_library( |
283 | - mirsharedgeometry STATIC |
284 | - |
285 | +add_library(mirsharedgeometry OBJECT |
286 | rectangle.cpp |
287 | rectangles.cpp |
288 | ostream.cpp |
289 | ) |
290 | |
291 | -list(APPEND MIR_COMMON_LIBRARIES |
292 | - mirsharedgeometry |
293 | +list(APPEND MIR_COMMON_SOURCES |
294 | + $<TARGET_OBJECTS:mirsharedgeometry> |
295 | ) |
296 | |
297 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
298 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
299 | |
300 | === modified file 'src/shared/input/CMakeLists.txt' |
301 | --- src/shared/input/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
302 | +++ src/shared/input/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
303 | @@ -24,20 +24,22 @@ |
304 | |
305 | add_subdirectory(android) |
306 | |
307 | -add_library( |
308 | - mirsharedinput STATIC |
309 | +add_library(mirsharedinput OBJECT |
310 | ${ANDROID_SHARED_INPUT_SOURCES} |
311 | -) |
312 | - |
313 | -target_link_libraries( |
314 | - mirsharedinput |
315 | - |
316 | - android-input |
317 | - ${XKBCOMMON_LIBRARIES} |
318 | -) |
319 | - |
320 | -list(APPEND MIR_COMMON_LIBRARIES |
321 | - mirsharedinput |
322 | -) |
323 | - |
324 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
325 | + udev_wrapper.cpp |
326 | +) |
327 | + |
328 | +list(APPEND MIR_COMMON_SOURCES |
329 | + $<TARGET_OBJECTS:mirsharedinput> |
330 | +) |
331 | + |
332 | +# As we put android-input in MIR_COMMON_REFERENCES it ends up linked into |
333 | +# libmirserver. But linking into libmircommon fails as it in turn needs |
334 | +# mir::mir::terminate_with_current_exception() from mirserver |
335 | +list(APPEND MIR_COMMON_REFERENCES android-input) |
336 | + |
337 | +list(APPEND MIR_COMMON_REFERENCES ${XKBCOMMON_LIBRARIES}) |
338 | +list(APPEND MIR_COMMON_REFERENCES ${UDEV_LDFLAGS} ${UDEV_LIBRARIES}) |
339 | + |
340 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
341 | +set(MIR_COMMON_REFERENCES ${MIR_COMMON_REFERENCES} PARENT_SCOPE) |
342 | |
343 | === modified file 'src/shared/input/android/CMakeLists.txt' |
344 | --- src/shared/input/android/CMakeLists.txt 2013-04-30 18:51:07 +0000 |
345 | +++ src/shared/input/android/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
346 | @@ -22,8 +22,4 @@ |
347 | ${CMAKE_CURRENT_SOURCE_DIR}/android_input_platform.cpp |
348 | ) |
349 | |
350 | -set( |
351 | - ANDROID_SHARED_INPUT_SOURCES |
352 | - ${ANDROID_SHARED_INPUT_SOURCES} |
353 | - PARENT_SCOPE |
354 | -) |
355 | +set(ANDROID_SHARED_INPUT_SOURCES ${ANDROID_SHARED_INPUT_SOURCES} PARENT_SCOPE) |
356 | |
357 | === renamed file 'src/platform/udev_wrapper.cpp' => 'src/shared/input/udev_wrapper.cpp' |
358 | === modified file 'src/shared/logging/CMakeLists.txt' |
359 | --- src/shared/logging/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
360 | +++ src/shared/logging/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
361 | @@ -14,13 +14,13 @@ |
362 | # |
363 | # Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
364 | |
365 | -add_library(mirsharedlogging STATIC |
366 | +add_library(mirsharedlogging OBJECT |
367 | dumb_console_logger.cpp |
368 | input_timestamp.cpp |
369 | ) |
370 | |
371 | -list(APPEND MIR_COMMON_LIBRARIES |
372 | - mirsharedlogging |
373 | +list(APPEND MIR_COMMON_SOURCES |
374 | + $<TARGET_OBJECTS:mirsharedlogging> |
375 | ) |
376 | |
377 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
378 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
379 | |
380 | === modified file 'src/shared/protobuf/CMakeLists.txt' |
381 | --- src/shared/protobuf/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
382 | +++ src/shared/protobuf/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
383 | @@ -13,27 +13,16 @@ |
384 | ${MIR_PROTOBUF_PROTOCOL_SPEC} |
385 | ) |
386 | |
387 | -add_library( |
388 | - mirprotobuf SHARED |
389 | - |
390 | +add_library(mirprotobuf OBJECT |
391 | google_protobuf_guard.cpp |
392 | ${GENERATED_PROTOBUF_SRCS} |
393 | - ${GENERATED_PROTOBUF_HDRS}) |
394 | - |
395 | -set(MIRPROTOBUF_ABI 0) |
396 | - |
397 | -set_target_properties( |
398 | - mirprotobuf |
399 | - |
400 | - PROPERTIES |
401 | - SOVERSION ${MIRPROTOBUF_ABI} |
402 | -) |
403 | - |
404 | -target_link_libraries( |
405 | - mirprotobuf |
406 | - |
407 | - ${PROTOBUF_LIBRARIES} |
408 | -) |
409 | + ${GENERATED_PROTOBUF_HDRS} |
410 | +) |
411 | + |
412 | +list(APPEND MIR_COMMON_SOURCES |
413 | + $<TARGET_OBJECTS:mirprotobuf> |
414 | +) |
415 | + |
416 | |
417 | # Export the include directories |
418 | list(APPEND MIR_GENERATED_INCLUDE_DIRECTORIES ${CMAKE_CURRENT_BINARY_DIR}) |
419 | @@ -43,21 +32,9 @@ |
420 | ${MIR_GENERATED_INCLUDE_DIRECTORIES} |
421 | PARENT_SCOPE) |
422 | |
423 | -# Install the protobuf library |
424 | -install( |
425 | - TARGETS mirprotobuf |
426 | - RUNTIME DESTINATION ${CMAKE_INSTALL_LIBDIR} |
427 | - LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} |
428 | - ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR} |
429 | -) |
430 | - |
431 | -install( |
432 | - FILES ${MIR_PROTOBUF_PROTOCOL_SPEC} |
433 | - DESTINATION share/mir/protocol |
434 | -) |
435 | - |
436 | -list(APPEND MIR_COMMON_LIBRARIES |
437 | - mirprotobuf |
438 | -) |
439 | - |
440 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
441 | \ No newline at end of file |
442 | +list(APPEND MIR_COMMON_REFERENCES |
443 | + ${PROTOBUF_LIBRARIES} |
444 | +) |
445 | + |
446 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
447 | +set(MIR_COMMON_REFERENCES ${MIR_COMMON_REFERENCES} PARENT_SCOPE) |
448 | |
449 | === modified file 'src/shared/report/lttng/CMakeLists.txt' |
450 | --- src/shared/report/lttng/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
451 | +++ src/shared/report/lttng/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
452 | @@ -1,13 +1,12 @@ |
453 | add_definitions(-DMIR_TRACEPOINT_LIB_INSTALL_PATH="${CMAKE_INSTALL_PREFIX}/${MIR_TRACEPOINT_LIB_INSTALL_DIR}") |
454 | |
455 | -add_library( |
456 | - mirsharedlttng STATIC |
457 | +add_library(mirsharedlttng OBJECT |
458 | |
459 | tracepoint_provider.cpp |
460 | ) |
461 | |
462 | -list(APPEND MIR_COMMON_LIBRARIES |
463 | - mirsharedlttng |
464 | +list(APPEND MIR_COMMON_SOURCES |
465 | + $<TARGET_OBJECTS:mirsharedlttng> |
466 | ) |
467 | |
468 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
469 | \ No newline at end of file |
470 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
471 | \ No newline at end of file |
472 | |
473 | === modified file 'src/shared/sharedlibrary/CMakeLists.txt' |
474 | --- src/shared/sharedlibrary/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
475 | +++ src/shared/sharedlibrary/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
476 | @@ -14,17 +14,15 @@ |
477 | # |
478 | # Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
479 | |
480 | -add_library(mirsharedsharedlibrary STATIC |
481 | +add_library(mirsharedsharedlibrary OBJECT |
482 | shared_library.cpp |
483 | ) |
484 | |
485 | -target_link_libraries( |
486 | - mirsharedsharedlibrary |
487 | - dl |
488 | -) |
489 | - |
490 | -list(APPEND MIR_COMMON_LIBRARIES |
491 | - mirsharedsharedlibrary |
492 | -) |
493 | - |
494 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
495 | +list(APPEND MIR_COMMON_SOURCES |
496 | + $<TARGET_OBJECTS:mirsharedsharedlibrary> |
497 | +) |
498 | + |
499 | +list(APPEND MIR_COMMON_REFERENCES dl) |
500 | + |
501 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
502 | +set(MIR_COMMON_REFERENCES ${MIR_COMMON_REFERENCES} PARENT_SCOPE) |
503 | |
504 | === modified file 'src/shared/thread/CMakeLists.txt' |
505 | --- src/shared/thread/CMakeLists.txt 2014-07-14 16:32:14 +0000 |
506 | +++ src/shared/thread/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
507 | @@ -14,17 +14,13 @@ |
508 | # |
509 | # Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
510 | |
511 | -add_library(mirsharedthread STATIC |
512 | +add_library(mirsharedthread OBJECT |
513 | thread_name.cpp |
514 | recursive_read_write_mutex.cpp |
515 | ) |
516 | |
517 | -target_link_libraries( |
518 | - mirsharedthread |
519 | -) |
520 | - |
521 | -list(APPEND MIR_COMMON_LIBRARIES |
522 | - mirsharedthread |
523 | -) |
524 | - |
525 | -set(MIR_COMMON_LIBRARIES ${MIR_COMMON_LIBRARIES} PARENT_SCOPE) |
526 | \ No newline at end of file |
527 | +list(APPEND MIR_COMMON_SOURCES |
528 | + $<TARGET_OBJECTS:mirsharedthread> |
529 | +) |
530 | + |
531 | +set(MIR_COMMON_SOURCES ${MIR_COMMON_SOURCES} PARENT_SCOPE) |
532 | \ No newline at end of file |
533 | |
534 | === modified file 'tests/mir_test/CMakeLists.txt' |
535 | --- tests/mir_test/CMakeLists.txt 2014-06-09 17:16:32 +0000 |
536 | +++ tests/mir_test/CMakeLists.txt 2014-07-25 13:59:43 +0000 |
537 | @@ -11,3 +11,5 @@ |
538 | wait_object.cpp |
539 | current_thread_name.cpp |
540 | ) |
541 | + |
542 | +add_dependencies(mir-test mircommonprotobuf) |
543 | |
544 | === modified file 'tests/unit-tests/graphics/mesa/test_platform.cpp' |
545 | --- tests/unit-tests/graphics/mesa/test_platform.cpp 2014-06-19 15:44:20 +0000 |
546 | +++ tests/unit-tests/graphics/mesa/test_platform.cpp 2014-07-25 13:59:43 +0000 |
547 | @@ -18,6 +18,7 @@ |
548 | |
549 | #include "mir/graphics/platform_ipc_package.h" |
550 | #include "mir/graphics/drm_authenticator.h" |
551 | +#include "mir/graphics/event_handler_register.h" |
552 | #include "src/platform/graphics/mesa/platform.h" |
553 | #include "src/platform/graphics/mesa/internal_client.h" |
554 | #include "src/server/report/null_report_factory.h" |
FAILED: Continuous integration, rev:1791 jenkins. qa.ubuntu. com/job/ mir-team- mir-development -branch- ci/2217/ jenkins. qa.ubuntu. com/job/ mir-android- utopic- i386-build/ 1051/console jenkins. qa.ubuntu. com/job/ mir-clang- utopic- amd64-build/ 1057/console jenkins. qa.ubuntu. com/job/ mir-mediumtests -utopic- touch/1043/ console jenkins. qa.ubuntu. com/job/ mir-team- mir-development -branch- utopic- amd64-ci/ 739/console jenkins. qa.ubuntu. com/job/ mir-team- mir-development -branch- utopic- armhf-ci/ 740/console jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- armhf/3592/ console
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/mir- team-mir- development- branch- ci/2217/ rebuild
http://