Skip to content

Commit

Permalink
improve windows builds
Browse files Browse the repository at this point in the history
- add required nvcuda.dll for those who don't have it
- lower CUDA (therefore nvidia driver) requirements
- untrack PTX

I am aware of an issue with ANSI escape sequences showing in windows
terminals. This seems related to
eyre-rs/color-eyre#80 and the owo_colors crate.

Not sure how to fix that. Maybe switch to anyhow and use crossterm
instead of owo colors would work, but we would loose the colorful error
reports.
  • Loading branch information
truelossless committed Jul 27, 2022
1 parent 07eb0cc commit 013f71e
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 24,654 deletions.
16 changes: 11 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
run: docker build -t rust-cuda .

- name: Start Docker
run: docker run -t -d -v $PWD:/root/rust-cuda --name builder rust-cuda
run: docker run -t -d -v $PWD:/root/rust-cuda --name builder rust-cuda

- name: Build executable
run: docker exec -w /root/rust-cuda/cli builder cargo build --release
Expand All @@ -37,7 +37,7 @@ jobs:
- name: Install CUDA
uses: Jimver/[email protected]
with:
cuda: '11.7.0'
cuda: "11.2.2"

- name: Build executable
run: |
Expand All @@ -53,11 +53,17 @@ jobs:
cd ../cli
cargo test --release --no-fail-fast
- name: Zip with DLL
run: |
cd cli/target/release/
cp C:\\Windows\\System32\\nvcuda.dll .
7z a cugparck-cli.zip nvcuda.dll cugparck-cli.exe
- name: Upload artifact
uses: actions/upload-artifact@v3
with:
name: cug-windows
path: cli/target/release/cugparck-cli.exe
path: cli/target/release/cugparck-cli.zip

upload:
needs: [build_windows, build_linux]
Expand Down Expand Up @@ -105,6 +111,6 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
upload_url: ${{ steps.create_release.outputs.upload_url }}
asset_path: cugparck-cli.exe
asset_name: cug-windows-${{ steps.commit.outputs.hash }}.exe
asset_path: cugparck-cli.zip
asset_name: cug-windows-${{ steps.commit.outputs.hash }}.zip
asset_content_type: application/zip
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
.vscode/
.devcontainer/

resources/module.ptx

# Generated by Cargo
# will have compiled files and executables
Expand Down
Loading

0 comments on commit 013f71e

Please sign in to comment.