Source: bind9
Version: 1:9.10.3.dfsg.P4-10
Severity: normal
Tags: patch

Hi,

while looking at [1] and [2], I noticed that the solution for unbound
might also apply to bind9. The bind9 package already splits off the
resolvconf bits, which is good.
I would suggest a small tweak to the service file though.
Instead of Requires=bind9.service I would use PartOf=bind9.service as I
think the semantics of PartOf better fit the desired behaviour better
[3]

Regards,
Michael


[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=826243
[2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=826241#45
[3] https://www.freedesktop.org/software/systemd/man/systemd.unit.html#PartOf=

-- System Information:
Debian Release: stretch/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (200, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.5.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
diff --git a/debian/bind9-resolvconf.service b/debian/bind9-resolvconf.service
index 3426c1f..2ac416a 100644
--- a/debian/bind9-resolvconf.service
+++ b/debian/bind9-resolvconf.service
@@ -1,7 +1,7 @@
 [Unit]
 Description=local BIND via resolvconf
 Documentation=man:named(8) man:resolvconf(8)
-Requires=bind9.service
+PartOf=bind9.service
 After=bind9.service
 ConditionFileIsExecutable=/sbin/resolvconf
 

Reply via email to