Skip to content

Commit 3434543

Browse files
committed
[Windows] Re-enable tokenizers
1 parent 9ffcf4b commit 3434543

File tree

2 files changed

+9
-7
lines changed

2 files changed

+9
-7
lines changed

.ci/scripts/setup-windows.ps1

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,11 @@ conda activate et
1313
# Install test dependencies
1414
pip install -r .ci/docker/requirements-ci.txt
1515

16+
# Create a symlink to work around path length issues when building submodules (tokenizers).
17+
Push-Location
18+
New-Item -ItemType SymbolicLink -Path "C:\_et" -Target "$CWD"
19+
cd C:\_et
20+
1621
if ($editable -eq 'true') {
1722
install_executorch.bat --editable
1823
} else {
@@ -22,3 +27,5 @@ if ($LASTEXITCODE -ne 0) {
2227
Write-Host "Installation was unsuccessful. Exit code: $LASTEXITCODE."
2328
exit $LASTEXITCODE
2429
}
30+
31+
Pop-Location

install_requirements.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -112,13 +112,8 @@ def install_requirements(use_pytorch_nightly):
112112

113113
LOCAL_REQUIREMENTS = [
114114
"third-party/ao", # We need the latest kernels for fast iteration, so not relying on pypi.
115-
] + (
116-
[
117-
"extension/llm/tokenizers", # TODO(larryliu0820): Setup a pypi package for this.
118-
]
119-
if sys.platform != "win32"
120-
else []
121-
) # TODO(gjcomer): Re-enable when buildable on Windows.
115+
"extension/llm/tokenizers", # TODO(larryliu0820): Setup a pypi package for this.
116+
]
122117

123118
# Install packages directly from local copy instead of pypi.
124119
# This is usually not recommended.

0 commit comments

Comments
 (0)