Skip to content

Commit c968854

Browse files
ramilbakhshyievabhinavdangeti
authored andcommitted
Add conda bin to path early in the cmake GitHub action (facebookresearch#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
1 parent 2e13c17 commit c968854

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/actions/build_cmake/action.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ runs:
3030
run: |
3131
conda install -y -q -c conda-forge gxx_linux-64=11.2 sysroot_linux-64=2.28
3232
conda install -y -q python=3.11 cmake make swig mkl=2023 mkl-devel=2023 numpy scipy pytest
33+
echo "$CONDA/bin" >> $GITHUB_PATH
3334
- name: Install CUDA
3435
if: inputs.gpu == 'ON' && inputs.raft == 'OFF'
3536
shell: bash
@@ -72,7 +73,6 @@ runs:
7273
shell: bash
7374
run: |
7475
conda install -y pytest
75-
echo "$CONDA/bin" >> $GITHUB_PATH
7676
- name: Python tests (CPU only)
7777
if: inputs.gpu == 'OFF'
7878
shell: bash

0 commit comments

Comments
 (0)