Improve Circular Dependency Error Message

[gissuebot]

From sberlin on August 06, 2007 17:56:32

Right now, a circular dependency produces the following error msg:
com.google.inject.ConfigurationException: Tried proxying Circle to support
a circular dependency, but it is not an interface.

It would be very useful if this message was improved to show who the
players in the circular dependency are.  That is, if class Circle depends
on SquareHole and SquareHole depends on Circle, the message could say
that's what it's failing.  This would make fixing the problem much easier.

Here's a fuller stack trace, for reference:

com.google.inject.ConfigurationException: Tried proxying Circle to support
a circular dependency, but it is not an interface.
        at com.google.inject.ConstructionContext.createProxy
(ConstructionContext.java:70)
        at com.google.inject.ConstructorInjector.construct
(ConstructorInjector.java:126)
        at [... lots of internal guice calls ...]
        at com.google.inject.InjectorImpl.getInstance
(InjectorImpl.java:728)
        at [... my code ...]
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke
(NativeMethodAccessorImpl.java:39)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke
(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:597)
        at com.limegroup.gnutella.gui.Main.main(Main.java:47)

Original issue: http://code.google.com/p/google-guice/issues/detail?id=139

[gissuebot]

From limpbizkit on November 02, 2008 01:46:48

Will be addressed by issue 220 ...

Status: Duplicate