[FIXED JENKINS-20487] call postBuild after module build completion

fingerprints are managed by MavenFingerprinter
jenkinsci · Dec 31, 2013 · b28d95f · b28d95f
1 parent 1e7a141
commit b28d95f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/src/main/java/hudson/maven/Maven3Builder.java b/src/main/java/hudson/maven/Maven3Builder.java
@@ -553,7 +553,7 @@ private void recordProjectEnded(ExecutionEvent event, Result result) {
 
             for ( MavenReporter mavenReporter : fixNull(mavenReporters)) {
                 try {
-                    mavenReporter.leaveModule( mavenBuildProxy2, event.getProject(), maven3Builder.listener);
+                    mavenReporter.postBuild( mavenBuildProxy2, event.getProject(), maven3Builder.listener);
                 } catch ( InterruptedException e ) {
                     e.printStackTrace();
                 } catch ( IOException e ) {

diff --git a/src/main/java/hudson/maven/reporters/MavenArtifactArchiver.java b/src/main/java/hudson/maven/reporters/MavenArtifactArchiver.java
@@ -125,7 +125,7 @@ public boolean postBuild(MavenBuildProxy build, MavenProject pom, final BuildLis
             }
 
             // record the action
-            build.executeAsync(new MavenBuildProxy.BuildCallable<Void, IOException>() {
+            build.execute(new MavenBuildProxy.BuildCallable<Void, IOException>() {
                 private static final long serialVersionUID = -7955474564875700905L;
 
                 public Void call(MavenBuild build) throws IOException, InterruptedException {
@@ -139,10 +139,6 @@ public Void call(MavenBuild build) throws IOException, InterruptedException {
                             repositoryId);
                     build.addAction(mar);
 
-                    // TODO kutzi: why are the fingerprints recorded here?
-                    // I thought that is the job of MavenFingerprinter
-                    mar.recordFingerprints();
-
                     return null;
                 }
             });