Solved some bugs

scripts/client/check.sh

@@ -2,7 +2,7 @@
 # @author Kevin Veen-Birkenbach
 # shellcheck source=/dev/null # Deactivate SC1090
 source "$(dirname "$(readlink -f "${0}")")/../base.sh" || (echo "Loading base.sh failed." && exit 1)
-echo "System check"
+echo "System check - Shows things which can be optimized:"
 echo
 info "Checking relevant home folders for duplicated files..."
 fdupes -r "$HOME/Documents/" "$HOME/Downloads/" "$HOME/Images/" "$HOME/Desktop/" "$HOME/Music/" "$HOME/Pictures/" "$HOME/Videos"
@@ -10,3 +10,5 @@ info "Searching for files which are in \"$HOME\" but don't belong to user \"$USE
 sudo find "$HOME" ! -user "$USER"
 info "Searching for files which are in \"$HOME\" and bigger then 100MB..."
 find ~ -type f -size +100M -exec ls -lh {} \;
+info "Showing the installed Java versions..." &&
+archlinux-java status

scripts/client/setup.sh

@@ -67,8 +67,13 @@ if [ ! -f "$ssh_key_path" ]; then
 	fi
 fi
 
-info "Installing nonfree drivers..." &&
+question "Should nonfree drivers be installed?(y/N)" && read -r install_nonfree_drivers
-sudo mhwd -a pci nonfree 0300 || error
+if [ "$install_nonfree_drivers" = "y" ]
+  then
+    info "Installing nonfree drivers..." &&
+    sudo mhwd -a pci nonfree 0300 || error
+    warning "Nonfree drivers had been installed. This can lead to unexpected side effects!"
+fi
 
 info "Setup, configuration and installation of dependencies for installed software..."
 
@@ -102,7 +107,7 @@ fi
 
 if [ ! "$(pacman -Qi "virtualbox")" ] ; then
 	info "Setting up virtualbox..." &&
-	pamac install virtualbox "$(pacman -Qsq "^linux" | grep "^linux[0-9]*[-rt]*$" | awk '{print $1"-virtualbox-host-modules"}' ORS=' ')" &&
+	sudo pacman -S virtualbox "$(pacman -Qsq "^linux" | grep "^linux[0-9]*[-rt]*$" | awk '{print $1"-virtualbox-host-modules"}' ORS=' ')" &&
 	sudo vboxreload &&
 	pamac build virtualbox-ext-oracle &&
 	sudo gpasswd -a "$USER" vboxusers || error
@@ -203,10 +208,5 @@ if [ "$(ls -A "$autostart_folder")" ]
 		info "No startup entries found. Skipped removing."
 fi
 
-info "Showing the installed Java versions..." &&
-archlinux-java status &&
-info "Keep in mind to set the right Java-Version if it's neccessary." ||
-error
-
 info "Please restart the computer, so that all updates get applied."
 success "Setup finished successfully :)" && exit 0

scripts/image/backup.sh

@@ -1,7 +1,9 @@
 #!/bin/bash
-# shellcheck disable=SC2010
+# shellcheck disable=SC2010  # ls  | grep allowed
 # shellcheck disable=SC2015  # Deactivating bool hint
 # shellcheck source=/dev/null # Deactivate SC1090
+# shellcheck disable=SC2154  # Deactivate not referenced link
+# shellcheck disable=SC2015  # Deactivate bools hints
 source "$(dirname "$(readlink -f "${0}")")/base.sh" || (echo "Loading base.sh failed." && exit 1)
 info "Backupscript for memory devices started..."
 echo

scripts/image/base.sh

@@ -3,7 +3,9 @@
 # Offers base functions for the image management
 #
 # shellcheck disable=SC2034 #Deactivate checking of unused variables
+# shellcheck disable=SC2010  # ls  | grep allowed
 # shellcheck source=/dev/null # Deactivate SC1090
+# shellcheck disable=SC2015  # Deactivate bools hints
 source "$(dirname "$(readlink -f "${0}")")/../base.sh" || (echo "Loading base.sh failed." && exit 1)
 
 # Writes the full partition name

scripts/image/chroot.sh

@@ -1,6 +1,7 @@
 #!/bin/bash
 # shellcheck source=/dev/null # Deactivate SC1090
 # shellcheck disable=SC2015  # Deactivating bool hint
+# shellcheck disable=SC2154  # Deactivate not referenced link
 source "$(dirname "$(readlink -f "${0}")")/base.sh" || (echo "Loading base.sh failed." && exit 1)
 
 info "Starting chroot..."

scripts/image/setup.sh

@@ -1,6 +1,8 @@
 #!/bin/bash
 # shellcheck disable=SC2010  # ls  | grep allowed
 # shellcheck source=/dev/null # Deactivate SC1090
+# shellcheck disable=SC2015  # Deactivate bools hints
+# shellcheck disable=SC2154  # Deactivate not referenced link
 source "$(dirname "$(readlink -f "${0}")")/base.sh" || (echo "Loading base.sh failed." && exit 1)
 
 info "Setupscript for images started..."
@@ -172,7 +174,7 @@ info "Generating os-image..."
 download_url="$base_download_url$imagename"
 image_path="$image_folder$imagename"
 
-question "Should the image download be forced?(y/n)" && read -r force_image_download
+question "Should the image download be forced?(y/N)" && read -r force_image_download
 if [ "$force_image_download" = "y" ]
   then
     if [ -f "$image_path" ]