CI: Add "set -e" to Azure DevOps scripts

Without setting this explicitly in every script, failing script lines in
multiline scripts will not fail the job themselves.
This commit is contained in:
Andrew Kaster 2023-03-19 13:17:36 -06:00 committed by Andrew Kaster
parent 958f9e960d
commit 5b5a76eadd
Notes: sideshowbarker 2024-07-17 07:48:42 +09:00
3 changed files with 14 additions and 0 deletions

View file

@ -64,6 +64,7 @@ jobs:
displayName: "Install NDK $(ndk_version)"
- script: |
set -e
cmake -GNinja -B tools-build \
-DBUILD_LAGOM=OFF \
-DENABLE_LAGOM_CCACHE=ON \
@ -89,6 +90,7 @@ jobs:
- ${{ elseif eq(parameters.fuzzer, 'Fuzz') }}:
- script: |
set -e
cmake -GNinja -B tools-build \
-DBUILD_LAGOM=OFF \
-DENABLE_LAGOM_CCACHE=ON \
@ -109,6 +111,7 @@ jobs:
CCACHE_DIR: '$(SERENITY_CCACHE_DIR)'
- ${{ elseif eq(parameters.fuzzer, 'NoFuzz') }}:
- script: |
set -e
cmake -GNinja -B Build \
-DBUILD_LAGOM=ON \
-DENABLE_LAGOM_CCACHE=ON \
@ -126,6 +129,7 @@ jobs:
CCACHE_DIR: '$(SERENITY_CCACHE_DIR)'
- script: |
set -e
cmake --build .
cmake --install . --prefix $(Build.SourcesDirectory)/Meta/Lagom/Install
displayName: 'Build'
@ -135,6 +139,7 @@ jobs:
- ${{ if and(eq(parameters.fuzzer, 'NoFuzz'), not(eq(parameters.os, 'Android')) ) }}:
- script: |
set -e
ninja test
displayName: 'Test'
workingDirectory: $(Build.SourcesDirectory)/Meta/Lagom/Build

View file

@ -42,6 +42,7 @@ jobs:
displayName: 'Create Build Directory'
- script: |
set -e
cmake -S Meta/CMake/Superbuild -B Build/superbuild -GNinja \
-DSERENITY_ARCH=${{ parameters.arch }} \
-DSERENITY_TOOLCHAIN=Clang \
@ -58,6 +59,7 @@ jobs:
CCACHE_DIR: '$(SERENITY_CCACHE_DIR)'
- script: |
set -e
cmake --build ./Build/superbuild
displayName: 'Build'
workingDirectory: $(Build.SourcesDirectory)
@ -65,11 +67,13 @@ jobs:
CCACHE_DIR: '$(SERENITY_CCACHE_DIR)'
- script: |
set -e
ninja install && ninja qemu-image
displayName: 'Create RootFS'
workingDirectory: $(Build.SourcesDirectory)/Build/${{ parameters.arch }}clang
- script: |
set -e
ninja run
echo "##[group]Verify Output File"
@ -102,6 +106,7 @@ jobs:
- ${{ if eq(parameters.coverage, 'ON') }}:
- script: |
set -e
./Meta/analyze-qemu-coverage.sh
displayName: 'Aggregate Coverage Results'
workingDirectory: $(Build.SourcesDirectory)

View file

@ -7,6 +7,7 @@ steps:
- ${{ if eq(parameters.os, 'Serenity') }}:
- script: |
set -e
wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
sudo add-apt-repository 'deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy-15 main'
sudo apt-get update
@ -15,6 +16,7 @@ steps:
- ${{ if eq(parameters.os, 'Linux') }}:
- script: |
set -e
sudo apt-get purge -y clang-12 gcc-10
wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
sudo add-apt-repository 'deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy-13 main'
@ -34,12 +36,14 @@ steps:
- ${{ if eq(parameters.os, 'macOS') }}:
# macOS ships an ancient Bash 3.x by default
- script: |
set -e
brew update
brew install bash ninja wabt ccache unzip qt llvm@15
displayName: 'Install Dependencies'
- ${{ if eq(parameters.os, 'Android') }}:
- script: |
set -e
sudo apt-get install ccache gcc-12 g++-12 libstdc++-12-dev ninja-build unzip
sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-12 100
sudo update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-12 100