Mir

Merge lp:~alan-griffiths/mir/fix-libmircommon into lp:mir

Proposed by Alan Griffiths
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
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/input/android/android_input_lexicon.cpp etc. which are still in libmircommon. These seem to be laregely to support the *client* code via InputPlatform::create(...), not the sever code. But android-input needs mir::mir::terminate_with_current_exception() which is also in mibmirserver.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
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.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
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_LIBRARIES, as printed in the cmake output, is empty. Do we need it any more?

review: Needs Information
Revision history for this message
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_LIBRARIES, as printed in the cmake output, is
> 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.

Revision history for this message
Kevin DuBois (kdub) wrote :

looks good

review: Approve
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
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

-c 1792 probably fixes the error - re-approving

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 '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"

Subscribers

People subscribed via source and target branches