Mir

Merge lp://staging/~alan-griffiths/mir/dlopen-graphicsplatform into lp://staging/~mir-team/mir/trunk

Proposed by Alan Griffiths
Status: Merged
Approved by: Kevin DuBois
Approved revision: no longer in the source branch.
Merged at revision: 767
Proposed branch: lp://staging/~alan-griffiths/mir/dlopen-graphicsplatform
Merge into: lp://staging/~mir-team/mir/trunk
Diff against target: 514 lines (+260/-24)
16 files modified
CMakeLists.txt (+1/-1)
debian/libmirserver0.install (+1/-0)
examples/render_to_fb.cpp (+4/-1)
include/server/mir/graphics/platform.h (+4/-4)
include/server/mir/shared_library.h (+55/-0)
src/server/CMakeLists.txt (+3/-1)
src/server/default_server_configuration.cpp (+5/-7)
src/server/graphics/CMakeLists.txt (+2/-1)
src/server/graphics/android/CMakeLists.txt (+5/-2)
src/server/graphics/android/android_platform.cpp (+3/-3)
src/server/graphics/gbm/CMakeLists.txt (+4/-3)
src/server/graphics/gbm/gbm_platform.cpp (+1/-1)
src/server/shared_library.cpp (+55/-0)
tests/integration-tests/CMakeLists.txt (+1/-0)
tests/unit-tests/CMakeLists.txt (+2/-0)
tests/unit-tests/shared_library_test.cpp (+114/-0)
To merge this branch: bzr merge lp://staging/~alan-griffiths/mir/dlopen-graphicsplatform
Reviewer Review Type Date Requested Status
Kevin DuBois (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Alexandros Frantzis (community) Approve
Review via email: mp+170797@code.staging.launchpad.net

Commit message

graphics, config: Dynamically load the graphics platform

Description of the change

graphics, config: Dynamically load the graphics platform

Will follow up with MP to make the library configurable

To post a comment you must log in.
Revision history for this message
Alexandros Frantzis (afrantzis) wrote :

Looks good, and works just fine locally.

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

looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches