#8268 closed (fixed)
admin_scripts test doesn't take into account the JYTHONPATH environment variable
| Reported by: | Leo Soto M. | Owned by: | Leo Soto M. |
|---|---|---|---|
| Component: | Testing framework | Version: | dev |
| Severity: | Keywords: | jython | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Very recently Jython started to recognize the JYTHONPATH environment variable.
Thus, it should be cleared before calling a jython subprocess and restored afterwards, on source:django/trunk/tests/regressiontests/admin_scripts/tests.py.
Attachments (2)
Change History (7)
comment:1 by , 16 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
by , 16 years ago
| Attachment: | 8268-r8371.diff added |
|---|
comment:3 by , 16 years ago
@russellm: Yes, it works.
I was thinking of also simplifying things by removing _sys_executable() and directly using JYTHONPATH instead of PYTHONPATH on the caller. If you like the idea, it is implemented on the attached patch, tested with both Jython and CPython.
by , 16 years ago
| Attachment: | admin_scripts_jythonpath.diff added |
|---|
slightly more invasive fix, gets rid of _sys_executable()
comment:4 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Possible patch for JYTHOPATH handling