Merge lp:~michael-sheldon/ubuntu-keyboard/fix-qt55 into lp:ubuntu-keyboard

Proposed by Michael Sheldon
Status: Merged
Approved by: Timo Jyrinki
Approved revision: 403
Merged at revision: 411
Proposed branch: lp:~michael-sheldon/ubuntu-keyboard/fix-qt55
Merge into: lp:ubuntu-keyboard
Diff against target: 41 lines (+5/-6)
2 files modified
plugins/westernsupport/westernlanguagesplugin.h (+3/-4)
src/lib/logic/abstractlanguageplugin.h (+2/-2)
To merge this branch: bzr merge lp:~michael-sheldon/ubuntu-keyboard/fix-qt55
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Timo Jyrinki Approve
Ken VanDine packaging Pending
Review via email: mp+266707@code.launchpad.net

Commit message

Fix header ordering to build with Qt 5.5.

Description of the change

Fix header ordering to build with Qt 5.5 and add dependency on libqt5xcbqpa5 to support xvfb tests under Qt 5.5.

To post a comment you must log in.
Revision history for this message
Michael Sheldon (michael-sheldon) wrote :

Are there any related MPs required for this MP to build/function as expected? Please list.

 * Yes, this requires Qt 5.5 to land first/at the same time. For testing on Qt 5.5 a patched version of maliit-framework is also required: https://bugs.launchpad.net/ubuntu/+source/maliit-framework/+bug/1474690

Is your branch in sync with latest trunk (e.g. bzr pull lp:trunk -> no changes)

 * Yes

Did you perform an exploratory manual test run of your code change and any related functionality on device or emulator?

 * Only changes header ordering and build-deps, no code changes.

Did you successfully run all tests found in your component's Test Plan (https://wiki.ubuntu.com/Process/Merges/TestPlan/ubuntu-keyboard) on device or emulator?

 * Yes

If you changed the UI, was the change specified/approved by design?

 * No change

If you changed UI labels, did you update the pot file?

 * No change

If you changed the packaging (debian), did you add a core-dev as a reviewer to this MP?

 * Yes

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

qtbase is changed to recommend libqt5xcbqpa5 now (build ongoing), please publish this change without the debian/control change instead of waiting for Qt 5.5.

Thanks again for the fix!

review: Needs Fixing
403. By Michael Sheldon

Remove dependency on libqt5xcbqpa5 (now handled by qtbase)

Revision history for this message
Timo Jyrinki (timo-jyrinki) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/westernsupport/westernlanguagesplugin.h'
2--- plugins/westernsupport/westernlanguagesplugin.h 2015-04-28 23:17:17 +0000
3+++ plugins/westernsupport/westernlanguagesplugin.h 2015-09-03 13:40:25 +0000
4@@ -1,6 +1,9 @@
5 #ifndef WESTERNLANGUAGESPLUGIN_H
6 #define WESTERNLANGUAGESPLUGIN_H
7
8+#include <QObject>
9+#include <presage.h>
10+
11 #include "languageplugininterface.h"
12 #include "candidatescallback.h"
13 #include "westernlanguagefeatures.h"
14@@ -8,10 +11,6 @@
15 #include "abstractlanguageplugin.h"
16 #include "spellpredictworker.h"
17
18-#include <presage.h>
19-
20-#include <QObject>
21-
22 class WesternLanguageFeatures;
23 class CandidatesCallback;
24
25
26=== modified file 'src/lib/logic/abstractlanguageplugin.h'
27--- src/lib/logic/abstractlanguageplugin.h 2015-04-13 10:54:12 +0000
28+++ src/lib/logic/abstractlanguageplugin.h 2015-09-03 13:40:25 +0000
29@@ -28,10 +28,10 @@
30 #ifndef ABSTRACTLANGUAGEPLUGIN_H
31 #define ABSTRACTLANGUAGEPLUGIN_H
32
33+#include <QObject>
34+
35 #include "languageplugininterface.h"
36
37-#include <QObject>
38-
39 class AbstractLanguagePlugin : public QObject, public LanguagePluginInterface
40 {
41 Q_OBJECT

Subscribers

People subscribed via source and target branches