Skip to content

Commit

Permalink
bpo-34855: Fix EXTERNALS_DIR build variable for Windows (GH-11177)
Browse files Browse the repository at this point in the history
(cherry picked from commit 6aedfa6)

Co-authored-by: antektek <45912913+antektek@users.noreply.github.com>
  • Loading branch information
miss-islington and antektek committed Jan 10, 2019
1 parent 1419000 commit 2bd5f7e
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .azure-pipelines/windows-appx-test.yml
Expand Up @@ -36,7 +36,7 @@ jobs:
Write-Host '##vso[task.setvariable variable=Py_IntDir]$(Build.BinariesDirectory)\obj'
# UNDONE: Do not build to a different directory because of broken tests
Write-Host '##vso[task.setvariable variable=Py_OutDir]$(Build.SourcesDirectory)\PCbuild'
Write-Host '##vso[task.setvariable variable=EXTERNAL_DIR]$(Build.BinariesDirectory)\externals'
Write-Host '##vso[task.setvariable variable=EXTERNALS_DIR]$(Build.BinariesDirectory)\externals'
displayName: Update build locations
- script: PCbuild\build.bat -e $(buildOpt)
Expand Down
2 changes: 1 addition & 1 deletion .azure-pipelines/windows-steps.yml
Expand Up @@ -8,7 +8,7 @@ steps:
Write-Host '##vso[task.setvariable variable=Py_IntDir]$(Build.BinariesDirectory)\obj'
# UNDONE: Do not build to a different directory because of broken tests
Write-Host '##vso[task.setvariable variable=Py_OutDir]$(Build.SourcesDirectory)\PCbuild'
Write-Host '##vso[task.setvariable variable=EXTERNAL_DIR]$(Build.BinariesDirectory)\externals'
Write-Host '##vso[task.setvariable variable=EXTERNALS_DIR]$(Build.BinariesDirectory)\externals'
displayName: Update build locations

- script: PCbuild\build.bat -e $(buildOpt)
Expand Down
2 changes: 1 addition & 1 deletion PCbuild/find_python.bat
Expand Up @@ -24,7 +24,7 @@
:begin_search
@set PYTHON=

@set _Py_EXTERNALS_DIR=%EXTERNAL_DIR%
@set _Py_EXTERNALS_DIR=%EXTERNALS_DIR%
@if "%_Py_EXTERNALS_DIR%"=="" (set _Py_EXTERNALS_DIR=%~dp0\..\externals)

@rem If we have Python in externals, use that one
Expand Down

0 comments on commit 2bd5f7e

Please sign in to comment.