Thread overview
[Issue 5375] New: Detection of cyclic module imports provides error findings on console, instead of exception msg
Dec 25, 2010
Michal Minich
Dec 18, 2012
Andrej Mitrovic
Feb 05, 2013
Andrej Mitrovic
December 25, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=5375

           Summary: Detection of cyclic module imports provides error
                    findings on console, instead of exception msg
           Product: D
           Version: D2
          Platform: Other
        OS/Version: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: druntime
        AssignedTo: sean@invisibleduck.org
        ReportedBy: michal.minich@gmail.com


--- Comment #0 from Michal Minich <michal.minich@gmail.com> 2010-12-25 11:26:18 PST ---
from object_.d:

> // print the cycle
> println("Cycle detected between modules with ctors/dtors:");
> foreach(cm; dtors[dtoridx..$])
> {
>    console(cm.name)(" -> ");
> }
> println(cycleModule.name);
> throw new Exception("Aborting!");

for non-console application, it is hard to find what is happening, except
"Aborting!" (adding AttachConsole(-1) call is needed, moreover this function is
not included in druntime win bindings)

All the cycle printing text should be included in exception message. (I see no problem printing it on console too)

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


Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |schveiguy@yahoo.com
         AssignedTo|sean@invisibleduck.org      |schveiguy@yahoo.com


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 18, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=5375


Andrej Mitrovic <andrej.mitrovich@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull
                 CC|                            |andrej.mitrovich@gmail.com
         AssignedTo|schveiguy@yahoo.com         |andrej.mitrovich@gmail.com


--- Comment #1 from Andrej Mitrovic <andrej.mitrovich@gmail.com> 2012-12-18 15:07:50 PST ---
https://github.com/D-Programming-Language/druntime/pull/365

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 19, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=5375



--- Comment #2 from github-bugzilla@puremagic.com 2012-12-18 19:19:48 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/druntime

https://github.com/D-Programming-Language/druntime/commit/84d60f5e599b9aca80afb782c24f5a9de568278c Fix Issue 5375 - Cyclic error should be embedded in the exception.

https://github.com/D-Programming-Language/druntime/commit/ffd252c07e7dc1f8adc5384cc1d1e3351d063ead Merge pull request #365 from AndrejMitrovic/Fix5375

Issue 5375 - Cyclic error message should be embedded in the Exception.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
February 05, 2013
http://d.puremagic.com/issues/show_bug.cgi?id=5375


Andrej Mitrovic <andrej.mitrovich@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


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