Merge lp:~pwlars/phablet-tools/new-autopilot into lp:phablet-tools

Proposed by Paul Larson
Status: Rejected
Rejected by: Sergio Schvezov
Proposed branch: lp:~pwlars/phablet-tools/new-autopilot
Merge into: lp:phablet-tools
Diff against target: 11 lines (+1/-1)
1 file modified
phablet-test-run (+1/-1)
To merge this branch: bzr merge lp:~pwlars/phablet-tools/new-autopilot
Reviewer Review Type Date Requested Status
Sergio Schvezov Needs Fixing
Mathieu Trudel-Lapierre Abstain
PS Jenkins bot continuous-integration Needs Fixing
Review via email:
To post a comment you must log in.
91. By Paul Larson

Install autopilot-touch instead of autopilot-phablet

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

FAILED: Continuous integration, rev:91
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
Executed test runs:

Click here to trigger a rebuild:

review: Needs Fixing (continuous-integration)
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Sounds good as soon as autopilot-touch is landed in distro. Should be soon :)

review: Abstain
Revision history for this message
Sergio Schvezov (sergiusens) wrote :

Can you actually remove the PPA adds and autopilot-touch as it's installed by default in the image?

review: Needs Fixing

Unmerged revisions

91. By Paul Larson

Install autopilot-touch instead of autopilot-phablet

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'phablet-test-run'
2--- phablet-test-run 2013-05-01 23:39:36 +0000
3+++ phablet-test-run 2013-05-09 22:35:30 +0000
4@@ -198,7 +198,7 @@
5 exec_with_adb apt-get -qq update
7 # TODO: remove --force-yes once autopilot-phablet is merged
8- exec_with_adb apt-get -y -q --force-yes install autopilot-phablet
9+ exec_with_adb apt-get -y -q --force-yes install autopilot-touch
11 fi


People subscribed via source and target branches