-
Notifications
You must be signed in to change notification settings - Fork 66
Updated link_nvidia_host_libraries.sh for better edge case handling #922
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
Updated link_nvidia_host_libraries.sh for better edge case handling #922
Conversation
…nd better maintainability.
|
Instance
|
|
Instance
|
|
Instance
|
|
Instance
|
|
Instance
|
|
Instance
|
…SI initialization step
…lt EESSI env vars
…jections directory path from EESSI_HOST_INJECTIONS
…s for symlinks, added fail fast on normal run.
…exit code handling
…exit code handling
…exit code handling
…exit code handling
… ldconfig and linking it
…k-nvidia-libs.sh to include all fake libraries
ocaisa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of minor tweaks
Co-authored-by: ocaisa <[email protected]>
Co-authored-by: ocaisa <[email protected]>
Co-authored-by: ocaisa <[email protected]>
Add newlines at end of files
ocaisa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for effort @Darkless012
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:x86_64/amd/zen2 |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
|
New job on instance
|
|
Label |
1 similar comment
|
Label |
|
Staging PR has been merged |
|
PR merged! Moved |
|
PR merged! Moved |
Included uncommitted remarks in #754 (review)
Script Overview:
Function definition LINES 70-585:
Script logic: LINES 585-650