Skip to content

Commit b8be792

Browse files
committed
Remove PinnedToolchainVarinat variable
1 parent 96b4038 commit b8be792

File tree

1 file changed

+10
-16
lines changed

1 file changed

+10
-16
lines changed

utils/build.ps1

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,6 @@ The toolchain snapshot to build the early components with.
5454
.PARAMETER PinnedSHA256
5555
The SHA256 for the pinned toolchain.
5656
57-
.PARAMETER PinnedToolchainVariant
58-
The toolchain variant to use while building the toolchain. Defaults to
59-
`Asserts`.
60-
6157
.PARAMETER AndroidNDKVersion
6258
The version number of the Android NDK to be used.
6359
@@ -132,8 +128,6 @@ param
132128
[ValidatePattern("^([A-Fa-f0-9]{64}|)$")]
133129
[string] $PinnedSHA256 = "",
134130
[string] $PinnedVersion = "",
135-
[ValidateSet("Asserts", "NoAsserts")]
136-
[string] $PinnedToolchainVariant = "Asserts",
137131
[ValidatePattern('^\d+(\.\d+)*$')]
138132
[string] $PythonVersion = "3.9.10",
139133
[ValidatePattern("^r(?:[1-9]|[1-9][0-9])(?:[a-z])?$")]
@@ -622,9 +616,9 @@ function Get-InstallDir([Hashtable] $Platform) {
622616

623617
# For dev productivity, install the host toolchain directly using CMake.
624618
# This allows iterating on the toolchain using ninja builds.
625-
$HostPlatform.ToolchainInstallRoot = "$(Get-InstallDir $HostPlatform)\Toolchains\$ProductVersion+$PinnedToolchainVariant"
619+
$HostPlatform.ToolchainInstallRoot = "$(Get-InstallDir $HostPlatform)\Toolchains\$ProductVersion+Asserts"
626620
$HostPlatform.NoAssertsToolchainInstallRoot = "$(Get-InstallDir $HostPlatform)\Toolchains\$ProductVersion+NoAsserts"
627-
$BuildPlatform.ToolchainInstallRoot = "$(Get-InstallDir $BuildPlatform)\Toolchains\$ProductVersion+$PinnedToolchainVariant"
621+
$BuildPlatform.ToolchainInstallRoot = "$(Get-InstallDir $BuildPlatform)\Toolchains\$ProductVersion+Asserts"
628622
$BuildPlatform.NoAssertsToolchainInstallRoot = "$(Get-InstallDir $BuildPlatform)\Toolchains\$ProductVersion+NoAsserts"
629623

630624
# Build functions
@@ -1195,15 +1189,15 @@ function Get-PinnedToolchainToolsDir() {
11951189
}
11961190
}
11971191

1198-
$VariantToolchainPath = [IO.Path]::Combine($ToolchainsRoot, "$(Get-PinnedToolchainVersion)+$PinnedToolchainVariant", "usr", "bin")
1192+
$VariantToolchainPath = [IO.Path]::Combine($ToolchainsRoot, "$(Get-PinnedToolchainVersion)+Asserts", "usr", "bin")
11991193

12001194
if (Test-Path $VariantToolchainPath) {
12011195
return $VariantToolchainPath
12021196
}
12031197

12041198
return [IO.Path]::Combine("$BinaryCache\", "toolchains", $PinnedToolchain,
12051199
"Library", "Developer", "Toolchains",
1206-
"unknown-$PinnedToolchainVariant-development.xctoolchain", "usr", "bin")
1200+
"unknown-Asserts-development.xctoolchain", "usr", "bin")
12071201
}
12081202

12091203
function Get-PinnedToolchainSDK() {
@@ -1710,7 +1704,7 @@ function Build-CMark([Hashtable] $Platform) {
17101704
Build-CMakeProject `
17111705
-Src $SourceCache\cmark `
17121706
-Bin (Get-CMarkBinaryCache $Platform) `
1713-
-InstallTo "$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+$PinnedToolchainVariant\usr" `
1707+
-InstallTo "$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+Asserts\usr" `
17141708
-Platform $Platform `
17151709
-Defines @{
17161710
BUILD_SHARED_LIBS = "YES";
@@ -1753,7 +1747,7 @@ function Build-BuildTools([Hashtable] $Platform) {
17531747
SWIFT_INCLUDE_APINOTES = "NO";
17541748
SWIFT_INCLUDE_DOCS = "NO";
17551749
SWIFT_INCLUDE_TESTS = "NO";
1756-
"cmark-gfm_DIR" = "$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+$PinnedToolchainVariant\usr\lib\cmake";
1750+
"cmark-gfm_DIR" = "$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+Asserts\usr\lib\cmake";
17571751
}
17581752
}
17591753

@@ -3255,8 +3249,8 @@ function Test-PackageManager() {
32553249
-Src $SrcDir `
32563250
-Bin "$BinaryCache\$($HostPlatform.Triple)\PackageManagerTests" `
32573251
-Platform $HostPlatform `
3258-
-Xcc "-I$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+$PinnedToolchainVariant\usr\include" `
3259-
-Xlinker "-L$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+$PinnedToolchainVariant\usr\lib"
3252+
-Xcc "-I$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+Asserts\usr\include" `
3253+
-Xlinker "-L$(Get-InstallDir $Platform)\Toolchains\$ProductVersion+Asserts\usr\lib"
32603254
}
32613255

32623256
function Build-Installer([Hashtable] $Platform) {
@@ -3338,7 +3332,7 @@ if (-not $SkipBuild) {
33383332
Invoke-BuildStep Build-BuildTools $BuildPlatform
33393333
if ($IsCrossCompiling) {
33403334
Invoke-BuildStep Build-XML2 $BuildPlatform
3341-
Invoke-BuildStep Build-Compilers $BuildPlatform -Variant $PinnedToolchainVariant
3335+
Invoke-BuildStep Build-Compilers $BuildPlatform -Variant "Asserts"
33423336
}
33433337
if ($IncludeDS2) {
33443338
Invoke-BuildStep Build-RegsGen2 $BuildPlatform
@@ -3446,7 +3440,7 @@ if (-not $IsCrossCompiling) {
34463440
"-TestLLVM" = $Test -contains "llvm";
34473441
"-TestSwift" = $Test -contains "swift";
34483442
}
3449-
Invoke-BuildStep Test-Compilers $HostPlatform $PinnedToolchainVariant $Tests
3443+
Invoke-BuildStep Test-Compilers $HostPlatform "Asserts" $Tests
34503444
}
34513445

34523446
# FIXME(jeffdav): Invoke-BuildStep needs a platform dictionary, even though the Test-

0 commit comments

Comments
 (0)