[Django] #25627: Django is swallowing ImportErrors in migrations for modules containing the string 'south'
Django
string 'south'
-------------------------------+--------------------
Reporter: gavinwahl | Owner: nobody
Type: Uncategorized | Status: new
Component: Migrations | Version: 1.9a1
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------+--------------------
If I put `from mymodule import north, east, west, south,
xxxdoesntexistxxx` in an initial migration, Django tells me:
{{{
django.db.migrations.exceptions.BadMigrationError: Migrated app 'foo'
contains South migrations. Make sure all numbered South migrations are
deleted prior to creating Django migrations.
}}}
I should get an ImportError so I can debug the problem.
--
Ticket URL: <https://code.djangoproject.com/ticket/25627>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
string 'south'
Reporter: gavinwahl | Owner: nobody
Type: Uncategorized | Status: new
Component: Migrations | Version: 1.9a1
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by timgraham):
* needs_better_patch: => 0
* needs_tests: => 0
* needs_docs: => 0
Comment:
That error message was added in #25618 but I guess reverting it won't fix
the issue either. I think this issue affects 1.8 and 1.7 too (silently
putting apps with such errors in `unmigrated_apps` before #25618), but
perhaps we could backport c4af8eb366be45420e13a400cc1dcc8fab91c1ad to 1.9.
I expect South is probably obsoleted by Django 1.9 too but I didn't want
to backport a non-release blocker at this point without a better
justification.
--
Ticket URL: <https://code.djangoproject.com/ticket/25627#comment:1>
Django
string 'south'
Reporter: gavinwahl | Owner: nobody
Type: Uncategorized | Status: closed
Component: Migrations | Version: 1.9a1
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Tim Graham <timograham@…>):
* status: new => closed
* resolution: => fixed
Comment:
In [changeset:"9e7d0d90cc35800cd309d4e0296523220864e884" 9e7d0d9]:
{{{
#!CommitTicketReference repository=""
revision="9e7d0d90cc35800cd309d4e0296523220864e884"
[1.9.x] Fixed #25627, refs #25618 -- Removed detection of south migrations
in loader.
Backport of c4af8eb366be45420e13a400cc1dcc8fab91c1ad from master
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25627#comment:2>