Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support cuda install from package manager #2

Closed
klem23 opened this issue Apr 18, 2019 · 3 comments · Fixed by #11
Closed

Support cuda install from package manager #2

klem23 opened this issue Apr 18, 2019 · 3 comments · Fixed by #11
Assignees
Labels

Comments

@klem23
Copy link
Member

klem23 commented Apr 18, 2019

Change our cmake cuda detection file so it can be able to both detect manual cuda installation and cuda installation through package manager.

@jeremad
Copy link

jeremad commented Apr 24, 2019

I think I've done that by accident when adding the CI!

@klem23
Copy link
Member Author

klem23 commented Apr 25, 2019

an happy accident 😂

I check on the ubuntu plugged on my tv, it just need to remove the NO_DEFAULT_PATH for libcudart.
The docker container you use for Travis are from nvidia, and it uses the nvidia install path (/usr/local/cuda), the deb/ubuntu package use the FHS (/usr/include, /usr/lib/x86_64-linux-gnu/). But luckily, the system path are default path for cmake, and they are check after path defined in HINTS (so a manual installed is used over system install when both are there).
Also with system packages, all header are directly in /usr/include/, what a mess.

@klem23
Copy link
Member Author

klem23 commented Apr 25, 2019

I still have the problem of nvcuvid include before we can compile nvcodec straight on linux.

@klem23 klem23 mentioned this issue Apr 25, 2019
@klem23 klem23 closed this as completed in #11 May 8, 2019
@w-m w-m added the CUDA label May 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants