Skip to content

Commit

Permalink
CI: Remove extraneous toolchain job from Azure CI
Browse files Browse the repository at this point in the history
This was useful when building both i686 and x86_64 SerenityOS targets as
we could use a single toolchain build for both targets. But now all this
extra job does is create the opportunity for the toolchain to need to be
built twice (i.e. if the pipelines are backed up and the toolchain cache
is busted between these jobs while the x86_64 step is waiting for a VM).
  • Loading branch information
trflynn89 committed Dec 28, 2022
1 parent 598fcfc commit 0a7d036
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 42 deletions.
2 changes: 1 addition & 1 deletion Meta/Azure/Serenity.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
serenity_ccache_path: '$(SERENITY_CCACHE_DIR)'

- script: ./Toolchain/BuildClang.sh --ci
displayName: Extract Toolchain
displayName: Build Toolchain
env:
TRY_USE_LOCAL_TOOLCHAIN: 'y'

Expand Down
34 changes: 0 additions & 34 deletions Meta/Azure/Toolchain.yml

This file was deleted.

8 changes: 1 addition & 7 deletions azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,14 +23,8 @@ stages:
parameters:
os: 'macOS'

- stage: Toolchain
dependsOn: []

jobs:
- template: Meta/Azure/Toolchain.yml

- stage: SerenityOS
dependsOn: Toolchain
dependsOn: []

jobs:
- template: Meta/Azure/Serenity.yml
Expand Down

0 comments on commit 0a7d036

Please sign in to comment.