Support dnf5 for qubes-download-dom0-update.sh

package-managers/qubes-download-dom0-updates.sh

@@ -17,7 +17,15 @@ CLEAN_OPTS=("${OPTS[@]}")
 # DNF verifies signatures implicitly, but yumdownloader does not.
 SIGNATURE_REGEX=""
 PKGLIST=()
-YUM_ACTION=
+
+# Executable (yum or dnf)
+UPDATE_CMD=
+# Action (install, search, upgrade, ...)
+UPDATE_ACTION=
+# Arguments (--downloadonly, -y, --refresh, ...)
+UPDATE_ARGUMENTS=()
+# Finall fakeroot command to be executed
+UPDATE_COMMAND=
 
 export LC_ALL=C
 
@@ -37,32 +45,40 @@ while [ -n "$1" ]; do
             ;;
         --check-only)
             CHECK_ONLY=1
+            UPDATE_ACTION=check-update
             ;;
         --action=*)
-            YUM_ACTION=${1#--action=}
+            UPDATE_ACTION=${1#--action=}
             ;;
         -*)
             # we already add these options for DNF, and Yum doesn’t support them
             case $1 in (--best|--allowerasing) :;; (*) OPTS+=("$1");; esac
             ;;
         *)
             PKGLIST+=( "${1}" )
-            if [ -z "$YUM_ACTION" ]; then
-                YUM_ACTION=install
+            if [ -z "$UPDATE_ACTION" ]; then
+                UPDATE_ACTION=install
             fi
             ;;
     esac
     shift
 done
 
-if [ -z "$YUM_ACTION" ]; then
-    YUM_ACTION=upgrade
+if [ -z "$UPDATE_ACTION" ]; then
+    UPDATE_ACTION=upgrade
 fi
 
 if type dnf >/dev/null 2>&1; then
-    YUM=(dnf --best --allowerasing --noplugins -y)
+    UPDATE_CMD=dnf
+    UPDATE_ARGUMENTS+=(--noplugins -y)
+    "$UPDATE_CMD" "$UPDATE_ACTION" --help | grep -q best && UPDATE_ARGUMENTS+=(--best)
+    "$UPDATE_CMD" "$UPDATE_ACTION" --help | grep -q allowerasing && UPDATE_ARGUMENTS+=(--allowerasing)
+    if "$UPDATE_CMD" --version | grep -q dnf5 && [ "$CHECK_ONLY" = "1" ]; then
+        UPDATE_ACTION=check-upgrade
+    fi
 else
-    YUM=(yum -y)
+    UPDATE_CMD=yum
+    UPDATE_ARGUMENTS=(-y)
 fi
 
 if ! [ -d "$DOM0_UPDATES_DIR" ]; then
@@ -100,15 +116,15 @@ rpm --root=$DOM0_UPDATES_DIR --rebuilddb
 
 if [ "$CLEAN" = "1" ]; then
     # shellcheck disable=SC2086
