You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by jv...@apache.org on 2004/03/12 20:46:33 UTC

cvs commit: maven-components/maven-core/src/java/org/apache/maven/plugin/plexus PlexusPluginManager.java

jvanzyl     2004/03/12 11:46:33

  Modified:    maven-core/src/java/org/apache/maven/plugin/plexus
                        PlexusPluginManager.java
  Log:
  o only attempt to configure the plugin if there is a valid configuration
    with content.
  
  Revision  Changes    Path
  1.12      +6 -2      maven-components/maven-core/src/java/org/apache/maven/plugin/plexus/PlexusPluginManager.java
  
  Index: PlexusPluginManager.java
  ===================================================================
  RCS file: /home/cvs/maven-components/maven-core/src/java/org/apache/maven/plugin/plexus/PlexusPluginManager.java,v
  retrieving revision 1.11
  retrieving revision 1.12
  diff -u -r1.11 -r1.12
  --- PlexusPluginManager.java	16 Feb 2004 20:38:45 -0000	1.11
  +++ PlexusPluginManager.java	12 Mar 2004 19:46:33 -0000	1.12
  @@ -32,7 +32,11 @@
       {
           Object plugin = plugins.get( pluginId );
   
  -        plugin = pluginConfigurator.configure( plugin, gd.getConfiguration(), project );
  +        // Only configure the plugin if there is a configuration with content.
  +        if ( gd.getConfiguration().getChildCount() != 0 )
  +        {
  +            plugin = pluginConfigurator.configure( plugin, gd.getConfiguration(), project );
  +        }
   
           Method m = plugin.getClass().getMethod( "execute", new Class[0] );
   
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org