-
Notifications
You must be signed in to change notification settings - Fork 283
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
enable installation of samples for CUDA > 10.1 #2374
Conversation
For CUDA > 5 and < 10.1 they are installed by default. For CUDA > 10.1 we need to add --samples to the installer command line. I could not find any evidence of libglut being needed as osdependency (even for CUDA 6), so deleted those comments.
Nevermind, already there. |
Test report by @bartoldeman Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
I just found out this works with 10.2 and 11.0 but fails with 10.1. Checking out why. |
Test report by @boegel Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
@bartoldeman Works fine for me with CUDA 10.1? |
Test report by @boegel Overview of tested easyconfigs (in order)
Build succeeded for 2 out of 2 (2 easyconfigs in total) |
@boegel it's odd, it works for me as normal user but if I use sudo as different user (our software installation user) it complains. I'm still digging to figure out why. |
samples are installed in two places with identical copies: self.installdir/samples and $HOME/NVIDIA_CUDA-11.2_Samples changing the second location to a scratch location (self.builddir) avoids the duplicate
@boegel two things happening here:
|
This avoids issues with eb is called via sudo -iu someuser ... as the CUDA installer tries to chown sample files to $SUDO_USER
Test report by @boegel Overview of tested easyconfigs (in order)
Build succeeded for 10 out of 10 (10 easyconfigs in total) |
For CUDA > 5 and < 10.1 they are installed by default.
For CUDA > 10.1 we need to add --samples to the installer command
line.
I could not find any evidence of libglut being needed as osdependency
(even for CUDA 6), so deleted those comments.