Opened 8 years ago

Closed 8 years ago

#8157 closed (fixed)

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

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

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 8 years ago.
diff against r8226

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by Jeff Balogh

Attachment: pyo.diff added

diff against r8226

comment:1 Changed 8 years ago by Malcolm Tredinnick

Triage Stage: UnreviewedReady for checkin

comment:2 Changed 8 years ago by Eric Holscher

Owner: changed from nobody to Eric Holscher
Status: newassigned

comment:3 Changed 8 years ago by Malcolm Tredinnick

Resolution: fixed
Status: assignedclosed

(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