Bug#862693: unblock: postfix/3.1.4-5

[Niels Thykier <niels@thykier.net>]

Control: tags -1 moreinfo

Scott Kitterman:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package postfix
> 
> This upload fixes two significant bugs and makes it possible for a user to
> recover from a third without hand editing configuration files:
> 
> 1.  Postfix-cdb will no longer fail to work after upgrade from jessie due to
>     upgrade ordering issues.
> 
> 2.  Postfix should wait to start until the network and DNS are actually
>     available, so it won't fail to start on boot.
> 
> 3.  If an upgrade is performed from a not fully updated system and the
>     dynamic maps locations are incorred in dynamicmaps.cf, dpkg-reconfigure
>     postfix (and whatever postfix map types are installed) will fix it.
> 
> Note: The last one will be followed by a bug to release-notes to mention this
> (it seems way better than a release note explaining how to hand edit files).
> 
> unblock postfix/3.1.4-5
> 

Hi,

Thanks for working on fixing postfix.

One remark / question about the following snippet:

"""
-
+if dpkg --compare-versions $new lt 3.1.4-5~; then
+    # This turned out to be the wrong way to solve the problem.
+    rm -rf /etc/systemd/system/postfix.service.d
+fi
"""


Given the script only seemed to create
/etc/systemd/system/postfix.service.d/override.conf, shouldn't it only
remove that file (and possibly the directory iff it is empty)?  Also,
what if the admin has changed the file locally?

Thanks,
~Niels