Skip to content

cuda.py: Handle cuda-installer.log#1835

Merged
bartoldeman merged 1 commit intoeasybuilders:developfrom
akesandgren:cuda-handle-cuda-installer.log
Oct 18, 2019
Merged

cuda.py: Handle cuda-installer.log#1835
bartoldeman merged 1 commit intoeasybuilders:developfrom
akesandgren:cuda-handle-cuda-installer.log

Conversation

@akesandgren
Copy link
Copy Markdown
Contributor

cuda-installer creates /tmp/cuda-installer.log (ignoring TMPDIR)
Try to remove it before running the installer.
This will fail with a usable error if it can't be removed instead of segfaulting in the cuda-installer.

@akesandgren akesandgren added this to the 4.x milestone Oct 17, 2019
Copy link
Copy Markdown
Contributor

@bartoldeman bartoldeman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@bartoldeman bartoldeman merged commit 9cc6d41 into easybuilders:develop Oct 18, 2019
@akesandgren akesandgren deleted the cuda-handle-cuda-installer.log branch October 18, 2019 12:33
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 this pull request may close these issues.

2 participants