Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

Commit

Permalink
add rust-toolchain and lock dependencies (#26)
Browse files Browse the repository at this point in the history
# What ❔

This PR adds rust-toolchain file to set a specific toolchain version,
adds Cargo.lock to source control to lick the dependencies and adjusts
the CI script to use the specific version from rust-toolchain.

## Why ❔

Unspecified toolchain and unlocked dependencies are a source of
compilation issues.

## Checklist

- [x] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
  • Loading branch information
robik75 authored Feb 9, 2024
1 parent 97bdb63 commit a356c2c
Show file tree
Hide file tree
Showing 5 changed files with 1,944 additions and 18 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,15 +49,15 @@ jobs:
- name: Setup rust
run: |
rustup set profile minimal
rustup toolchain install nightly-2023-04-17
rustup default nightly-2023-04-17
rustup toolchain install nightly-2023-06-25
rustup default nightly-2023-06-25
- name: Build tests and copy binaries to a separate dir
shell: bash
run: |
mkdir artifacts
CUDAARCHS=80 CARGO_TARGET_DIR=./build \
cargo +nightly-2023-04-17 test --no-run --release --message-format=json -q \
cargo +nightly-2023-06-25 test --no-run --release --message-format=json -q \
| jq -r 'select(.executable != null) | .executable' \
| while read binary; do
cp "$binary" artifacts/
Expand Down
4 changes: 0 additions & 4 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,6 @@
# will have compiled files and executables
/target/

# Remove Cargo.lock from gitignore if creating an executable, leave it for libraries
# More information here https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html
Cargo.lock

# These are backup files generated by rustfmt
**/*.rs.bk

Expand Down
Loading

0 comments on commit a356c2c

Please sign in to comment.