Mir

Merge lp:~mterry/mir/missing-links into lp:mir

Proposed by Michael Terry
Status: Merged
Approved by: Daniel van Vugt
Approved revision: no longer in the source branch.
Merged at revision: 1528
Proposed branch: lp:~mterry/mir/missing-links
Merge into: lp:mir
Diff against target: 28 lines (+4/-0)
2 files modified
src/platform/graphics/android/CMakeLists.txt (+3/-0)
src/platform/graphics/mesa/CMakeLists.txt (+1/-0)
To merge this branch: bzr merge lp:~mterry/mir/missing-links
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Daniel van Vugt Approve
Kevin DuBois (community) Approve
Daniel d'Andrada (community) test Approve
Review via email: mp+213906@code.launchpad.net

Commit message

Add some missing library link lines to mirplatformgraphicsandroid.

Description of the change

Add some missing library link lines to mirplatformgraphicsandroid.

These were causing crashes in unity8 for me. You can see that dpkg-shlibs will actually warn about these missing libraries with something like: "symbol X used by Y found in none of the libraries."

But there is no way to make that an error, which would help avoid these problems in the future. See Debian bug https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=644791

To post a comment you must log in.
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

That's tracked on bug https://bugs.launchpad.net/mir/+bug/1301040

Better sync-up with kdbub.

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

Solves my problem!

review: Approve (test)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Kevin DuBois (kdub) wrote :

> That's tracked on bug https://bugs.launchpad.net/mir/+bug/1301040
>
> Better sync-up with kdbub.

looks good to me, lets go with this.

slight preference for
${Boost_PROGRAM_OPTIONS_LIBRARY} over ${Boost_LIBRARIES}

review: Approve
Revision history for this message
Michael Terry (mterry) wrote :

OK, updated to use the more specific Boost variable.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel van Vugt (vanvugt) :
review: Approve
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) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/platform/graphics/android/CMakeLists.txt'
2--- src/platform/graphics/android/CMakeLists.txt 2014-03-26 05:48:59 +0000
3+++ src/platform/graphics/android/CMakeLists.txt 2014-04-03 02:02:20 +0000
4@@ -45,9 +45,12 @@
5 target_link_libraries(
6 mirplatformgraphicsandroid
7
8+ miroptions
9 mirplatform
10 mirsharedandroid
11+ mirsharedenv
12
13+ ${Boost_PROGRAM_OPTIONS_LIBRARY}
14 ${LIBHARDWARE_LIBRARIES}
15 ${EGL_LDFLAGS} ${EGL_LIBRARIES}
16 ${GLESv2_LDFLAGS} ${GLESv2_LIBRARIES}
17
18=== modified file 'src/platform/graphics/mesa/CMakeLists.txt'
19--- src/platform/graphics/mesa/CMakeLists.txt 2014-03-10 03:02:32 +0000
20+++ src/platform/graphics/mesa/CMakeLists.txt 2014-04-03 02:02:20 +0000
21@@ -47,6 +47,7 @@
22 mirplatformgraphicsmesa
23 mirplatform
24
25+ ${Boost_PROGRAM_OPTIONS_LIBRARY}
26 ${DRM_LDFLAGS} ${DRM_LIBRARIES}
27 ${GBM_LDFLAGS} ${GBM_LIBRARIES}
28 ${EGL_LDFLAGS} ${EGL_LIBRARIES}

Subscribers

People subscribed via source and target branches