putting this into one place is better for readability Signed-off-by: Fabian Grünbichler <f.gruenbich...@proxmox.com> --- debian/postinst | 35 +++++++++++++++++------------------ 1 file changed, 17 insertions(+), 18 deletions(-)
diff --git a/debian/postinst b/debian/postinst index ebce2536..f2ac0ab2 100755 --- a/debian/postinst +++ b/debian/postinst @@ -31,15 +31,6 @@ case "$1" in mkdir /etc/pve 2>/dev/null || true - if dpkg --compare-versions "$2" '<=' '5.1-47'; then - # remove cron update job, superseded by systemd timer - rm -f /etc/cron.d/pveupdate - - # remove old/unused init.d files - OLD_INITD_FILES="pvebanner pvenetcommit pve-manager pvedaemon pveproxy pvestatd spiceproxy" - for f in ${OLD_INITD_FILES}; do rm -f "/etc/init.d/$f"; done - fi - if test ! -e /var/lib/pve-manager/apl-info/download.proxmox.com; then mkdir -p /var/lib/pve-manager/apl-info cp /usr/share/doc/pve-manager/aplinfo.dat /var/lib/pve-manager/apl-info/download.proxmox.com @@ -78,15 +69,23 @@ case "$1" in done fi - if test -z "$2"; then - : # no old version, nothing to do - else - # "$2" is the most recently configured version - if dpkg --compare-versions "$2" '<=' '5.0-23'; then - # 5.0-23 temporarily reverted the removal of the startcom CA in - # ca-certificates; we've since switched to let's encrypt - update-ca-certificates >/dev/null 2>&1 - fi + if test -n "$2"; then + # "$2" is the most recently configured version + + if dpkg --compare-versions "$2" '<=' '5.0-23'; then + # 5.0-23 temporarily reverted the removal of the startcom CA in + # ca-certificates; we've since switched to let's encrypt + update-ca-certificates >/dev/null 2>&1 + fi + + if dpkg --compare-versions "$2" '<=' '5.1-47'; then + # remove cron update job, superseded by systemd timer + rm -f /etc/cron.d/pveupdate + + # remove old/unused init.d files + OLD_INITD_FILES="pvebanner pvenetcommit pve-manager pvedaemon pveproxy pvestatd spiceproxy" + for f in ${OLD_INITD_FILES}; do rm -f "/etc/init.d/$f"; done + fi fi ;; -- 2.14.2 _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel