Skip to content

Commit dec2f24

Browse files
patelchaitanypatelchaitany
authored
fix: Pin build dependencies of source builds to the CI lock (#7104)
sqlglotc has no Python 3.10 wheels on PyPI, so on 3.10 uv builds sqlglotc==30.21.0 from source. Its build pulls the newest unpinned sqlglot, whose Schema class is no longer a mypyc trait, and mypyc fails with "Multiple inheritance is not supported (except for traits)". Pass the CI lock as --build-constraints so source builds use the locked versions (sqlglot==30.21.0) instead of the newest release. Signed-off-by: Chaitany Patel <[email protected]>
1 parent e0eb369 commit dec2f24

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

‎Makefile‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,11 @@ install-python-dependencies-ci: ## Install Python CI dependencies using uv pip s
122122
# Must match the --torch-backend used to compile the CI lock, otherwise the
123123
# recorded CPU-wheel hashes (from download.pytorch.org) fail to verify.
124124
# The lock is universal, so one command serves both Linux and macOS.
125-
uv pip sync --torch-backend cpu sdk/python/requirements/py$(PYTHON_VERSION)-ci-requirements.txt
125+
# --build-constraints pins build-time deps of source builds to the lock too
126+
# (e.g. sqlglotc on Python 3.10 has no wheel and must compile the locked sqlglot).
127+
uv pip sync --torch-backend cpu \
128+
--build-constraints sdk/python/requirements/py$(PYTHON_VERSION)-ci-requirements.txt \
129+
sdk/python/requirements/py$(PYTHON_VERSION)-ci-requirements.txt
126130
uv pip install --no-deps -e .
127131

128132
# Used in github actions/ci

0 commit comments

Comments
 (0)