Skip to content

Commit

Permalink
fix: Use full installer path for installer extraction (#93)
Browse files Browse the repository at this point in the history
- Fixes an oversight from 16b31de
- Fixes the check for whether the installer file was found valid or not

Should fix #91
  • Loading branch information
zelikos authored Jun 20, 2024
1 parent 9f5bdaa commit d089ea1
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
container_type=""
container_create_prefix=""
container_run_prefix=""
installer_path=""
nvidia_gpu=false
valid_installer=false

Expand Down Expand Up @@ -34,7 +35,7 @@ get_gpu_type () {
}

check_davinci_installer () {
installer=$(readlink -e $1)
installer=$1
if [[ ! -f $installer ]]; then
echo "$1 is not a valid filename."
echo "Re-run this script with a valid DaVinci Resolve installer."
Expand Down Expand Up @@ -116,11 +117,12 @@ elif [[ $1 == "upgrade" ]]; then
echo "as you would for a fresh installation."
echo "e.g. ./setup.sh DaVinci_Resolve_18.5.1_Linux.run"
else
installer_path=$(readlink -e $1)
# Create davincibox container on user's system
create_davincibox_container
# Check that provided installer path is valid
check_davinci_installer $1
if [[ valid_installer ]]; then
run_davinci_setup $1
check_davinci_installer $installer_path
if [[ $valid_installer == true ]]; then
run_davinci_setup $installer_path
fi
fi

0 comments on commit d089ea1

Please sign in to comment.