Separate package manager detection and cache update functions

Signed-off-by: Christian König <github@yubiuser.dev>

advanced/Scripts/update.sh

@@ -114,7 +114,6 @@ main() {
 
 
     # Install packages used by this installation script (necessary if users have removed e.g. git from their systems)
-    check_fresh_install
     package_manager_detect
     build_dependency_package
     install_dependent_packages

automated install/basic-install.sh

@@ -254,10 +254,6 @@ package_manager_detect() {
         PKG_COUNT="${PKG_MANAGER} -s -o Debug::NoLocking=true upgrade | grep -c ^Inst || true"
         # The command we will use to remove packages (used in the uninstaller)
         PKG_REMOVE="${PKG_MANAGER} -y remove --purge"
-        # Update package cache only on fresh installs
-        if [[ "${fresh_install}" == true ]]; then
-            update_package_cache || exit 1
-        fi
 
     # If apt-get is not found, check for rpm.
     elif is_command rpm; then
@@ -2208,6 +2204,11 @@ main() {
     # Check for supported package managers so that we may install dependencies
     package_manager_detect
 
+    # Update package cache only on fresh installs and apt based systems
+    if [[ "${fresh_install}" == true ]] && is_command apt-get; then
+            update_package_cache || exit 1
+    fi
+
     # Notify user of package availability
     notify_package_updates_available
 

test/test_any_automated_install.py

@@ -471,6 +471,7 @@ def test_package_manager_has_pihole_deps(host):
         """
     source /opt/pihole/basic-install.sh
     package_manager_detect
+    update_package_cache
     build_dependency_package
     install_dependent_packages
     """
@@ -487,6 +488,7 @@ def test_meta_package_uninstall(host):
         """
     source /opt/pihole/basic-install.sh
     package_manager_detect
+    update_package_cache
     build_dependency_package
     install_dependent_packages
     """