Skip to content

Commit 01216c0

Browse files
committed
Update Python setup and maturin invocation paths
Add id to setup-python action in both Python jobs. Use resolved interpreter path for maturin calls in both the macOS and Windows builds to ensure correct Python version is utilized during the build process.
1 parent 25264c3 commit 01216c0

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/build.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@ jobs:
121121
- uses: actions/checkout@v5
122122

123123
- uses: actions/setup-python@v6
124+
id: setup-python
124125
with:
125126
python-version: ${{ matrix.python-version }}
126127

@@ -181,7 +182,7 @@ jobs:
181182
MATURIN_FEATURES: ${{ endsWith(matrix.python-version, 't') && 'substrait' || 'substrait,py-limited-api' }}
182183
run: |
183184
uv sync --dev --no-install-package datafusion
184-
uv run --no-project maturin build --release --strip --features "${MATURIN_FEATURES}" --interpreter python
185+
uv run --no-project maturin build --release --strip --features "${MATURIN_FEATURES}" --interpreter "${{ steps.setup-python.outputs.python-path }}"
185186
186187
- name: List Windows wheels
187188
if: matrix.os == 'windows-latest'
@@ -226,6 +227,7 @@ jobs:
226227
- uses: actions/checkout@v5
227228

228229
- uses: actions/setup-python@v6
230+
id: setup-python
229231
with:
230232
python-version: ${{ matrix.python-version }}
231233

@@ -286,7 +288,7 @@ jobs:
286288
MATURIN_FEATURES: ${{ endsWith(matrix.python-version, 't') && 'substrait' || 'substrait,py-limited-api' }}
287289
run: |
288290
uv sync --dev --no-install-package datafusion
289-
uv run --no-project maturin build --release --strip --features "${MATURIN_FEATURES}" --interpreter python
291+
uv run --no-project maturin build --release --strip --features "${MATURIN_FEATURES}" --interpreter "${{ steps.setup-python.outputs.python-path }}"
290292
291293
- name: List Mac wheels
292294
run: find target/wheels/

0 commit comments

Comments
 (0)