Bug#724602: dnsmasq: Please enable systemd unit on install and fix it

[Simon Kelley]

On 11/05/14 15:11, Andreas Metzler wrote:
> On 2014-05-11 Michael Biebl <biebl at debian.org> wrote:
>> Am 11.05.2014 10:13, schrieb Andreas Metzler:
>>> On 2013-09-25 Julian Andres Klode <jak at debian.org> wrote:
> [...]
>>>> dnsmasq currently does not enable its systemd unit in postinst. It should
>>>> probably use dh-systemd.
> [...]
> 
>>> Isn't this severity serious? I have just gone from sysv to systemd
> 
>> Agreed, I bumped the severity accordingly.
> 
>>> (using apt-get --purge install systemd-sysv), and was switched from
>>> dnsmasq enabled before the upgrade to dnsmasq disabled, completely
>>> breaking DNS resolution.
> 
>> Simon, first of all, thanks for shipping a native .service file for
>> systemd. If you need help getting the service enabled (using
>> dh-systemd), please let us know. We are happy to assist you with this or
>> provide a patch.
> 
> Hello,
> 
> dnsmasq is packaged without debhelper, making this a little bit more
> work. However I have simply checked what dh_systemd /would/ do and
> have manually applied the changes to maintainerscripts and
> dependencies to come up with the the attached patch. Could you please
> doublecheck?
> 
> Thanks, cu Andreas
> 

This looks fine to me, and I'm happy to apply the patch, but I have no
way to definitively test it. (I'm working on a laptop with a
two-year-old Ubuntu installation and a sid chroot for package building.
My net connection is flaky 3G which won't allow me to pull enough stuff
to build a sid VM.)

I propose to upload a new version, if someone can test it ASAP and file
a new serious bug if I've botched it.

Cheers,


Simon.