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.

cu Andreas
-- 
`What a good friend you are to him, Dr. Maturin. His other friends are
so grateful to you.'
`I sew his ears on from time to time, sure'

Reply via email to