Debian systemd fixes.

diff --git a/debian/changelog b/debian/changelog
index d75766b755874ae66707b2c7234a17bace02f369..1c52e985803031ce00bb91f92997b12ed9997105 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -6,7 +6,19 @@ dnsmasq (2.73-1) unstable; urgency=low
    * Add newline at the end of example config file. (LP: #1416895)
    * Make Debian package build reproducible. (closes: #777323)
 
- -- Simon Kelley <simon@thekelleys.org.uk>  Mon, 09 Feb 2015 11:50:42 +0000
+ -- Simon Kelley <simon@thekelleys.org.uk>  Wed, 11 Feb 2015 21:59:42 +0000
+
+dnsmasq (2.72-3) unstable; urgency=medium
+
+   * debian/systemd.service: switch from Type=dbus to Type=forking.
+     dnsmasq does not depend on dbus, but Type=dbus systemd services cannot
+     work without it. (Closes: #769486, #776530)
+     - debian/init: when called with systemd-exec argument, let dnsmasq
+       go into the background, so Type=forking can detect when it is ready
+   * Remove line containing only whitespace in debian/contol.
+     (closes: #777571)
+
+ -- Simon Kelley <simon@thekelleys.org.uk>  Wed, 11 Feb 2015 21:56:12 +0000
 
 dnsmasq (2.72-2) unstable; urgency=low
 

diff --git a/debian/control b/debian/control
index 6ed259d4e8abd2aa8811137ef8d3bbb9422437ab..f44a562fc0f7ed68db35d353a1630311dccb124a 100644 (file)
--- a/debian/control
+++ b/debian/control
@@ -42,5 +42,3 @@ Description: Utilities for manipulating DHCP leases
  Small utilities to query a DHCP server's lease database and
  remove leases from it. These programs are distributed with dnsmasq
  and may not work correctly with other DHCP servers.
-
- 

diff --git a/debian/init b/debian/init
index f4df909524b474e772205da020cbe026bf152e22..dc45d38bca59d6b9301fd1be874db8cb5d1ee80a 100644 (file)
--- a/debian/init
+++ b/debian/init
@@ -293,9 +293,7 @@ case "$1" in
            mkdir /var/run/dnsmasq || return 2
            chown dnsmasq:nogroup /var/run/dnsmasq || return 2
         fi
-# Enable DBus by default because we use DBus activation with systemd.
-       exec $DAEMON --keep-in-foreground --enable-dbus \
-           -x /var/run/dnsmasq/$NAME.pid \
+       exec $DAEMON -x /var/run/dnsmasq/$NAME.pid \
            ${MAILHOSTNAME:+ -m $MAILHOSTNAME} \
            ${MAILTARGET:+ -t $MAILTARGET} \
            ${DNSMASQ_USER:+ -u $DNSMASQ_USER} \

diff --git a/debian/systemd.service b/debian/systemd.service
index 65929dd40369fd1178ebce11e8def8c0f0325c80..aca30dc426479c0cb41652b87e64323e104c486b 100644 (file)
--- a/debian/systemd.service
+++ b/debian/systemd.service
@@ -2,8 +2,8 @@
 Description=dnsmasq - A lightweight DHCP and caching DNS server
 
 [Service]
-Type=dbus
-BusName=uk.org.thekelleys.dnsmasq
+Type=forking
+PIDFile=/var/run/dnsmasq/dnsmasq.pid
 
 # Test the config file and refuse starting if it is not valid.
 ExecStartPre=/usr/sbin/dnsmasq --test
@@ -11,11 +11,6 @@ ExecStartPre=/usr/sbin/dnsmasq --test
 # We run dnsmasq via the /etc/init.d/dnsmasq script which acts as a
 # wrapper picking up extra configuration files and then execs dnsmasq
 # itself, when called with the "systemd-exec" function.
-#
-# It also adds the command-line flags
-#    --keep-in-foreground --enable-dbus 
-# to enable DBus by default because we use DBus activation.
-#
 ExecStart=/etc/init.d/dnsmasq systemd-exec
 
 # The systemd-*-resolvconf functions configure (and deconfigure)