I did some research, and I can hit the logic in question, if app_label == settings.AUTH_USER_MODEL and ignore_swappable: continue
... on both master and with Claude's patch if I follow the steps in this comment: https://code.djangoproject.com/ticket/22563#comment:8 Andrew, first you added an error message to say that changing AUTH_USER_MODEL after creating an initial migration was invalid: https://github.com/django/django/commit/fc974313b85da932a70f1f993b33207d78d31831 and then someone reporting hitting that error with a false positive and you changed it to the current version: https://github.com/django/django/commit/5182efce8d73ec552a1d7f3e86a24369bb261044 Your comment was "I've committed a patch that suppresses the error in that one case (where we know it's safe to do so), and I can now switch AUTH_USER_MODEL even midway through a migration set without errors." Maybe this helps refresh your memory? On Saturday, December 20, 2014 5:53:03 AM UTC-5, Andrew Godwin wrote: > > Clause, I believe that line is to allow people to run makemigrations when > AUTH_USER_MODEL points to an app that doesn't yet have migrations; before, > it would fail hard, as the AUTH_USER_MODEL was not in the migrate state and > so nothing could use it, and you couldn't run makemigrations to create it > as it would error out. > > I'd test variations of that to see if your patch still works; if not, > we'll need a workaround again. > > Andrew > > On Sat, Dec 20, 2014 at 10:47 AM, Claude Paroz <[email protected] > <javascript:>> wrote: >> >> On Friday, December 19, 2014 6:30:32 PM UTC+1, Tim Graham wrote: >>> >>> Yes. Claude has worked on the deprecation in https://github.com/django/ >>> django/pull/3220 but it requires adding more migrations to our test >>> suite and he noted that each migration we add to Django's test suite adds >>> up to ~3.5 seconds to the run time of the test suite. He also worked on >>> some speed ups to mitigate this in https://github.com/django/ >>> django/pull/3484 but there are some unsolved issues. >>> >> >> I think that the first mentioned patch I worked on (PR 3220) shows that >> it should be possible to use the new schema infrastructure without >> requiring migrations, at least for apps that don't depend on other migrated >> apps. Tell me if I miss anything. >> >> But surely, I'd really like to solve first the speed issue of migrations >> (#23745, PR 3484). Chris, if you have the opportunity to test that patch >> with your database, it would be great. The way forward for this patch is to >> first test the "if app_label == settings.AUTH_USER_MODEL and >> ignore_swappable:" line which is currently not tested and is probably not >> addressed by my patch. Andrew, do you remember why you introduced that part >> of the code? >> >> Claude >> >> -- >> You received this message because you are subscribed to the Google Groups >> "Django developers (Contributions to Django itself)" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected] <javascript:>. >> To post to this group, send email to [email protected] >> <javascript:>. >> Visit this group at http://groups.google.com/group/django-developers. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/django-developers/1f5f37b2-5309-4af6-8b3c-aa76eee3bde1%40googlegroups.com >> >> <https://groups.google.com/d/msgid/django-developers/1f5f37b2-5309-4af6-8b3c-aa76eee3bde1%40googlegroups.com?utm_medium=email&utm_source=footer> >> . >> >> For more options, visit https://groups.google.com/d/optout. >> > -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/60e08f9c-4292-45d3-9da8-35a5ab5f5081%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
