Modify ↓

Ticket #197 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Recursive dependancies go cause inifinte recursion

Reported by: mike@… Owned by: andrew
Priority: major Milestone: 0.6
Component: commands Version: 0.6-pre
Keywords: Cc:

Description

I have an inherited object model and have the same problem as #179. I download the current "stabilish" (Changeset 287:edc9ee8e81f9) and now it tries to recurse forever when doing an initial migration:

[....]
  File "/Users/michaelhostetler/dev/5Q/env/django10/lib/python2.4/site-packages/South-0.6_pre-py2.4.egg/south/management/commands/startmigration.py", line 676, in field_dependencies
    depends.update(field_dependencies(field.rel.to._meta.pk, last_models))
  File "/Users/michaelhostetler/dev/5Q/env/django10/lib/python2.4/site-packages/South-0.6_pre-py2.4.egg/south/management/commands/startmigration.py", line 676, in field_dependencies
    depends.update(field_dependencies(field.rel.to._meta.pk, last_models))
RuntimeError: maximum recursion depth exceeded

Attachments

recurisve-patch.diff Download (1.3 KB) - added by mike@… 3 years ago.
fix recursive fields

Change History

comment:1 Changed 3 years ago by andrew

  • Status changed from new to closed
  • Version set to subversion
  • Resolution set to fixed
  • Milestone set to 0.6

This should be fixed in [0aa5b5fb6e74] (tagged as new 'stableish').

Changed 3 years ago by mike@…

fix recursive fields

comment:2 Changed 3 years ago by mike@…

attached is a patch that fixes my recursion issue.

comment:3 Changed 3 years ago by anonymous

OK, that was field deps too. A similar patch (without the problems with set() as a default) added in [b40737ec465d].

View ↑

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.