Merge lp://staging/~kissiel/checkbox/autopilot-venv-fix into lp://staging/checkbox
Proposed by
Maciej Kisielewski
Status: | Merged |
---|---|
Approved by: | Zygmunt Krynicki |
Approved revision: | 3965 |
Merged at revision: | 3967 |
Proposed branch: | lp://staging/~kissiel/checkbox/autopilot-venv-fix |
Merge into: | lp://staging/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://staging/~kissiel/checkbox/autopilot-venv-fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Zygmunt Krynicki (community) | Approve | ||
Review via email: mp+269546@code.staging.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/
to use python3 that is available in the current environment. The default file
that's used when invoking autopilot (`/usr/
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_
Plainbox qml module in the `./lib/(...)` directory instead of the venv one.
To post a comment you must log in.
+1