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
1=== modified file 'debian/libmirclient-dev.install'
2--- debian/libmirclient-dev.install 2015-03-20 09:27:53 +0000
3+++ debian/libmirclient-dev.install 2015-03-26 04:13:20 +0000
4@@ -1,4 +1,5 @@
5 usr/include/mirclient/mir_toolkit/*.h
6+usr/include/mirclient/mir/events/*.h
7 usr/include/mirclient/mir_toolkit/events
8 usr/lib/*/pkgconfig/mirclient.pc
9 usr/lib/*/libmirclient.so
10
11=== renamed directory 'src/include/client/mir/events' => 'include/client/mir/events'
12=== modified file 'src/client/CMakeLists.txt'
13--- src/client/CMakeLists.txt 2015-03-24 16:02:48 +0000
14+++ src/client/CMakeLists.txt 2015-03-26 04:13:20 +0000
15@@ -110,6 +110,7 @@
16
17 install(
18 DIRECTORY ${CMAKE_SOURCE_DIR}/include/client/mir_toolkit
19+ ${CMAKE_SOURCE_DIR}/include/client/mir
20 DESTINATION "include/mirclient"
21 )
22
23
24=== modified file 'src/client/symbols.map'
25--- src/client/symbols.map 2015-03-24 16:02:48 +0000
26+++ src/client/symbols.map 2015-03-26 04:13:20 +0000
27@@ -181,6 +181,7 @@
28 global:
29 extern "C++" {
30 mir::events::make_event*;
31+ mir::events::add_touch*;
32 mir::input::android::Lexicon::translate*;
33 };
34 };

Subscribers

People subscribed via source and target branches