Unity 8

Merge lp:~canonical-platform-qa/unity8/remove-qt4dep into lp:unity8

  1. remove-qt4dep
  2. Merge into trunk
Proposed by Leo Arias
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 1631
Merged at revision: 1681
Proposed branch: lp:~canonical-platform-qa/unity8/remove-qt4dep
Merge into: lp:unity8
Prerequisite: lp:~canonical-platform-qa/unity8/nopy2
Diff against target: 22 lines (+3/-2)
1 file modified
debian/control (+3/-2)
To merge this branch: bzr merge lp:~canonical-platform-qa/unity8/remove-qt4dep
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Ubuntu Unity PS integration team Pending
Review via email: mp+252118@code.launchpad.net

Commit message

Changed the autopilot dependencies so they do not require qt4.

Description of the change

There are many other packages depending on libautopilot-qt. Mainly the toolkit, so this change will be useful when the other branches are changed. I'm reporting a bug about that.

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

Just the prerequisite, already approved. And added to avoid possible conflicts, not really a dep.

 * Did you perform an exploratory manual test run of your code change and any related functionality?

Ran the tests.

 * Did you make sure that your branch does not contain spurious tags?

Yes.

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?

ok.

 * If you changed the UI, has there been a design review?

No UI changes.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:1631
http://jenkins.qa.ubuntu.com/job/unity8-ci/5395/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/1681
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/559
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/560
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/560
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/1492
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1679
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1679/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/18630

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/5395/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Dependency only chain, needed stuff still is installed

 * Did CI run pass? If not, please explain why.
autopilot is flacky again

 * Did you make sure that the branch does not contain spurious tags?
Yes

review: Approve

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 2015-03-06 14:41:49 +0000
3+++ debian/control 2015-03-06 14:41:49 +0000
4@@ -137,15 +137,16 @@
5
6 Package: unity8-autopilot
7 Architecture: all
8-Depends: gir1.2-glib-2.0,
9+Depends: autopilot-qt5 (>= 1.4),
10+ gir1.2-glib-2.0,
11 gir1.2-notify-0.7,
12- libautopilot-qt (>= 1.4),
13 libqt5test5,
14 libqt5widgets5,
15 python3-autopilot,
16 python3-evdev,
17 python3-fixtures,
18 python3-gi,
19+ qttestability-autopilot (>= 1.4),
20 ubuntu-ui-toolkit-autopilot,
21 unity-scope-click,
22 unity8 (= ${source:Version}),

Subscribers

People subscribed via source and target branches