Mir

Merge lp:~raof/mir/fix-0.6-rdepends into lp:mir/0.6

Proposed by Chris Halse Rogers
Status: Merged
Approved by: Daniel van Vugt
Approved revision: no longer in the source branch.
Merged at revision: 1816
Proposed branch: lp:~raof/mir/fix-0.6-rdepends
Merge into: lp:mir/0.6
Diff against target: 50 lines (+5/-1)
3 files modified
debian/changelog (+2/-0)
debian/control (+2/-0)
src/client/mirclient.pc.in (+1/-1)
To merge this branch: bzr merge lp:~raof/mir/fix-0.6-rdepends
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Daniel van Vugt Approve
Review via email: mp+229155@code.launchpad.net

Commit message

Fix the dependencies of mirclient-dev and mirserver-dev

Both mirclient.pc and mirserver.pc Require.Private protobuf >= 2.4.1, so add the needed dependency to libmirclient-dev and libmirserver-dev.

Fixes: https://bugs.launchpad.net/mir/+bug/1351133

Description of the change

Fix the build of our reverse dependencies.

To post a comment you must log in.
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Sounds like a significant fix. Got a bug#?

review: Needs Information
Revision history for this message
Chris Halse Rogers (raof) wrote :

Ask Kevin?

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

Needs a bug number and an entry in debian/changelog too.

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

Wrong indentation level :) s/ - / . /

7 . Added support for droping stale frames when a surface becomes exposed.
8 + - libmirprotobuf folded into new libmircommon library.

Arguably still needs a bug number too. Since we've branched 0.6 it's good to give fixes unique IDs like bug numbers.

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

Still wrong indentation level for an enhancement.

Also typo in bug number:
   Finish the removal of mirprotobuf library (LP: #1381133)

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

Conflict with changes Cemil just made :P

Revision history for this message
Cemil Azizoglu (cemil-azizoglu) wrote :

Right... merge conflict... (I did give a heads up on IRC :-) ). And note that jenkins won't run against 0.6 (unless fginther config'ed 0.6 already, but I don't think so). So we'll have to TA without a jenkins review.

Revision history for this message
Chris Halse Rogers (raof) wrote :

For what it's worth I've built these packages locally and build platform-api against them.

Revision history for this message
Daniel van Vugt (vanvugt) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
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/changelog'
2--- debian/changelog 2014-08-01 04:39:59 +0000
3+++ debian/changelog 2014-08-01 05:01:22 +0000
4@@ -27,6 +27,7 @@
5 . Added automated test cases for detecting ABI breakage.
6 . examples/fingerpaint: Enable frame dropping so it's faster and more
7 responsive.
8+ . mirprotobuf folded into new libmircommon
9 - Bugs fixed:
10 . Mir servers crash with SIGABRT - assertion failed at
11 buffer_queue.cpp:136 - "!pending_client_notifications.empty()"
12@@ -59,6 +60,7 @@
13 . Building Mir outputs lots of warnings about Android logging functions
14 in 3rd_party/ (LP: #1348467)
15 . The "shared" code should be a shared library (LP: #1341502)
16+ . Finish the removal of mirprotobuf library (LP: #1351133)
17 . Release overlay buffers when screen is turned off (LP: #1350961)
18
19 -- Cemil Azizoglu <cemil.azizoglu@canonical.com> Wed, 30 Jul 2014 16:35:00 +0300
20
21=== modified file 'debian/control'
22--- debian/control 2014-07-31 15:16:17 +0000
23+++ debian/control 2014-08-01 05:01:22 +0000
24@@ -131,6 +131,7 @@
25 Pre-Depends: ${misc:Pre-Depends}
26 Depends: libmirserver24 (= ${binary:Version}),
27 libmircommon-dev (= ${binary:Version}),
28+ libprotobuf-dev (>= 2.4.1),
29 libglm-dev,
30 libboost-program-options-dev,
31 ${misc:Depends},
32@@ -162,6 +163,7 @@
33 Pre-Depends: ${misc:Pre-Depends}
34 Depends: libmirclient8 (= ${binary:Version}),
35 libmircommon-dev (= ${binary:Version}),
36+ libprotobuf-dev (>= 2.4.1),
37 ${misc:Depends},
38 Description: Display server for Ubuntu - development headers
39 Mir is a display server running on linux systems, with a focus on efficiency,
40
41=== modified file 'src/client/mirclient.pc.in'
42--- src/client/mirclient.pc.in 2013-04-24 05:22:20 +0000
43+++ src/client/mirclient.pc.in 2014-08-01 05:01:22 +0000
44@@ -8,5 +8,5 @@
45 Version: @MIR_VERSION_MAJOR@.@MIR_VERSION_MINOR@.@MIR_VERSION_PATCH@
46 Requires.private: protobuf >= 2.4.1
47 Requires: mircommon
48-Libs: -L@LIBDIR@ -lmirclient -lmirprotobuf -lprotobuf
49+Libs: -L@LIBDIR@ -lmirclient
50 Cflags: -I@INCLUDEDIR@

Subscribers

People subscribed via source and target branches