[pve-devel] [PATCH manager 1/1] d/postinst: remove old pvemailforward migration logic
Lukas Wagner
l.wagner at proxmox.com
Wed Jun 25 14:20:11 CEST 2025
Before the separate proxmox-mail-forward helper was split out into its
own package, pve-manager shipped its own pvemailforward script. Arguably
this is the reason why the code for migrating the path to the helper in
/root/.forward is part of pve-manager's d/postinst script, not
proxmox-mail-forward's.
The pvemailforward -> proxmox-mail-forward migration happened in
pve-manager 7.2-12. Since we don't support skipping major versions (e.g.
7 -> 9), we should be able to drop the migration code for PVE 9.
This also means that proxmox-mail-forward is now fully in charge of the
contents of /root/.forward.
Signed-off-by: Lukas Wagner <l.wagner at proxmox.com>
---
debian/postinst | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/debian/postinst b/debian/postinst
index aba39904..6b706951 100755
--- a/debian/postinst
+++ b/debian/postinst
@@ -155,16 +155,6 @@ case "$1" in
pveam update || true
fi
- # Always try to clean old entry, even when proxmox-mail-forward entry is already present.
- # This ensures it will still be cleaned after an upgrade following a downgrade.
- if test -f /root/.forward; then
- sed -i '\!|/usr/bin/pvemailforward!d' /root/.forward
- fi
-
- if ! test -f /root/.forward || ! grep -q '|/usr/bin/proxmox-mail-forward' /root/.forward; then
- echo '|/usr/bin/proxmox-mail-forward' >>/root/.forward
- fi
-
systemctl --system daemon-reload >/dev/null || true
# same as dh_systemd_enable (code copied)
--
2.39.5
More information about the pve-devel
mailing list