Thread overview
[Issue 5053] New: Better error message for cyclic dependencies.
Oct 14, 2010
Jacob Carlborg
Oct 14, 2010
Jacob Carlborg
October 14, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=5053

           Summary: Better error message for cyclic dependencies.
           Product: D
           Version: D2
          Platform: Other
        OS/Version: Mac OS X
            Status: NEW
          Severity: normal
          Priority: P2
         Component: druntime
        AssignedTo: sean@invisibleduck.org
        ReportedBy: doob@me.com


--- Comment #0 from Jacob Carlborg <doob@me.com> 2010-10-14 07:59:35 PDT ---
I created a patch which gives a better error message when a cyclic dependency occurs between two modules due to both having module constructors and importing each other.

When two modules "A" and "B" importing each other and having module constructors the error message with the patch will look like this: "Cyclic dependency in module B for import A" instead of "Cyclic dependency in module A".

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 14, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=5053



--- Comment #1 from Jacob Carlborg <doob@me.com> 2010-10-14 08:01:20 PDT ---
Created an attachment (id=784)
Patch for giving better cyclic dependency error messages

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
November 08, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=5053


Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |schveiguy@yahoo.com
         Resolution|                            |FIXED


--- Comment #2 from Steven Schveighoffer <schveiguy@yahoo.com> 2010-11-08 06:36:51 PST ---
This bug is obsolete in light of the fixes I just checked in.  See bug 4384.

The runtime will now print the full cycle when a cyclic dependency is detected.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------