Skip to content

Commit

Permalink
GA resets environment variables making CUDA and .local/bin undetectable.
Browse files Browse the repository at this point in the history
The git version availible on Baldur at the moment is too old to be
used with the --recursive option. This breaks the GA checkout
action, which defaults to downloading the repo using the GitHub's
REST API.
The cloned repo does not contain the .git directory, i.e. the
cloned files are not an actual git repository. This breaks any
action that relies on git commands to deploy/checkout/etc,
including initializing and updating submodules. Without
cloning pybind11 we can't build.
Additionally, still unclear how to correctly set the
environment to detect correct CUDA compiler. This breaks
build.
  • Loading branch information
DinoBektesevic committed Jan 10, 2023
1 parent 79a1717 commit a664efe
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/test_build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,16 +7,18 @@ jobs:
test-build:
runs-on: self-hosted
steps:
- name: Link to a more modern git
run: echo "$HOME/.local/bin" >> $GITHUB_PATH
- name: Checkout code
uses: actions/checkout@v3
with:
submodules: recursive
- name: Set-up Python
uses: actions/setup-python@v4
with:
python-version: 3.x
python-version: '3.10'
- name: Setup nvcc
run: export PATH="/usr/local/cuda-11.8/bin:$PATH"
run: echo "/usr/local/cuda-11.8/bin" >> $GITHUB_PATH
- name: Build KBMoD
run: python -m pip install .

Expand Down

0 comments on commit a664efe

Please sign in to comment.