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

Fixes #23

Closed
Baddest-Juju opened this issue Mar 16, 2018 · 2 comments
Closed

Fixes #23

Baddest-Juju opened this issue Mar 16, 2018 · 2 comments

Comments

@Baddest-Juju
Copy link

Baddest-Juju commented Mar 16, 2018

Script changes needed in this repo

PATH variable of cuda bat files not correctly stated, leading to errors when it hits parantheses

set PATH=%CUDA_PATH_V9_1%\bin;%PATH%
should be
set "PATH=%CUDA_PATH_V9_1%\bin;%PATH%"

setup.bat needs to declare the proper version of VC to compile with.

call "%VS15VCVARSALL%" x86_amd64
becomes
call "%VS15VCVARSALL%" x86_amd64 -vcvars_ver=14.11.25503

What the user has to do (that should be documented)

The proper components must be installed within the Individual Components tab of Visual Studio Installer (except for the VC2015 one)

image

A CUDA file must then be modified because it has a check that breaks everything because the _MSC_VER variable does not get correctly passed to it.

C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v9.1\include\crt\host_config.h

#if _MSC_VER < 1600 || _MSC_VER > 1911
becomes
#if _MSC_VER < 1600

After all that, it'll compile.

@peterjc123
Copy link
Owner

@Baddest-Juju Thanks for your description. Can you make a PR on this?

@Baddest-Juju
Copy link
Author

Nope sorry, spent enough time on this already, no offense. I got to get back to paid work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants