Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#6038 closed (fixed)

syncdb: reraise ImportError in app.management.py

Reported by: Thomas Güttler <hv@…> Owned by: nobody
Component: Core (Management commands) 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

Up to now ImportErrors in app.management.py are ignored. This
small patch reraises the ImportError, if it was caused by an existing management.py.

If app.management.py does not exist, it ignores the ImportError.

Attachments (1)

syncdb_reraise_import_error.diff (621 bytes ) - added by Thomas Güttler <hv@…> 16 years ago.

Download all attachments as: .zip

Change History (6)

by Thomas Güttler <hv@…>, 16 years ago

comment:1 by mcroydon, 16 years ago

Component: Uncategorizeddjango-admin.py

comment:2 by Simon G <dev@…>, 16 years ago

Triage Stage: UnreviewedReady for checkin

comment:3 by Malcolm Tredinnick, 16 years ago

This has the standard problem of being very fragile because it relies on the human-readable string for computer processing. I'm going to check it in, since it adds utility, but if it causes problems on, say, Jython, by causing spurious exceptions to be raised, we should back it out immediately.

comment:4 by Malcolm Tredinnick, 16 years ago

Resolution: fixed
Status: newclosed

(In [6829]) Fixed #6038 -- Don't hide some types of errors in syncdb. Thanks, Thomas Güttler.

comment:5 by Thomas Güttler <hv@…>, 16 years ago

Cc: hv@… removed
Note: See TracTickets for help on using tickets.
Back to Top