> Please use the actual path, not some non-existing abbreviation. will do so
> This is upgrade code from pre-jessie. This can be dropped. i'm fine with - but we need some kind of daemon restart > We should probably check for both paths here, for upgrade safety, i.e > whether /etc/resolv.conf points to the old *or* new path. ok, better safe than sorry.
in case the symlink points to the old location just remove the symlink - an alternative would be the empty file - so NetworkManger can recreate the link to the new location at the next start - there is no risk in this. this is the same as in the other p-scripts - but you are right, the intention should be documenteddiff --git a/debian/network-manager.preinst b/debian/network-manager.preinst index 9fff61d..b4c8a05 100644 --- a/debian/network-manager.preinst +++ b/debian/network-manager.preinst @@ -45,6 +45,11 @@ case "$1" in if dpkg --compare-versions "$2" lt "1.2.2-2"; then deb-systemd-helper purge NetworkManager-wait-online.service fi + + # Delete a symlink that points to /var/run + if [ "$(readlink /etc/resolv.conf)" = /var/run/NetworkManager/resolv.conf ] ; then + rm -f /etc/resolv.conf + fiWhy is that necessary? This appears to be a risky change with no clear explanation why it's needed.
Will rework the patch

