Skip to content

Commit

Permalink
Add conda bin to path early in the cmake GitHub action (facebookresea…
Browse files Browse the repository at this point in the history
…rch#3512)

Summary:
Pull Request resolved: facebookresearch#3512

Pull Request resolved: facebookresearch#3510

GitHub hosted runners some with the build-essentials package pre-installed, self-hosted runners on AWS do not have this package. This made it all steps other than the `all targets` one fall back to the system executables which unintentially worked on GitHub hosted runners but not on the self-hosted ones. This diff fixes it by pulling the line that adds conda bin to path early in the cmake build action.

Reviewed By: asadoughi

Differential Revision: D58513853

fbshipit-source-id: 23e95459e0031c96bd142515db07d1b700d713cf
  • Loading branch information
ramilbakhshyiev authored and aalekhpatel07 committed Oct 17, 2024
1 parent a27fe5c commit 6c3504e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/actions/build_cmake/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ runs:
run: |
conda install -y -q -c conda-forge gxx_linux-64=11.2 sysroot_linux-64=2.28
conda install -y -q python=3.11 cmake make swig mkl=2023 mkl-devel=2023 numpy scipy pytest
echo "$CONDA/bin" >> $GITHUB_PATH
- name: Install CUDA
if: inputs.gpu == 'ON' && inputs.raft == 'OFF'
shell: bash
Expand Down Expand Up @@ -72,7 +73,6 @@ runs:
shell: bash
run: |
conda install -y pytest
echo "$CONDA/bin" >> $GITHUB_PATH
- name: Python tests (CPU only)
if: inputs.gpu == 'OFF'
shell: bash
Expand Down

0 comments on commit 6c3504e

Please sign in to comment.