Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#18827 closed Bug (fixed)

Exclude __pycache__ from module detection

Reported by: Aymeric Augustin Owned by: nobody
Component: Python 3 Version: dev
Severity: Normal Keywords:
Cc: bhuztez@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

If you make a typo in the database backend you get this error message on Python 3:

Try using django.db.backends.XXX, where XXX is one of:
    '__pycache__', 'dummy', 'mysql', 'oracle', 'postgresql_psycopg2', 'sqlite3'

At least __pycache__ should be filtered out. Maybe dummy too.

It would be nice to check if this pattern occurs in other places in Django.

(One more reason not to attempt listing modules in a package...)

Attachments (1)

18827-1.diff​ (900 bytes ) - added by Claude Paroz 13 years ago.
Use pkgutil to list packages

Download all attachments as: .zip

Change History (5)

by Claude Paroz, 13 years ago

Attachment: 18827-1.diff​ added

Use pkgutil to list packages

comment:1 by Claude Paroz, 13 years ago

Has patch: set
Triage Stage: Unreviewed → Accepted

Related tickets: #8280, #15084

comment:2 by bhuztez, 13 years ago

Cc: bhuztez@… added

also #18603

comment:3 by Ramiro Morales, 13 years ago

Resolution: → fixed
Status: new → closed

Fixed in 2100da9dcdba7a8bc18f6180fd7e61640319b48a.

comment:4 by Claude Paroz <claude@…>, 13 years ago

In 34a736b7521def321b2104d541e634134c5d1c62:

Used pkgutil to get list of backend modules

Refs #18827.

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