Merge lp:~kgunn72/unity-system-compositor/usc-mir-0.1.8 into lp:unity-system-compositor

Proposed by kevin gunn
Status: Rejected
Rejected by: kevin gunn
Proposed branch: lp:~kgunn72/unity-system-compositor/usc-mir-0.1.8
Merge into: lp:unity-system-compositor
Diff against target: 12 lines (+1/-1)
1 file modified
debian/control (+1/-1)
To merge this branch: bzr merge lp:~kgunn72/unity-system-compositor/usc-mir-0.1.8
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Unity System Compositor Development Team Pending
Review via email: mp+212010@code.launchpad.net

Commit message

bump debian dep to mir0.1.8

Description of the change

bump debian dep to mir0.1.8

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
kevin gunn (kgunn72) wrote :

> Why do we need changes like that? Did API change?
>
> FWIW, 0.1.8 >= 0.1.7 already, and this change only adds confusion, as unity-
> mir happily works against both 0.1.7 and 0.1.8, AFAICT. While building the
> package, dpkg will mark the runtime dependency between unity-mir and mir to
> the actual binary version of the mir it built against, and that system should
> be enough. We should only bump the mir requirement when there's actual code
> changes needed in unity-mir that would be incompatible with earlier mir.

you are right, it isn't technically needed. this only became the way-of-doing in order to have a rebuild forced, back in the old days. but now with ci train, you are right we should be able to have an empty commit rebuild since its in our control.
I'm going to start trying the empty commit method. I'll just double check with the release team in case there was some other non-obvious reason we were bumping.

Unmerged revisions

127. By kevin gunn

bump build dep to mir0.1.8

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2014-03-14 20:27:39 +0000
3+++ debian/control 2014-03-20 17:35:28 +0000
4@@ -13,7 +13,7 @@
5 libboost-system-dev,
6 libboost-thread-dev,
7 libgles2-mesa-dev,
8- libmirserver-dev (>= 0.1.7),
9+ libmirserver-dev (>= 0.1.8),
10 libprotobuf-dev,
11 pkg-config,
12 python (>= 2.7),

Subscribers

People subscribed via source and target branches