Skip to content

Commit 144d22e

Browse files
committed
Add dedicated PATH fix step after Conan installation
The get-conan action installs conan to ~/.local/bin via pip, but on CentOS 7 containers there's also a system conan at /usr/bin/conan (v1.47). Previously, the turtlebrowser/get-conan action would add its own path to GITHUB_PATH, but the order wasn't guaranteed. This caused vfx2022 builds to find the old system conan instead of the newly-installed one. By explicitly adding $HOME/.local/bin to GITHUB_PATH after both uv and conan installation, we ensure that: - uv-installed Python 3.11 is found before system Python 3.7 - pip-installed Conan 2.x is found before system Conan 1.47 Signed-off-by: Gary Oberbrunner <[email protected]>
1 parent f640623 commit 144d22e

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

.github/workflows/build.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -209,9 +209,6 @@ jobs:
209209
# This ensures the sigstore action can find 'python' (not just 'python3')
210210
uv python install --default 3.11
211211
212-
# Add ~/.local/bin to PATH for future steps
213-
echo "$HOME/.local/bin" >> $GITHUB_PATH
214-
215212
- name: Check python, uv paths
216213
run: |
217214
echo "PATH=$PATH"
@@ -226,6 +223,12 @@ jobs:
226223
with:
227224
version: ${{ matrix.conan_version }}
228225

226+
# Ensure ~/.local/bin is first in PATH for uv-installed Python and pip-installed Conan
227+
# This is especially important for old CentOS 7 containers with old system python/conan
228+
- name: Fix PATH for local tools
229+
run: |
230+
echo "$HOME/.local/bin" >> $GITHUB_PATH
231+
229232
- name: Set up conan
230233
run: |
231234
which conan

0 commit comments

Comments
 (0)