Merge lp://qastaging/~kissiel/checkbox/autopilot-venv-fix into lp://qastaging/checkbox

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3965
Merged at revision: 3967
Proposed branch: lp://qastaging/~kissiel/checkbox/autopilot-venv-fix
Merge into: lp://qastaging/checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
checkbox-touch/tests/autopilot/run (+1/-1)
To merge this branch: bzr merge lp://qastaging/~kissiel/checkbox/autopilot-venv-fix
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+269546@code.qastaging.launchpad.net

Description of the change

This MR makes autopilot use venv's python, which in turn uses venv's plainbox.

As stated in the commit message:
This patch makes invocation of autopilot tests (through ./tests/autopilot/run)
to use python3 that is available in the current environment. The default file
that's used when invoking autopilot (`/usr/bin/autopilot`) is a generated file
with shebang pointing to the system wide python interpreter, which - when
cannot import plainbox (or worse, uses out-of-date one).
This resulted in _get_plainbox_qml_modules_path() method trying to find
Plainbox qml module in the `./lib/(...)` directory instead of the venv one.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches