Skip to content

Commit 3353a7e

Browse files
committed
[presets] Add clang install to bots that test indexstore-db/sourcekit-lsp
At a minimum, we need to install libIndexStore to enable testing the indexer, but it's better to also include clang/clangd, since this lets us enable all of our tests on these bots. This commit uses the swift-install-components and llvm-install-components from the Package bots, which increases the similarity between our PR tests and package bots. In addition to the sourcekit-lsp/indexstore-db PR test bots, this affects the following bots that test indexstore-db and didn't already install clang: * Linux * incremental * incremental long test * swiftpm PR test * macOS * PR test * PR smoke test * incremental * swiftpm PR test Based on looking at recent build logs for the package bots, this should only add <5 seconds to the incremental build.
1 parent d6c2b9e commit 3353a7e

File tree

1 file changed

+39
-13
lines changed

1 file changed

+39
-13
lines changed

utils/build-presets.ini

Lines changed: 39 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,9 @@ dash-dash
1818

1919
swift-install-components=compiler;clang-builtin-headers;stdlib;sdk-overlay;parser-lib;editor-integration;tools;toolchain-tools;testsuite-tools;sourcekit-xpc-service;swift-remote-mirror;swift-remote-mirror-headers;
2020

21+
[preset: mixin_buildbot_install_components_with_clang]
22+
swift-install-components=compiler;clang-resource-dir-symlink;stdlib;sdk-overlay;parser-lib;toolchain-tools;license;sourcekit-xpc-service;swift-remote-mirror;swift-remote-mirror-headers
23+
llvm-install-components=llvm-cov;llvm-profdata;IndexStore;clang;clang-headers;compiler-rt;clangd
2124

2225
[preset: mixin_buildbot_trunk_base]
2326
# Build standard library and SDK overlay for iOS device and simulator.
@@ -322,7 +325,9 @@ tvos
322325
watchos
323326

324327
[preset: buildbot_incremental,tools=RA,stdlib=RA]
325-
mixin-preset=buildbot_incremental_base_all_platforms
328+
mixin-preset=
329+
buildbot_incremental_base_all_platforms
330+
mixin_buildbot_install_components_with_clang
326331

327332
build-subdir=buildbot_incremental
328333

@@ -351,6 +356,7 @@ skip-test-watchos
351356
install-swift
352357
install-llbuild
353358
install-swiftpm
359+
install-libcxx
354360

355361
[preset: buildbot_incremental,tools=RA,stdlib=RA,xcode]
356362
mixin-preset=buildbot_incremental,tools=RA,stdlib=RA
@@ -518,6 +524,7 @@ swift-stdlib-build-type=RelWithDebInfo
518524
mixin-preset=
519525
buildbot_incremental_base
520526
lldb-smoketest,tools=RA
527+
mixin_buildbot_install_components_with_clang
521528
build-subdir=buildbot_incremental
522529

523530
# We build release+asserts.
@@ -551,6 +558,7 @@ sourcekit-lsp
551558
install-swift
552559
install-llbuild
553560
install-swiftpm
561+
install-libcxx
554562

555563
# We need to build the unittest extras so we can test
556564
build-swift-stdlib-unittest-extra
@@ -717,8 +725,14 @@ swift-enable-ast-verifier=0
717725
#===------------------------------------------------------------------------===#
718726
# Linux Builders
719727
#===------------------------------------------------------------------------===#
728+
[preset: mixin_linux_install_components_with_clang]
729+
swift-install-components=autolink-driver;compiler;clang-resource-dir-symlink;stdlib;swift-remote-mirror;sdk-overlay;parser-lib;toolchain-tools;license;sourcekit-inproc
730+
llvm-install-components=llvm-cov;llvm-profdata;IndexStore;clang;clang-headers;compiler-rt;clangd
731+
720732
[preset: mixin_linux_installation]
721-
mixin-preset=mixin_lightweight_assertions
733+
mixin-preset=
734+
mixin_lightweight_assertions
735+
mixin_linux_install_components_with_clang
722736

723737
llbuild
724738
swiftpm
@@ -736,8 +750,6 @@ install-swiftpm
736750
install-xctest
737751
install-libicu
738752
install-prefix=/usr
739-
swift-install-components=autolink-driver;compiler;clang-resource-dir-symlink;stdlib;swift-remote-mirror;sdk-overlay;parser-lib;toolchain-tools;license;sourcekit-inproc
740-
llvm-install-components=llvm-cov;llvm-profdata;IndexStore;clang;clang-headers;compiler-rt;clangd
741753
install-libcxx
742754
build-swift-static-stdlib
743755
build-swift-static-sdk-overlay
@@ -968,7 +980,9 @@ build-ninja
968980
reconfigure
969981

970982
[preset: buildbot_incremental_linux]
971-
mixin-preset=buildbot_incremental_linux_base
983+
mixin-preset=
984+
buildbot_incremental_linux_base
985+
mixin_linux_install_components_with_clang
972986
build-subdir=buildbot_incremental
973987

974988
libicu
@@ -988,6 +1002,7 @@ install-swiftpm
9881002
install-foundation
9891003
install-libdispatch
9901004
install-xctest
1005+
install-libcxx
9911006

9921007
[preset: buildbot_incremental_linux,long_test]
9931008
mixin-preset=buildbot_incremental_linux
@@ -1064,6 +1079,7 @@ sourcekit-lsp=0
10641079
# OS X Package Builders
10651080
#===------------------------------------------------------------------------===#
10661081
[preset: mixin_osx_package_base]
1082+
mixin-preset=mixin_buildbot_install_components_with_clang
10671083
ios
10681084
tvos
10691085
watchos
@@ -1105,6 +1121,7 @@ skip-install-swiftsyntax-module
11051121
install-skstresstester
11061122
install-swiftevolve
11071123
install-playgroundsupport
1124+
install-libcxx
11081125

11091126
install-destdir=%(install_destdir)s
11101127

@@ -1125,10 +1142,6 @@ test-installable-package
11251142
# If someone uses this for incremental builds, force reconfiguration.
11261143
reconfigure
11271144

1128-
swift-install-components=compiler;clang-resource-dir-symlink;stdlib;sdk-overlay;parser-lib;toolchain-tools;license;sourcekit-xpc-service;swift-remote-mirror;swift-remote-mirror-headers
1129-
llvm-install-components=llvm-cov;llvm-profdata;IndexStore;clang;clang-headers;compiler-rt;clangd
1130-
install-libcxx
1131-
11321145
# Path to the .tar.gz package we would create.
11331146
installable-package=%(installable_package)s
11341147

@@ -1304,6 +1317,7 @@ swift-stdlib-build-type=Release
13041317
mixin-preset=
13051318
buildbot_incremental_base_all_platforms
13061319
lldb-smoketest,tools=RA
1320+
mixin_buildbot_install_components_with_clang
13071321

13081322
build-subdir=buildbot_incremental
13091323

@@ -1322,6 +1336,7 @@ sourcekit-lsp
13221336
install-swift
13231337
install-llbuild
13241338
install-swiftpm
1339+
install-libcxx
13251340

13261341
# Build Playground support
13271342
playgroundsupport
@@ -1368,7 +1383,9 @@ skip-test-osx
13681383
#===------------------------------------------------------------------------===#
13691384

13701385
[preset: buildbot_swiftpm_macos_platform,tools=RA,stdlib=RA]
1371-
mixin-preset=buildbot_incremental_base
1386+
mixin-preset=
1387+
buildbot_incremental_base
1388+
mixin_buildbot_install_components_with_clang
13721389

13731390
build-subdir=buildbot_incremental
13741391

@@ -1385,6 +1402,7 @@ sourcekit-lsp
13851402
install-swift
13861403
install-llbuild
13871404
install-swiftpm
1405+
install-libcxx
13881406

13891407
dash-dash
13901408

@@ -1397,7 +1415,9 @@ skip-test-llbuild
13971415
#===------------------------------------------------------------------------===#
13981416

13991417
[preset: buildbot_swiftpm_linux_platform,tools=RA,stdlib=RA]
1400-
mixin-preset=buildbot_incremental_base
1418+
mixin-preset=
1419+
buildbot_incremental_base
1420+
mixin_linux_install_components_with_clang
14011421

14021422
build-subdir=buildbot_incremental
14031423

@@ -1419,6 +1439,7 @@ install-swiftpm
14191439
install-foundation
14201440
install-libdispatch
14211441
install-xctest
1442+
install-libcxx
14221443

14231444
skip-test-swift
14241445
skip-test-cmark
@@ -1469,22 +1490,27 @@ sourcekit-lsp
14691490
install-swift
14701491
install-llbuild
14711492
install-swiftpm
1493+
install-libcxx
14721494

14731495
skip-test-swift
14741496
skip-test-cmark
14751497
skip-test-llbuild
14761498
skip-test-swiftpm
14771499

14781500
[preset: buildbot_swiftpm_package_macos]
1479-
mixin-preset=buildbot_swiftpm_package_base
1501+
mixin-preset=
1502+
buildbot_swiftpm_package_base
1503+
mixin_buildbot_install_components_with_clang
14801504

14811505
# Build stdlib for all platforms.
14821506
ios
14831507
tvos
14841508
watchos
14851509

14861510
[preset: buildbot_swiftpm_package_linux]
1487-
mixin-preset=buildbot_swiftpm_package_base
1511+
mixin-preset=
1512+
buildbot_swiftpm_package_base
1513+
mixin_linux_install_components_with_clang
14881514

14891515
libdispatch
14901516
foundation

0 commit comments

Comments
 (0)