Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#6038 closed (fixed)

syncdb: reraise ImportError in

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


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

If does not exist, it ignores the ImportError.

Attachments (1)

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

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by Thomas Güttler <hv@…>

comment:1 Changed 9 years ago by mcroydon

  • Component changed from Uncategorized to
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 9 years ago by Simon G <dev@…>

  • Triage Stage changed from Unreviewed to Ready for checkin

comment:3 Changed 9 years ago by mtredinnick

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 Changed 9 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from new to closed

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

comment:5 Changed 9 years ago by Thomas Güttler <hv@…>

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