strict mode of bash operation: pipefail, stop on errors, stop on undefined variables

fixes https://www.shellcheck.net/wiki/SC2181 -- Check exit code directly with e.g...

fixes nvidia-smi exit code checking bug. without pipefail check always successful because of `head` command.
This commit is contained in:
Vladislav Yarmak 2019-01-05 00:36:48 +02:00
parent d10c45c232
commit 5182417990
No known key found for this signature in database
GPG Key ID: F6C5633BE5DE7127

View File

@ -1,5 +1,7 @@
#!/bin/bash #!/bin/bash
set -euo pipefail
backup_path="/opt/nvidia/libnvidia-encode-backup" backup_path="/opt/nvidia/libnvidia-encode-backup"
silent_flag='' silent_flag=''
rollback_flag='' rollback_flag=''
@ -68,18 +70,15 @@ declare -A object_list=(
["415.25"]='libnvcuvid.so' ["415.25"]='libnvcuvid.so'
) )
driver_version=$(/usr/bin/nvidia-smi --query-gpu=driver_version --format=csv,noheader,nounits | head -n 1)
if [[ ! $? -eq 0 ]]; then if ! driver_version=$(/usr/bin/nvidia-smi --query-gpu=driver_version --format=csv,noheader,nounits | head -n 1) ; then
echo 'Something went wrong. Check nvidia driver' echo 'Something went wrong. Check nvidia driver'
exit 1; exit 1;
fi fi
echo "Detected nvidia driver version: $driver_version" echo "Detected nvidia driver version: $driver_version"
patch="${patch_list[$driver_version]}" if [[ ! -v "patch_list[$driver_version]" || ! -v "object_list[$driver_version]" ]]; then
object="${object_list[$driver_version]}"
if [[ ( ! "$patch" ) || ( ! "$object" ) ]]; then
echo "Patch for this ($driver_version) nvidia driver not found." 1>&2 echo "Patch for this ($driver_version) nvidia driver not found." 1>&2
echo "Available patches for: " 1>&2 echo "Available patches for: " 1>&2
for drv in "${!patch_list[@]}"; do for drv in "${!patch_list[@]}"; do
@ -88,6 +87,9 @@ if [[ ( ! "$patch" ) || ( ! "$object" ) ]]; then
exit 1; exit 1;
fi fi
patch="${patch_list[$driver_version]}"
object="${object_list[$driver_version]}"
if [[ $rollback_flag ]]; then if [[ $rollback_flag ]]; then
if [[ -f "$backup_path/$object.$driver_version" ]]; then if [[ -f "$backup_path/$object.$driver_version" ]]; then
cp -p "$backup_path/$object.$driver_version" \ cp -p "$backup_path/$object.$driver_version" \