Merge lp:~dandrader/qtmir/removeHotspot into lp:qtmir

Proposed by Daniel d'Andrada
Status: Merged
Approved by: Lukáš Tinkl
Approved revision: 490
Merged at revision: 497
Proposed branch: lp:~dandrader/qtmir/removeHotspot
Merge into: lp:qtmir
Diff against target: 47 lines (+4/-4)
3 files modified
CMakeLists.txt (+1/-1)
debian/control (+2/-2)
debian/gles-patches/convert-to-gles.patch (+1/-1)
To merge this branch: bzr merge lp:~dandrader/qtmir/removeHotspot
Reviewer Review Type Date Requested Status
Lukáš Tinkl (community) Approve
Michael Terry (community) Abstain
Unity8 CI Bot (community) continuous-integration Needs Fixing
Review via email: mp+293629@code.launchpad.net

Commit message

Bump unity-api versions

Description of the change

Need to bump unity-api versions because of this:
https://code.launchpad.net/~dandrader/unity-api/removeHotspot/+merge/293627

No code changes are needed in qtmir as this property wasn't used here.

Goes along with:
https://code.launchpad.net/~dandrader/unity8/animatedCursors/+merge/293628

To post a comment you must log in.
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

LGTM

review: Approve
Revision history for this message
Michael Terry (mterry) wrote :

This breaks the gles patch (and thus jenkins builds). Test with "QUILT_PATCHES=debian/gles-patches quilt push -a" and see that it doesn't apply.

Just need to update debian/gles-patches/convert-to-gles.patch presumably.

review: Needs Fixing
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

> Just need to update debian/gles-patches/convert-to-gles.patch presumably.

Done.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
lp:~dandrader/qtmir/removeHotspot updated
487. By Daniel d'Andrada

Fix ProxySurfaceListModel::countChanged signal
Approved by: Unity8 CI Bot

488. By Daniel d'Andrada

Move prompt surfaces from MirSurface to Application and emit firstChanged signal

489. By CI Train Bot Account

Releasing 0.4.8+16.04.20160518.1-0ubuntu1

490. By Daniel d'Andrada

Bump unity-api versions

Only unity8 uses the changes interfaces, but we have to follow the version
change anyway

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Michael Terry (mterry) wrote :

(You fixed the build issue with the gles patch, thanks!)

review: Abstain
Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

Re-top approving

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2016-05-18 11:15:56 +0000
3+++ CMakeLists.txt 2016-05-18 20:57:44 +0000
4@@ -76,7 +76,7 @@
5 pkg_check_modules(GSETTINGS_QT REQUIRED gsettings-qt)
6 pkg_check_modules(QTDBUSTEST libqtdbustest-1 REQUIRED)
7 pkg_check_modules(QTDBUSMOCK libqtdbusmock-1 REQUIRED)
8-pkg_check_modules(APPLICATION_API REQUIRED unity-shell-application=16)
9+pkg_check_modules(APPLICATION_API REQUIRED unity-shell-application=17)
10
11 include_directories(${APPLICATION_API_INCLUDE_DIRS})
12
13
14=== modified file 'debian/control'
15--- debian/control 2016-05-17 19:41:22 +0000
16+++ debian/control 2016-05-18 20:57:44 +0000
17@@ -22,7 +22,7 @@
18 libubuntu-app-launch2-dev (>= 0.9),
19 libubuntu-application-api-dev (>= 2.1.0),
20 libudev-dev,
21- libunity-api-dev (>= 7.112),
22+ libunity-api-dev (>= 7.113),
23 liburl-dispatcher1-dev,
24 libxkbcommon-dev,
25 libxrender-dev,
26@@ -93,7 +93,7 @@
27 Conflicts: libqtmir,
28 libunity-mir1,
29 Provides: unity-application-impl,
30- unity-application-impl-16,
31+ unity-application-impl-17,
32 Description: Qt plugin for Unity specific Mir APIs
33 QtMir provides Qt/QML bindings for Mir features that are exposed through the
34 qtmir-desktop or qtmir-android QPA plugin such as Application management
35
36=== modified file 'debian/gles-patches/convert-to-gles.patch'
37--- debian/gles-patches/convert-to-gles.patch 2016-05-17 19:41:22 +0000
38+++ debian/gles-patches/convert-to-gles.patch 2016-05-18 20:57:44 +0000
39@@ -84,7 +84,7 @@
40 -Conflicts: libqtmir,
41 - libunity-mir1,
42 -Provides: unity-application-impl,
43-- unity-application-impl-16,
44+- unity-application-impl-17,
45 -Description: Qt plugin for Unity specific Mir APIs
46 - QtMir provides Qt/QML bindings for Mir features that are exposed through the
47 - qtmir-desktop or qtmir-android QPA plugin such as Application management

Subscribers

People subscribed via source and target branches