fixed a bug in dependency handling with aggregator m2 project.

git-svn-id: https://hudson.dev.java.net/svn/hudson/trunk/hudson/main@4554 71c3de6d-444a-0410-be80-ed276b4c234a

core/src/main/java/hudson/maven/MavenModule.java

@@ -299,12 +299,18 @@ public final class MavenModule extends AbstractMavenProject<MavenModule,MavenBui
             modules.put(m.getModuleName(),m);
         }
 
+        // if the build style is the aggregator build, define dependencies against project,
+        // not module.
         AbstractProject dest = getParent().isAggregatorStyleBuild() ? getParent() : this;
 
         for (ModuleName d : dependencies) {
             MavenModule src = modules.get(d);
-            if(src!=null)
-                graph.addDependency(src,dest);
+            if(src!=null) {
+                if(src.getParent().isAggregatorStyleBuild())
+                    graph.addDependency(src.getParent(),dest);
+                else
+                    graph.addDependency(src,dest);
+            }
         }
     }
 

core/src/main/java/hudson/model/DependencyGraph.java

@@ -187,7 +187,8 @@ public final class DependencyGraph {
             set = new ArrayList<AbstractProject>();
             map.put(src,set);
         }
-        set.add(dst);
+        if(!set.contains(dst))
+            set.add(dst);
     }
 
     private Map<AbstractProject, List<AbstractProject>> finalize(Map<AbstractProject, List<AbstractProject>> m) {