Merge lp:~veebers/autopilot/fix-launch_dir-arg_1331270 into lp:autopilot

Proposed by Christopher Lee
Status: Merged
Approved by: Christopher Lee
Approved revision: 496
Merged at revision: 496
Proposed branch: lp:~veebers/autopilot/fix-launch_dir-arg_1331270
Merge into: lp:autopilot
Diff against target: 21 lines (+2/-2)
1 file modified
autopilot/application/_launcher.py (+2/-2)
To merge this branch: bzr merge lp:~veebers/autopilot/fix-launch_dir-arg_1331270
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Thomi Richards (community) Approve
Review via email: mp+223660@code.launchpad.net

Commit message

Reverted NormalApplicationLauncher's launch arg to be launch_dir not cwd.

Description of the change

Revert a recent change where a named arg was changed from launch_dir to cwd.

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

PASSED: Continuous integration, rev:496
http://jenkins.qa.ubuntu.com/job/autopilot-ci/745/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-utopic-amd64-ci/19
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-utopic-amd64-ci/19/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-utopic-armhf-ci/19
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-utopic-armhf-ci/19/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-utopic-i386-ci/19
        deb: http://jenkins.qa.ubuntu.com/job/autopilot-utopic-i386-ci/19/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic-autopilot/143
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic-touch/171
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic-autopilot/235
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/1064
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/1064/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/1797
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/1797/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/6818
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/8612

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

review: Approve (continuous-integration)
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

LGTM

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'autopilot/application/_launcher.py'
2--- autopilot/application/_launcher.py 2014-06-17 04:25:28 +0000
3+++ autopilot/application/_launcher.py 2014-06-18 23:47:15 +0000
4@@ -289,7 +289,7 @@
5 """Fixture to manage launching an application."""
6 __doc__ += ApplicationLauncher.__doc__
7
8- def launch(self, application, arguments=[], app_type=None, cwd=None,
9+ def launch(self, application, arguments=[], app_type=None, launch_dir=None,
10 capture_output=True):
11 """Launch an application and return a proxy object.
12
13@@ -377,7 +377,7 @@
14 app_path, arguments = self._setup_environment(
15 app_path, app_type, arguments)
16 process = self._launch_application_process(
17- app_path, capture_output, cwd, arguments)
18+ app_path, capture_output, launch_dir, arguments)
19 proxy_object = get_proxy_object_for_existing_process(
20 dbus_bus=self.dbus_bus,
21 emulator_base=self.proxy_base,

Subscribers

People subscribed via source and target branches