$(TailcallingOption) not needed anymore

add missing ${{ parameters.TailcallingOption }}

remove unneeded TailcallingOption: ''

Author: chris-eibl

windows-release/build-steps-pgo.yml

@@ -22,7 +22,7 @@ steps:
 - ${{ if eq(parameters.PGInstrument, 'true') }}:
   - powershell: |
       $env:SigningCertificate = $null
-      .\PCbuild\build.bat -v -p $(Platform) -c PGInstrument $(ExtraOptions) $(TailcallingOption)
+      .\PCbuild\build.bat -v -p $(Platform) -c PGInstrument $(ExtraOptions)
     displayName: 'Run build'
     env:
       IncludeUwp: true
@@ -100,7 +100,7 @@ steps:
 
   - powershell: |
       $env:SigningCertificate = $null
-      .\PCbuild\build.bat -v -p $(Platform) -c PGUpdate $(ExtraOptions) $(TailcallingOption)
+      .\PCbuild\build.bat -v -p $(Platform) -c PGUpdate $(ExtraOptions)
     displayName: 'Run build with PGO'
     env:
       IncludeUwp: true

windows-release/build-steps.yml

@@ -18,7 +18,7 @@ steps:
 - ${{ if eq(parameters.ShouldPGO, 'false') }}:
   - powershell: |
       $env:SigningCertificate = $null
-      .\PCbuild\build.bat -v -p $(Platform) -c $(Configuration) $(ExtraOptions) $(TailcallingOption)
+      .\PCbuild\build.bat -v -p $(Platform) -c $(Configuration) $(ExtraOptions)
     displayName: 'Run build'
     env:
       IncludeUwp: true
@@ -27,7 +27,7 @@ steps:
 - ${{ else }}:
   - powershell: |
       $env:SigningCertificate = $null
-      .\PCbuild\build.bat -v -p $(Platform) --pgo $(ExtraOptions) $(TailcallingOption)
+      .\PCbuild\build.bat -v -p $(Platform) --pgo $(ExtraOptions)
     displayName: 'Run build with PGO'
     env:
       IncludeUwp: true

windows-release/stage-build.yml

@@ -53,7 +53,6 @@ jobs:
         Configuration: Release
         _HostPython: .\python
         ExtraOptions: ${{ parameters.ExtraOptions }}
-        TailcallingOption: ''
         ${{ if eq(parameters.ToBeSigned, 'true') }}:
           Artifact: unsigned_win32
         ${{ else }}:
@@ -65,7 +64,6 @@ jobs:
         Configuration: Debug
         _HostPython: .\python
         ExtraOptions: ${{ parameters.ExtraOptions }}
-        TailcallingOption: ''
         Artifact: bin_win32_d
       ${{ if ne(parameters.DoPGO, 'true') }}:
         amd64:
@@ -86,7 +84,6 @@ jobs:
         Configuration: Debug
         _HostPython: .\python
         ExtraOptions: ${{ parameters.ExtraOptions }}
-        TailcallingOption: ''
         Artifact: bin_amd64_d
       ${{ if or(ne(parameters.DoPGO, 'true'), ne(parameters.DoPGOARM64, 'true')) }}:
         arm64:
@@ -96,7 +93,6 @@ jobs:
           Configuration: Release
           _HostPython: python
           ExtraOptions: ${{ parameters.ExtraOptions }}
-          TailcallingOption: ''
           ${{ if eq(parameters.ToBeSigned, 'true') }}:
             Artifact: unsigned_arm64
           ${{ else }}:
@@ -108,7 +104,6 @@ jobs:
         Configuration: Debug
         _HostPython: python
         ExtraOptions: ${{ parameters.ExtraOptions }}
-        TailcallingOption: ''
         Artifact: bin_arm64_d
       ${{ if eq(parameters.DoFreethreaded, 'true') }}:
         win32_t:
@@ -118,7 +113,6 @@ jobs:
           Configuration: Release
           _HostPython: .\python
           ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }}
-          TailcallingOption: ''
           ${{ if eq(parameters.ToBeSigned, 'true') }}:
             Artifact: unsigned_win32_t
           ${{ else }}:
@@ -130,7 +124,6 @@ jobs:
           Configuration: Debug
           _HostPython: .\python
           ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }}
-          TailcallingOption: ''
           Artifact: bin_win32_td
         ${{ if ne(parameters.DoPGO, 'true') }}:
           amd64_t:
@@ -139,8 +132,7 @@ jobs:
             Platform: x64
             Configuration: Release
             _HostPython: .\python
-            ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }}
-            TailcallingOption: ${{ parameters.TailcallingOption }}
+            ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }} ${{ parameters.TailcallingOption }}
             ${{ if eq(parameters.ToBeSigned, 'true') }}:
               Artifact: unsigned_amd64_t
             ${{ else }}:
@@ -152,7 +144,6 @@ jobs:
           Configuration: Debug
           _HostPython: .\python
           ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }}
-          TailcallingOption: ''
           Artifact: bin_amd64_td
         ${{ if or(ne(parameters.DoPGO, 'true'), ne(parameters.DoPGOARM64, 'true')) }}:
           arm64_t:
@@ -162,7 +153,6 @@ jobs:
             Configuration: Release
             _HostPython: python
             ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }}
-            TailcallingOption: ''
             ${{ if eq(parameters.ToBeSigned, 'true') }}:
               Artifact: unsigned_arm64_t
             ${{ else }}:
@@ -174,7 +164,6 @@ jobs:
           Configuration: Debug
           _HostPython: python
           ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }}
-          TailcallingOption: ''
           Artifact: bin_arm64_td
 
   steps:
@@ -197,8 +186,7 @@ jobs:
           Platform: x64
           _HostPython: .\python
           PythonExePattern: python.exe
-          ExtraOptions: ${{ parameters.ExtraOptions }}
-          TailcallingOption: ${{ parameters.TailcallingOption }}
+          ExtraOptions: ${{ parameters.ExtraOptions }} ${{ parameters.TailcallingOption }}
           ${{ if eq(parameters.ToBeSigned, 'true') }}:
             Artifact: unsigned_amd64
           ${{ else }}:
@@ -210,8 +198,7 @@ jobs:
             Platform: x64
             _HostPython: .\python
             PythonExePattern: python3*t.exe
-            ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }}
-            TailcallingOption: ${{ parameters.TailcallingOption }}
+            ExtraOptions: ${{ parameters.ExtraOptionsFreethreaded }} ${{ parameters.TailcallingOption }}
             ${{ if eq(parameters.ToBeSigned, 'true') }}:
               Artifact: unsigned_amd64_t
             ${{ else }}: