You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jd...@apache.org on 2008/04/22 20:05:11 UTC

svn commit: r650601 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java

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 );