Skip to content

Commit 8375773

Browse files
authored
Update build-windows to use any windows runner (#3532)
Also make the workflow consistent with other windows workflows: use venv, clean up old workspaces.
1 parent 46e79e9 commit 8375773

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

.github/workflows/build-windows.yml

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ jobs:
2222
name: Build
2323
runs-on:
2424
- windows
25-
- avc336
2625
steps:
2726
- name: Enable long paths
2827
run: |
@@ -36,27 +35,30 @@ jobs:
3635
with:
3736
python-version: '3.9'
3837

38+
- name: Clean up old workspaces
39+
shell: bash
40+
run: |
41+
rm -rf /c/gh*
42+
3943
# Copy workspace to a temporary location with a shorter name.
4044
- name: Copy workspace
4145
run: |
4246
Copy-Item -Path ${{ github.workspace }} -Destination ${{ env.NEW_WORKSPACE }} -Recurse
4347
48+
- name: Create venv
49+
run:
50+
python -m venv .venv
51+
4452
# We need ninja >= 1.12.0 to support long names on Windows. At the moment there is no required
4553
# version in pypi, so instead of installing ninja with pip we use a preinstalled 1.12.1 on the
4654
# runner.
47-
- name: Build Triton
55+
- name: Setup Triton
4856
run: |
57+
.venv\Scripts\activate.ps1
58+
Invoke-BatchFile "C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\Build\vcvarsall.bat" x64
4959
cd ${{ env.NEW_WORKSPACE }}
50-
51-
cmd /c '"C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\Build\vcvarsall.bat" x64 && set' | ForEach-Object {
52-
if ($_ -match '^(.*?)=(.*)$') {
53-
[Environment]::SetEnvironmentVariable($matches[1], $matches[2])
54-
}
55-
}
56-
5760
cd python
58-
pip install -U wheel pybind11 certifi cython cmake setuptools>=65.6.1
59-
python -m certifi
61+
pip install -U wheel pybind11 cython cmake 'setuptools>=65.6.1'
6062
pip install -v --no-build-isolation '.[build]'
6163
6264
- name: Clean

0 commit comments

Comments
 (0)