Skip to content

Commit

Permalink
HV-1882 Always run JQAssistant in the default build only in the Jenki…
Browse files Browse the repository at this point in the history
…nsfile
  • Loading branch information
yrodiere authored and gsmet committed Feb 24, 2022
1 parent c5b5308 commit 24f9905
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,6 @@ import org.hibernate.jenkins.pipeline.helpers.alternative.AlternativeMultiMap

@Field boolean enableDefaultBuild = false
@Field boolean enableDefaultBuildIT = false
@Field boolean enableDefaultBuildJQAssistant = false
@Field boolean enableDefaultBuildSigtest = false
@Field boolean deploySnapshot = false

Expand All @@ -121,7 +120,7 @@ stage('Configure') {
condition: TestCondition.BEFORE_MERGE,
isDefault: true),
new JdkBuildEnvironment(version: '8', buildJdkTool: 'OracleJDK8 Latest',
enableSigtest: true, enableJQAssistant: true,
enableSigtest: true,
condition: TestCondition.BEFORE_MERGE),
new JdkBuildEnvironment(version: '11', buildJdkTool: 'OpenJDK 11 Latest',
condition: TestCondition.BEFORE_MERGE)
Expand Down Expand Up @@ -250,8 +249,8 @@ stage('Default build') {
install \
"} \
-Pdist \
-Pjqassistant \
${enableDefaultBuildSigtest ? '-Psigtest' : ''} \
${enableDefaultBuildJQAssistant ? '-Pjqassistant' : ''} \
${enableDefaultBuildIT ? '' : '-DskipITs'} \
${toTestJdkArg(environments.content.jdk.default)} \
"""
Expand All @@ -274,7 +273,6 @@ stage('Non-default environments') {
mavenNonDefaultBuild buildEnv, """ \
clean install \
${buildEnv.enableSigtest ? '-Psigtest' : ''} \
${buildEnv.enableJQAssistant ? '-Pjqassistant' : ''} \
"""
}
}
Expand Down Expand Up @@ -343,7 +341,6 @@ class JdkBuildEnvironment extends BuildEnvironment {
String buildJdkTool
String testJdkTool
boolean enableSigtest
boolean enableJQAssistant
@Override
String getTag() { "jdk-$version" }
@Override
Expand Down

0 comments on commit 24f9905

Please sign in to comment.