Circular dependencies are not resolved by new Migrate framework

[Aidan Lister]

It's very easy and not explicitly not-supported to create circular dependencies in your apps. I have attached a test project which has two circular dependency resolution failures.

This fails to create migrations:

./manage.py makemigrations contractors inspections organisations properties reports tasks timeline

This creates the migrations:

./manage.py makemigrations properties reports tasks timeline
./manage.py makemigrations contractors inspections

But then ./manage.py migrate fails:

django.db.migrations.graph.CircularDependencyError: [('contractors', u'0002_auto_20140821_0156'), ('reports', u'0001_initial'), ('inspections', u'0001_initial'), ('contractors', u'0002_auto_20140821_0156')]

I have tried manually breaking up the migrations into lots of little steps, but you'll have the same problem with the reports and inspections app. Even with lots of tiny migrations it eventually loops around.

[Andrew Godwin]

Tried to reproduce from example project, and it worked perfectly with the line:

./manage.py makemigrations tasks timeline organisations reports contractors inspections properties

The generated migrations then also ran successfully.

Closing as INVALID as I suspect this was encountered just before the fixes to #23322 and #23315 landed in master earlier, and in particular #23315 would have manifested as the first command not being able to run.

If you can reproduce this on the most recent commit in master, please reopen and this time provide full console output from when the command runs.