Merge lp:~lvh/divmod.org/remove-pythonpath-toxini-1269428 into lp:divmod.org

[Laurens Van Houtven]

This fixes #1269428

[Tristan Seligmann (mithrandi)]

Looks good to me. I tested it locally as well, and it seems to do the right thing now.

review: Approve

[Tristan Seligmann (mithrandi)]

Sorry, spoke too soon, it's still running the tests against the wrong Axiom locally, trying to figure out why now.

review: Needs Fixing

[Tristan Seligmann (mithrandi)]

If I do "changedir = {toxworkdir}" in tox.ini additionally, then I get the expected result (upgrader tests fail, due to missing files; I think the batch process test failure is for the same reason).

review: Needs Fixing

[lp:~lvh/divmod.org/remove-pythonpath-toxini-1269428]

2736. By Laurens Van Houtven on 2014-01-15

Also change the working directory to the tox workdir to make sure we're not secretly including PYTHONPATH that way

=== modified file 'Axiom/tox.ini'
--- Axiom/tox.ini	2014-01-15 10:06:02 +0000
+++ Axiom/tox.ini	2014-01-15 19:29:50 +0000
@@ -4,8 +4,7 @@
 [testenv]
 deps =
     Twisted
-setenv =
-    PYTHONPATH = {toxinidir}
+changedir = {toxworkdir}
 commands =
     pip install -r {toxinidir}/requirements-testing.txt
     coverage run {envdir}/bin/trial \