[MNG-2184] Commenting this out again, until I can figure out how to make aggregators run once and only once, regardless of the contents of the reactor.

git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@619724 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
John Dennis Casey 2008-02-08 01:20:22 +00:00
parent 80f538bc52
commit 0fc4704f30
1 changed files with 9 additions and 5 deletions

View File

@ -523,11 +523,15 @@ public class DefaultLifecycleExecutor
MojoDescriptor mojoDescriptor = pluginDescriptor.getMojo( mojoBinding.getGoal() ); MojoDescriptor mojoDescriptor = pluginDescriptor.getMojo( mojoBinding.getGoal() );
if ( mojoDescriptor.isAggregator() && ( project != rm.getTopLevelProject() ) ) // TODO: Figure out how to make this logic produce the same result when the binding is in a module.
{ // At times, the module will build in isolation, in which case this logic would allow the aggregator to run.
getLogger().debug( "Skipping mojo execution: " + MojoBindingUtils.toString( mojoBinding ) + "\nfor project: " + project.getId() + "\n\nIt is an aggregator mojo, and the current project is not the root project for the reactor." ); // In other cases, the module will be part of a reactor build, and the aggregator won't run, because it's not
return; // bound to the root project.
} // if ( mojoDescriptor.isAggregator() && ( project != rm.getTopLevelProject() ) )
// {
// getLogger().debug( "Skipping mojo execution: " + MojoBindingUtils.toString( mojoBinding ) + "\nfor project: " + project.getId() + "\n\nIt is an aggregator mojo, and the current project is not the root project for the reactor." );
// return;
// }
validateMojoExecution( mojoBinding, mojoDescriptor, project, allowAggregators ); validateMojoExecution( mojoBinding, mojoDescriptor, project, allowAggregators );