Mir

Merge lp:~afrantzis/mir/setup-platform-libs-at-installation into lp:mir

Proposed by Alexandros Frantzis
Status: Merged
Approved by: Alexandros Frantzis
Approved revision: no longer in the source branch.
Merged at revision: 1444
Proposed branch: lp:~afrantzis/mir/setup-platform-libs-at-installation
Merge into: lp:mir
Diff against target: 123 lines (+39/-9)
5 files modified
debian/rules (+3/-1)
src/client/android/CMakeLists.txt (+9/-2)
src/client/mesa/CMakeLists.txt (+9/-2)
src/platform/graphics/android/CMakeLists.txt (+9/-2)
src/platform/graphics/mesa/CMakeLists.txt (+9/-2)
To merge this branch: bzr merge lp:~afrantzis/mir/setup-platform-libs-at-installation
Reviewer Review Type Date Requested Status
Daniel van Vugt Abstain
Alan Griffiths Approve
Andreas Pokorny (community) Approve
Kevin DuBois (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+208766@code.launchpad.net

Commit message

build: Set up symlinks for platform libraries at make install

Description of the change

build: Set up symlinks for platform libraries at make install

Create symlinks to the default platform libraries (default platform = the first in the cmake MIR_PLATFORM list) so we get a working installation after 'make install'.

NB: doing a 'make install' over an existing Mir package installation will mess up the dpkg alternatives mechanism used for Mir platforms.

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
Kevin DuBois (kdub) wrote :

looks sensible, in as much as I understand the cmake code...

review: Approve
Revision history for this message
Andreas Pokorny (andreas-pokorny) wrote :

works & looks good

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

Works. (But I can't help wondering if cmake offers a clearer solution - not that I can suggest one.)

review: Approve
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Enough people seem to approve already so I'll abstain.

I will note however we still need to clean up our library names:
78 +install(TARGETS mirplatformgraphicsandroid

review: Abstain

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/rules'
2--- debian/rules 2014-02-17 09:20:04 +0000
3+++ debian/rules 2014-02-28 11:12:06 +0000
4@@ -39,7 +39,9 @@
5 dh_makeshlibs -V
6
7 override_dh_install:
8- dh_install --fail-missing
9+ dh_install --fail-missing \
10+ -Xusr/lib/$(DEB_HOST_MULTIARCH)/libmirplatformgraphics.so \
11+ -Xusr/lib/$(DEB_HOST_MULTIARCH)/libmirclientplatform.so
12 sh debian/install_ld_so_conf.sh $(DEB_HOST_MULTIARCH)
13
14 override_dh_installdeb:
15
16=== modified file 'src/client/android/CMakeLists.txt'
17--- src/client/android/CMakeLists.txt 2014-02-18 21:40:05 +0000
18+++ src/client/android/CMakeLists.txt 2014-02-28 11:12:06 +0000
19@@ -26,6 +26,8 @@
20 ${LIBHARDWARE_LIBRARIES}
21 )
22
23+install(TARGETS mirclientplatformandroid LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/clientplatform/android)
24+
25 if (MIR_TEST_PLATFORM STREQUAL "android")
26 add_custom_command(TARGET mirclientplatformandroid
27 POST_BUILD
28@@ -33,6 +35,11 @@
29 COMMAND ${CMAKE_COMMAND} -E create_symlink android/$<TARGET_FILE_NAME:mirclientplatformandroid> libmirclientplatform.so
30 WORKING_DIRECTORY ${LIBRARY_OUTPUT_PATH}
31 )
32+
33+ install(CODE
34+ "execute_process(
35+ COMMAND ln -sf mir/clientplatform/android/libmirclientplatform.so
36+ WORKING_DIRECTORY \$ENV{DESTDIR}\${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR}
37+ )"
38+ )
39 endif()
40-
41-install(TARGETS mirclientplatformandroid LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/clientplatform/android)
42
43=== modified file 'src/client/mesa/CMakeLists.txt'
44--- src/client/mesa/CMakeLists.txt 2014-02-18 21:40:05 +0000
45+++ src/client/mesa/CMakeLists.txt 2014-02-28 11:12:06 +0000
46@@ -27,6 +27,8 @@
47 ${DRM_LDFLAGS} ${DRM_LIBRARIES}
48 )
49
50+install(TARGETS mirclientplatformmesa LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/clientplatform/mesa)
51+
52 if (MIR_TEST_PLATFORM STREQUAL "mesa")
53 add_custom_command(TARGET mirclientplatformmesa
54 POST_BUILD
55@@ -34,7 +36,12 @@
56 COMMAND ${CMAKE_COMMAND} -E create_symlink mesa/$<TARGET_FILE_NAME:mirclientplatformmesa> libmirclientplatform.so
57 WORKING_DIRECTORY ${LIBRARY_OUTPUT_PATH}
58 )
59+
60+ install(CODE
61+ "execute_process(
62+ COMMAND ln -sf mir/clientplatform/mesa/libmirclientplatform.so
63+ WORKING_DIRECTORY \$ENV{DESTDIR}\${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR}
64+ )"
65+ )
66 endif()
67
68-install(TARGETS mirclientplatformmesa LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/clientplatform/mesa)
69-
70
71=== modified file 'src/platform/graphics/android/CMakeLists.txt'
72--- src/platform/graphics/android/CMakeLists.txt 2014-02-18 21:40:05 +0000
73+++ src/platform/graphics/android/CMakeLists.txt 2014-02-28 11:12:06 +0000
74@@ -51,6 +51,8 @@
75 ${GLESv2_LDFLAGS} ${GLESv2_LIBRARIES}
76 )
77
78+install(TARGETS mirplatformgraphicsandroid LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/platformgraphics/android)
79+
80 if (MIR_TEST_PLATFORM STREQUAL "android")
81 add_custom_command(TARGET mirplatformgraphicsandroid
82 POST_BUILD
83@@ -58,6 +60,11 @@
84 COMMAND ${CMAKE_COMMAND} -E create_symlink android/$<TARGET_FILE_NAME:mirplatformgraphicsandroid> libmirplatformgraphics.so
85 WORKING_DIRECTORY ${LIBRARY_OUTPUT_PATH}
86 )
87+
88+ install(CODE
89+ "execute_process(
90+ COMMAND ln -sf mir/platformgraphics/android/libmirplatformgraphics.so
91+ WORKING_DIRECTORY \$ENV{DESTDIR}\${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR}
92+ )"
93+ )
94 endif()
95-
96-install(TARGETS mirplatformgraphicsandroid LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/platformgraphics/android)
97
98=== modified file 'src/platform/graphics/mesa/CMakeLists.txt'
99--- src/platform/graphics/mesa/CMakeLists.txt 2014-02-18 21:40:05 +0000
100+++ src/platform/graphics/mesa/CMakeLists.txt 2014-02-28 11:12:06 +0000
101@@ -54,6 +54,8 @@
102 ${GLESv2_LDFLAGS} ${GLESv2_LIBRARIES}
103 )
104
105+install(TARGETS mirplatformgraphicsmesa LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/platformgraphics/mesa)
106+
107 if (MIR_TEST_PLATFORM STREQUAL "mesa")
108 add_custom_command(TARGET mirplatformgraphicsmesa
109 POST_BUILD
110@@ -61,6 +63,11 @@
111 COMMAND ${CMAKE_COMMAND} -E create_symlink mesa/$<TARGET_FILE_NAME:mirplatformgraphicsmesa> libmirplatformgraphics.so
112 WORKING_DIRECTORY ${LIBRARY_OUTPUT_PATH}
113 )
114+
115+ install(CODE
116+ "execute_process(
117+ COMMAND ln -sf mir/platformgraphics/mesa/libmirplatformgraphics.so
118+ WORKING_DIRECTORY \$ENV{DESTDIR}\${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR}
119+ )"
120+ )
121 endif()
122-
123-install(TARGETS mirplatformgraphicsmesa LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}/mir/platformgraphics/mesa)

Subscribers

People subscribed via source and target branches