Opened 17 years ago

Closed 17 years ago

#8157 closed (fixed)

utils/autoreload.py doesn't find .pyo files

Reported by: Jeff Balogh Owned by: Eric Holscher
Component: Uncategorized Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

It's using filename.endswith("*.pyo") in the file finder, but endswith doesn't accept globs.

This could be tested by pulling the endswith logic into a separate function, but I don't think it's worth it.

Attachments (1)

pyo.diff​ (699 bytes ) - added by Jeff Balogh 17 years ago.
diff against r8226

Download all attachments as: .zip

Change History (4)

by Jeff Balogh, 17 years ago

Attachment: pyo.diff​ added

diff against r8226

comment:1 by Malcolm Tredinnick, 17 years ago

Triage Stage: Unreviewed → Ready for checkin

comment:2 by Eric Holscher, 17 years ago

Owner: changed from nobody to Eric Holscher
Status: new → assigned

comment:3 by Malcolm Tredinnick, 17 years ago

Resolution: → fixed
Status: assigned → closed

(In [8235]) Made auto-reloading for the dev server a little more friendly in the Jython
case. Patch from Leo Soto. Fixed #8147.

In passing, also corrected a typo when reloading .pyo files. Fixed #8157.

Note: See TracTickets for help on using tickets.
Back to Top