revi-archive/llvm-premerge-checks
1
0
Fork 0
Code Activity

Don't fail buildif ninja all / check all has failed

Browse source
This commit is contained in:
Mikhail Goncharov 2019-12-18 08:57:29 +01:00
parent a6bfda48f8
commit 89e7c8ad24
1 changed files with 24 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
Jenkins/Phabricator-pipeline/Jenkinsfile vendored
View file

@ -33,6 +33,7 @@ pipeline {
steps {
echo "Building diff ${DIFF_ID} with PHID ${PHID}"
script {
def success = true
def revision_phid = sh(script:
'echo \'{"constraints":{"ids":[' + DIFF_ID + ']}}\' | '+
'arc call-conduit --conduit-uri=$PHABRICATOR_HOST --conduit-token $CONDUIT_TOKEN differential.diff.search | ' +
@ -76,12 +77,26 @@ pipeline {
}
stage('ninja all') {
steps {
sh "${SCRIPT_DIR}/run_ninja.sh all"
script {
try {
sh(script: "${SCRIPT_DIR}/run_ninja.sh all")
} catch (e) {
success = false;
echo e.toString()
}
}
}
}
stage('ninja check-all') {
steps {
sh "${SCRIPT_DIR}/run_ninja.sh check-all"
script {
try {
sh(script: "${SCRIPT_DIR}/run_ninja.sh check-all")
} catch (e) {
success = false;
echo e.toString()
}
}
}
}
stage('linters') {
@ -92,7 +107,13 @@ pipeline {
}
post {
always {
script {
if (success) {
currentBuild.result = "SUCCESS"
} else {
currentBuild.result = "FAILURE"
}
}
echo "Console log is available at ${RESULT_URL}"
dir("${TARGET_DIR}") {
// copy console log to result folder