#14130: Catching ImportError in manage.py considered dangerous ------------------------------------+--------------------------------------- Reporter: Setok | Owner: nobody Status: new | Milestone: Component: Core framework | Version: 1.1 Keywords: manage settings import | Stage: Unreviewed Has_patch: 0 | ------------------------------------+--------------------------------------- The basic implementation of manage.py does little else than check for an ImportError on settings. Unfortunately using try-except to check for module existence in Python is not a very good practise. The problem is that it hides any possible import errors that may occur within settings.py. Sure, the error displayed by manage.py does mention that possibility, but only in parenthesis, after bold text about settings.py missing.
Having just spend several hours trying to figure out what was going on I believe that, in balance, it would be better just to let the Python error fall through and for a proper stacktrace to occur. Alternatively try to figure out a way to detect if it was, in fact, the settings module which was missing, or something else. Annoyingly Python's ImportError does not seem to easily have that information available :( -- Ticket URL: <http://code.djangoproject.com/ticket/14130> Django <http://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To post to this group, send email to django-upda...@googlegroups.com. To unsubscribe from this group, send email to django-updates+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/django-updates?hl=en.
