[maven-jenkins-lib] branch master updated: Aha\! first was being evaluated after adding the task

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[maven-jenkins-lib] branch master updated: Aha\! first was being evaluated after adding the task

stephenc
This is an automated email from the ASF dual-hosted git repository.

stephenc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-jenkins-lib.git


The following commit(s) were added to refs/heads/master by this push:
     new 59e19e3  Aha\! first was being evaluated after adding the task
59e19e3 is described below

commit 59e19e327d0c37092a325f8707c512fda6b335a9
Author: Stephen Connolly <[hidden email]>
AuthorDate: Thu Nov 30 12:29:56 2017 +0000

    Aha\! first was being evaluated after adding the task
---
 vars/asfMavenTlpStdBuild.groovy | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/vars/asfMavenTlpStdBuild.groovy b/vars/asfMavenTlpStdBuild.groovy
index 09f42e7..568bf34 100644
--- a/vars/asfMavenTlpStdBuild.groovy
+++ b/vars/asfMavenTlpStdBuild.groovy
@@ -47,6 +47,17 @@ def call(Map params = [:]) {
           echo "Skipping ${os}-jdk${jdk} as unsupported by Jenkins Environment"
           continue;
         }
+        def cmd = [
+          'mvn',
+          '-P+run-its',
+          '-Dmaven.test.failure.ignore=true',
+          '-Dfindbugs.failOnError=false',
+        ]
+        if (!first) {
+          cmd += '-Dfindbugs.skip=true'
+        }
+        cmd += 'clean'
+        cmd += 'verify'
         String stageId = "${os}-jdk${jdk}"
         tasks[stageId] = {
           node(label) {
@@ -56,17 +67,6 @@ def call(Map params = [:]) {
               }
             }
             stage("Build ${stageId}") {
-              def cmd = [
-                'mvn',
-                '-P+run-its',
-                '-Dmaven.test.failure.ignore=true',
-                '-Dfindbugs.failOnError=false',
-              ]
-              if (!first) {
-                cmd += '-Dfindbugs.skip=true'
-              }
-              cmd += 'clean'
-              cmd += 'verify'
               withMaven(jdk:jdkName, maven:mvnName, mavenLocalRepo:'.repository', options: [
                 artifactsPublisher(disabled: !first),
                 junitPublisher(ignoreAttachments: false),

--
To stop receiving notification emails like this one, please contact
['"[hidden email]" <[hidden email]>'].