Hello Andreas,
Hello Simon,
Andreas Metzler schrieb/wrote:
> Control: reopen -1
> Control: severity -1 serious
>
> On 2016-04-30 Thilo Six <deb...@xk2c.de> wrote:
>> -- <snip> --
>
>>> Conditions we want to track:
>>> * installation state of deb 'dnsmasq', better yet:
>>> deb 'dnsmasq' = removed (not purged)
>>> deb 'dnsmasq-base' = installed
>>> * better not use '/etc/dnsmasq.d/README' as flag for that as it could be
>>> erased
>>> by local admin for what ever reason (a README, is a README...)
>>>
>>> The attached patch has been successfully tested to meet this conditions.
>>> Additionally this patch fixes syntax highlighting of the init script in Vim.
>
>> minor patch update. I added a reference to #819856 in the comments.
> [...]
>> -test -e /etc/dnsmasq.d/README || exit 0
>> +# The following test ensures the dnsmasq service is not started, when the
>> +# package 'dnsmasq' is removed but not purged:
>> +# https://bugs.debian.org/819856
>> +test -d /usr/share/doc/dnsmasq || exit 0
> [...]
>
> Checking for /usr/share/doc/dnsmasq is the wrong fix, it violates
> policy 12.3.
> | Packages must not require the existence of any files in
> | `/usr/share/doc/' in order to function [1].
> | [...]
> | [1] The system administrator should be able to delete files in
> | `/usr/share/doc/' without causing any programs to break.
Thank you Andreas for the heads up. Obviously i have not been aware of this.
@Simon:
I tested this patch successfully to meet all currently known conditions:
-test -d /usr/share/doc/dnsmasq || exit 0
+test -e /usr/lib/resolvconf/dpkg-event.d/dnsmasq || exit 0
kind regards,
Thilo
