[Pkg-varnish-devel] Bug#637110: invoke-rc.d varnish status should exit non-zero unless daemon running

[Daniel Barlow]

Package: varnish
Version: 3.0.0-4
Severity: normal

It Would Be Nice If /etc/init.d/varnish were to adopt the convention
(as seen in /etc/init.d/skeleton and many other packages) that "status"
returns a non-zero exit code if the daemon is not running.  This
would enable the presence of varnish to be tested programmatically,
e.g. by puppet and other configuration automation thingies

I believe the necessary change would be to replace the line

    status_of_proc -p "${PIDFILE}" "${DAEMON}" "${NAME}"

with

    status_of_proc -p "${PIDFILE}" "${DAEMON}" "${NAME}" && exit 0 || exit $?

- but I haven't actually tried it.




-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (600, 'unstable'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.30-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages varnish depends on:
ii  adduser                  3.112           add and remove users and groups
ii  gcc                      4:4.4.3-1       The GNU C compiler
ii  libc6                    2.13-10         Embedded GNU C Library: Shared lib
ii  libc6-dev [libc-dev]     2.13-10         Embedded GNU C Library: Developmen
ii  libedit2                 2.11-20080614-2 BSD editline and history libraries
ii  libncurses5              5.7+20100313-4  shared libraries for terminal hand
ii  libpcre3                 8.12-3          Perl 5 Compatible Regular Expressi
ii  libvarnishapi1           3.0.0-4         shared libraries for Varnish

varnish recommends no packages.

Versions of packages varnish suggests:
pn  varnish-doc                   <none>     (no description available)

-- no debconf information