-    "${YUM[@]}" "${CLEAN_OPTS[@]}" clean all
+    $UPDATE_CMD clean all "${UPDATE_ARGUMENTS[@]}" "${CLEAN_OPTS[@]}"
     rm -f "$DOM0_UPDATES_DIR"/packages/*
     rm -rf "$DOM0_UPDATES_DIR"/var/cache/*
 fi
 
 # just check for updates, but don't download any package
 if [ ${#PKGLIST[@]} -eq 0 ] && [ "$CHECK_ONLY" = "1" ]; then
     # shellcheck disable=SC2086
-    UPDATES_FULL=$("${YUM[@]}" "${OPTS[@]}" check-update)
+    UPDATES_FULL=$($UPDATE_CMD $UPDATE_ACTION "${UPDATE_ARGUMENTS[@]}" "${OPTS[@]}")
     check_update_retcode=$?
     if [ "$check_update_retcode" -eq 1 ]; then
         # Exit here if yum have reported an error. Exit code 100 isn't an
@@ -129,10 +145,11 @@ if [ ${#PKGLIST[@]} -eq 0 ] && [ "$CHECK_ONLY" = "1" ]; then
 fi
 
 # now, we will download something
-YUM_COMMAND=(fakeroot "${YUM[@]}" "$YUM_ACTION" --downloadonly)
+UPDATE_COMMAND=(fakeroot $UPDATE_CMD $UPDATE_ACTION "${UPDATE_ARGUMENTS[@]}")
+"$UPDATE_CMD" "$UPDATE_ACTION" --help | grep -q downloadonly && UPDATE_COMMAND+=(--downloadonly)
 # check for --downloadonly option - if not supported (Debian), fallback to
 # yumdownloader
-if ! "${YUM[@]}" --help | grep -q downloadonly; then
+if ! "$UPDATE_CMD" --help | grep -q downloadonly && ! "$UPDATE_CMD" --version | grep -q dnf5; then
     if dpkg --compare-versions \
             "$(dpkg-query --show --showformat='${version}' rpm)" gt 4.14; then
         SIGNATURE_REGEX="^[A-Za-z0-9._+-/]{1,128}\.rpm: digests signatures OK$"
@@ -144,11 +161,11 @@ if ! "${YUM[@]}" --help | grep -q downloadonly; then
     if [ ! -e "$DOM0_UPDATES_DIR/etc/yum.conf" ]; then
         ln -nsf dnf/dnf.conf "$DOM0_UPDATES_DIR/etc/yum.conf"
     fi
-    if [ "$YUM_ACTION" = "install" ]; then
-        YUM_COMMAND=(yumdownloader "--destdir=$DOM0_UPDATES_DIR/packages" --resolve)
-    elif [ "$YUM_ACTION" = "upgrade" ]; then
+    if [ "$UPDATE_ACTION" = "install" ]; then
+        UPDATE_COMMAND=(yumdownloader "--destdir=$DOM0_UPDATES_DIR/packages" --resolve)
+    elif [ "$UPDATE_ACTION" = "upgrade" ]; then
         # shellcheck disable=SC2086
-        UPDATES_FULL=$("${YUM[@]}" "${OPTS[@]}" check-update "${PKGLIST[@]}")
+        UPDATES_FULL=$($UPDATE_CMD check-update "${UPDATE_ARGUMENTS[@]}" "${OPTS[@]}" "${PKGLIST[@]}")
         check_update_retcode=$?
         UPDATES_FULL=$(echo "$UPDATES_FULL" | grep -v "^Loaded plugins:\|^Last metadata\|^$")
         mapfile -t PKGLIST < <(echo "$UPDATES_FULL" | grep -v "^Obsoleting\|Could not" | cut -f 1 -d ' ')
@@ -157,19 +174,19 @@ if ! "${YUM[@]}" --help | grep -q downloadonly; then
             echo "No new updates available" >&2
             exit 0
         fi
-        YUM_COMMAND=(yumdownloader "--destdir=$DOM0_UPDATES_DIR/packages" --resolve)
-    elif [ "$YUM_ACTION" == "list" ] || [ "$YUM_ACTION" == "search" ]; then
+        UPDATE_COMMAND=(yumdownloader "--destdir=$DOM0_UPDATES_DIR/packages" --resolve)
+    elif [ "$UPDATE_ACTION" == "list" ] || [ "$UPDATE_ACTION" == "search" ]; then
         # those actions do not download any package, so lack of --downloadonly is irrelevant
-        YUM_COMMAND=("${YUM[@]}" -- "$YUM_ACTION")
-    elif [ "$YUM_ACTION" == "reinstall" ]; then
+        UPDATE_COMMAND=($UPDATE_CMD "${UPDATE_ARGUMENTS[@]}" -- "$UPDATE_ACTION")
+    elif [ "$UPDATE_ACTION" == "reinstall" ]; then
         # this is just approximation of 'reinstall' action...
         mapfile -t PKGLIST < <(rpm "--root=$DOM0_UPDATES_DIR" -q "${PKGLIST[@]}")
-        YUM_COMMAND=(yumdownloader "--destdir=$DOM0_UPDATES_DIR/packages" --resolve)
+        UPDATE_COMMAND=(yumdownloader "--destdir=$DOM0_UPDATES_DIR/packages" --resolve)
     else
         echo "ERROR: yum version installed in VM $(hostname) does not suppport --downloadonly option" >&2
-        echo "ERROR: only 'install' and 'upgrade' actions supported ($YUM_ACTION not)" >&2
+        echo "ERROR: only 'install' and 'upgrade' actions supported ($UPDATE_ACTION not)" >&2
         if [ "$GUI" = 1 ]; then
-            zenity --error --text="yum version too old for '$YUM_ACTION' action, see console for details"
+            zenity --error --text="yum version too old for '$UPDATE_ACTION' action, see console for details"
         fi
         exit 1
     fi
@@ -179,7 +196,7 @@ mkdir -p "$DOM0_UPDATES_DIR/packages"
 
 set -e
 
-"${YUM_COMMAND[@]}" "${OPTS[@]}" "${PKGLIST[@]}"
+"${UPDATE_COMMAND[@]}" "${OPTS[@]}" "${PKGLIST[@]}"
 
 find "$DOM0_UPDATES_DIR/var/cache" -name '*.rpm' -print0 2>/dev/null |\
     xargs -0 -r ln -f -t "$DOM0_UPDATES_DIR/packages/"