Mir

Merge lp:~vanvugt/mir/public-builders into lp:mir

Proposed by Daniel van Vugt
Status: Merged
Approved by: Daniel van Vugt
Approved revision: no longer in the source branch.
Merged at revision: 2427
Proposed branch: lp:~vanvugt/mir/public-builders
Merge into: lp:mir
Diff against target: 34 lines (+3/-0)
3 files modified
debian/libmirclient-dev.install (+1/-0)
src/client/CMakeLists.txt (+1/-0)
src/client/symbols.map (+1/-0)
To merge this branch: bzr merge lp:~vanvugt/mir/public-builders
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Kevin DuBois (community) Approve
Robert Carr (community) Approve
Alan Griffiths Approve
Review via email: mp+254044@code.launchpad.net

Commit message

Republish a missing header and missing symbol that is used by both
QtMir and USC (tests), but got lost in Mir r2408. This fixes build
failures for both QtMir and USC. (LP: #1436212)

To post a comment you must log in.
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

There are no acceptance tests for functionality supported by this header.

Is this an omission?

Or should downstream not rely on this header?

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

OK, it does look as though this is genuinely useful for writing downstream tests.

But it needs adding to libmirclient-dev.install (or, possibly better, to a new testing package)

I've logged lp:1436315 for the missing acceptance tests.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Carr (robertcarr) wrote :

+1

review: Approve
Revision history for this message
Kevin DuBois (kdub) :
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
=== modified file 'debian/libmirclient-dev.install'
--- debian/libmirclient-dev.install 2015-03-20 09:27:53 +0000
+++ debian/libmirclient-dev.install 2015-03-26 04:13:20 +0000
@@ -1,4 +1,5 @@
1usr/include/mirclient/mir_toolkit/*.h1usr/include/mirclient/mir_toolkit/*.h
2usr/include/mirclient/mir/events/*.h
2usr/include/mirclient/mir_toolkit/events3usr/include/mirclient/mir_toolkit/events
3usr/lib/*/pkgconfig/mirclient.pc4usr/lib/*/pkgconfig/mirclient.pc
4usr/lib/*/libmirclient.so5usr/lib/*/libmirclient.so
56
=== renamed directory 'src/include/client/mir/events' => 'include/client/mir/events'
=== modified file 'src/client/CMakeLists.txt'
--- src/client/CMakeLists.txt 2015-03-24 16:02:48 +0000
+++ src/client/CMakeLists.txt 2015-03-26 04:13:20 +0000
@@ -110,6 +110,7 @@
110110
111install(111install(
112 DIRECTORY ${CMAKE_SOURCE_DIR}/include/client/mir_toolkit112 DIRECTORY ${CMAKE_SOURCE_DIR}/include/client/mir_toolkit
113 ${CMAKE_SOURCE_DIR}/include/client/mir
113 DESTINATION "include/mirclient"114 DESTINATION "include/mirclient"
114)115)
115116
116117
=== modified file 'src/client/symbols.map'
--- src/client/symbols.map 2015-03-24 16:02:48 +0000
+++ src/client/symbols.map 2015-03-26 04:13:20 +0000
@@ -181,6 +181,7 @@
181 global:181 global:
182 extern "C++" { 182 extern "C++" {
183 mir::events::make_event*;183 mir::events::make_event*;
184 mir::events::add_touch*;
184 mir::input::android::Lexicon::translate*;185 mir::input::android::Lexicon::translate*;
185 }; 186 };
186};187};

Subscribers

People subscribed via source and target branches