Merge lp:~elopio/ubuntu-ui-toolkit/launch_qmlscene into lp:ubuntu-ui-toolkit

Proposed by Leo Arias
Status: Work in progress
Proposed branch: lp:~elopio/ubuntu-ui-toolkit/launch_qmlscene
Merge into: lp:ubuntu-ui-toolkit
Diff against target: 38 lines (+1/-9)
2 files modified
debian/control (+0/-1)
tests/autopilot/ubuntuuitoolkit/base.py (+1/-8)
To merge this branch: bzr merge lp:~elopio/ubuntu-ui-toolkit/launch_qmlscene
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Ubuntu SDK team Pending
Review via email: mp+192576@code.launchpad.net

Commit message

Rolled back to launch qmlscene just calling the binary on the path.

Description of the change

DO NOT MERGE YET!

To post a comment you must log in.
808. By Leo Arias

Removed the no-longer needed dependency.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:808
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/1087/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/82
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/81/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-amd64-ci/35
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/35
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-trusty-armhf-ci/35/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/73
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/82
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/82/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/81
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/81/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2708/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2759/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/457
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/454

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/ubuntu-ui-toolkit-ci/1087/rebuild

review: Needs Fixing (continuous-integration)

Unmerged revisions

808. By Leo Arias

Removed the no-longer needed dependency.

807. By Leo Arias

Try launching qmlscene directly to see if the arch is no longer needed.

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 2013-10-04 19:49:11 +0000
3+++ debian/control 2013-10-24 19:52:22 +0000
4@@ -113,7 +113,6 @@
5 Depends: ${misc:Depends},
6 ${shlibs:Depends},
7 libautopilot-qt,
8- dpkg-dev,
9 libqt5test5,
10 libqt5widgets5,
11 python-autopilot (>= 1.3),
12
13=== modified file 'tests/autopilot/ubuntuuitoolkit/base.py'
14--- tests/autopilot/ubuntuuitoolkit/base.py 2013-10-04 06:26:43 +0000
15+++ tests/autopilot/ubuntuuitoolkit/base.py 2013-10-24 19:52:22 +0000
16@@ -16,8 +16,6 @@
17
18 """Base classes for Autopilot tests using the Ubuntu UI Toolkit."""
19
20-import subprocess
21-
22 from autopilot import (
23 input,
24 platform,
25@@ -27,12 +25,7 @@
26
27 def get_qmlscene_launch_command():
28 """Return the command to launch qmlscene for autopilot tests."""
29- # We need to specify qt5 because qtchooser doesn't have a default
30- # configuration on devices and it seems the environment variable
31- # QT_SELECT=qt5 doesn't work for autopilot tests. --Mirv - 2013-10-03
32- arch = subprocess.check_output(
33- ["dpkg-architecture", "-qDEB_HOST_MULTIARCH"]).strip()
34- return '/usr/lib/' + arch + '/qt5/bin/qmlscene'
35+ return 'qmlscene'
36
37
38 class UbuntuUIToolkitAppTestCase(testcase.AutopilotTestCase):

Subscribers

People subscribed via source and target branches

to status/vote changes: