Author: jdcasey
Date: Tue Apr 22 11:05:10 2008
New Revision: 650601

URL: http://svn.apache.org/viewvc?rev=650601&view=rev
Log:
Cleaning up the error message formatting some more.

Modified:
    
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java

Modified: 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java?rev=650601&r1=650600&r2=650601&view=diff
==============================================================================
--- 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
 (original)
+++ 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
 Tue Apr 22 11:05:10 2008
@@ -608,34 +608,34 @@
         {
             if ( getLogger().isFatalErrorEnabled() )
             {
-                getLogger().fatalError(
-                                        mojoDescriptor.getImplementation() + 
"#execute() caused a linkage error ("
-                                            + e.getClass().getName() + ") and 
may be out-of-date. Check the realms:" );
-
                 StringBuffer sb = new StringBuffer();
-                sb.append( "Plugin realm = " + pluginRealm.getId() ).append( 
'\n' );
-                for ( int i = 0; i < pluginRealm.getURLs().length; i++ )
-                {
-                    sb.append( "urls[" + i + "] = " + pluginRealm.getURLs()[i] 
);
-                    if ( i != ( pluginRealm.getURLs().length - 1 ) )
-                    {
-                        sb.append( '\n' );
-                    }
-                }
-                getLogger().fatalError( sb.toString() );
+                sb.append( mojoDescriptor.getImplementation() ).append( 
"#execute() caused a linkage error (" );
+                sb.append( e.getClass().getName() ).append( "). Check the 
realms:" );
+
+                ClassRealm r = pluginDescriptor.getClassRealm();
+                sb.append( "\n\nNOTE:\nPlugin realm is: " ).append( r.getId() 
);
+                sb.append( "\nContainer realm is: " ).append( 
container.getContainerRealm().getId() );
+                sb.append( "\n\n" );
 
-                ClassRealm containerRealm = container.getContainerRealm();
-                sb = new StringBuffer();
-                sb.append( "Container realm = " + containerRealm.getId() 
).append( '\n' );
-                for ( int i = 0; i < containerRealm.getURLs().length; i++ )
+                do
                 {
-                    sb.append( "urls[" + i + "] = " + 
containerRealm.getURLs()[i] );
-                    if ( i != ( containerRealm.getURLs().length - 1 ) )
+                    sb.append( "Realm ID: " ).append( r.getId() ).append( '\n' 
);
+                    for ( int i = 0; i < r.getURLs().length; i++ )
                     {
-                        sb.append( '\n' );
+                        sb.append( "urls[" ).append( i ).append( "] = " 
).append( r.getURLs()[i] );
+                        if ( i != ( r.getURLs().length - 1 ) )
+                        {
+                            sb.append( '\n' );
+                        }
                     }
+
+                    sb.append( "\n\n" );
+                    r = r.getParentRealm();
                 }
-                getLogger().fatalError( sb.toString() );
+                while( r != null );
+
+                getLogger().fatalError(
+                                       sb.toString(), e );
             }
 
             session.getEventDispatcher().dispatchError( event, goalExecId, e );


Reply via email to