Opened 13 years ago
Closed 13 years ago
#16523 closed Bug (fixed)
manage.py's runserver autoreloading is broken in Jython
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | Core (Management commands) | Version: | 1.3 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Currently, because Jython files are compiled to have the suffix $py.class, and these are the files referenced in Jython, changes to the original source files aren't picked up by the Django autoreloader.
I have a fix in a pull request on the Github mirror: https://github.com/django/django/pull/28
Attachments (1)
Change History (3)
comment:1 by , 13 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
by , 13 years ago
| Attachment: | 16523.diff added |
|---|
Note:
See TracTickets
for help on using tickets.
We already have a comparable Jython-specific fix here:
https://code.djangoproject.com/browser/django/trunk/django/core/management/__init__.py#L406
I'm attaching the patch from the pull request. It looks good to me; the only reason why I don't mark it as RFC is that I don't have access to a Jython installation to test it.