Merge lp:~thomir-deactivatedaccount/autopilot/fix-packaging-bugs into lp:autopilot

Proposed by Thomi Richards
Status: Superseded
Proposed branch: lp:~thomir-deactivatedaccount/autopilot/fix-packaging-bugs
Merge into: lp:autopilot
Diff against target: 66 lines (+9/-8)
3 files modified
autopilot/tests/functional/test_autopilot_functional.py (+0/-4)
bin/autopilot (+5/-4)
debian/control (+4/-0)
To merge this branch: bzr merge lp:~thomir-deactivatedaccount/autopilot/fix-packaging-bugs
Reviewer Review Type Date Requested Status
Autopilot Hackers Pending
Review via email: mp+163842@code.launchpad.net

This proposal has been superseded by a proposal from 2013-05-15.

To post a comment you must log in.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'autopilot/tests/functional/test_autopilot_functional.py'
2--- autopilot/tests/functional/test_autopilot_functional.py 2013-05-05 02:39:14 +0000
3+++ autopilot/tests/functional/test_autopilot_functional.py 2013-05-15 05:41:25 +0000
4@@ -93,10 +93,6 @@
5 )
6
7 environment_patch = dict(DISPLAY=':0')
8- # Set PYTHONPATH always, since we can't tell what sys.path will be in the
9- # child process.
10- environment_patch['PYTHONPATH'] = ap_base_path
11-
12 bin_path = os.path.join(ap_base_path, 'bin', 'autopilot')
13 if not os.path.exists(bin_path):
14 bin_path = subprocess.check_output(['which', 'autopilot']).strip()
15
16=== modified file 'bin/autopilot'
17--- bin/autopilot 2013-05-06 23:21:59 +0000
18+++ bin/autopilot 2013-05-15 05:41:25 +0000
19@@ -226,12 +226,13 @@
20 return TestSuite(requested_tests.values())
21
22
23-def patch_python_path():
24+def maybe_patch_python_path():
25 """Prepend the current directory to sys.path to ensure that we can load & run
26- autopilot tests if the caller is in the parent directory.
27+ autopilot tests if the caller is in the parent directory, but only if we're
28+ not somewhere in /usr/...
29
30 """
31- if os.getcwd() not in sys.path:
32+ if os.getcwd() not in sys.path and not os.getcwd().startswith('/usr/'):
33 sys.path.insert(0, os.getcwd())
34
35
36@@ -302,5 +303,5 @@
37
38
39 if __name__ == "__main__":
40- patch_python_path()
41+ maybe_patch_python_path()
42 main()
43
44=== modified file 'debian/control'
45--- debian/control 2013-05-09 16:26:45 +0000
46+++ debian/control 2013-05-15 05:41:25 +0000
47@@ -35,6 +35,8 @@
48 udev,
49 Recommends: libautopilot-gtk (>= 1.3),
50 libautopilot-qt (>= 1.3),
51+ python-autopilot-trace,
52+ recordmydesktop,
53 Description: Utility to write and run integration tests easily
54 The autopilot engine enables to ease the writing of python tests
55 for your application manipulating your inputs like the mouse and
56@@ -97,8 +99,10 @@
57 Depends: ${misc:Depends},
58 ${python:Depends},
59 python-autopilot,
60+ libautopilot-gtk,
61 libautopilot-qt,
62 python-windowmocker,
63+ recordmydesktop,
64 Description: Tests for the autopilot functional test tool.
65 This package contains tests for the python-autopilot package.
66 You can use this package to verify that autopilot is functioning

Subscribers

People subscribed via source and target branches