Mail archive
alpine-aports

Re: [alpine-aports] [PATCH] testing/inetutils-syslogd: fix deinstall script

From: Natanael Copa <ncopa_at_alpinelinux.org>
Date: Fri, 13 Nov 2015 08:41:51 +0100

On Thu, 12 Nov 2015 20:24:17 +0000
Stuart Cardall <developer_at_it-offshore.co.uk> wrote:

> ---
> testing/inetutils-syslogd/inetutils-syslogd.post-deinstall | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/testing/inetutils-syslogd/inetutils-syslogd.post-deinstall b/testing/inetutils-syslogd/inetutils-syslogd.post-deinstall
> index 009b5b8..ffb8318 100644
> --- a/testing/inetutils-syslogd/inetutils-syslogd.post-deinstall
> +++ b/testing/inetutils-syslogd/inetutils-syslogd.post-deinstall
> _at_@ -1,7 +1,7 @@
> #!/bin/sh
> echo; rc-update del inetutils-syslogd boot; rc-update add syslog boot
> syslog_state=$(rc-service syslog status)
> -inetutils_state=$(rc-service inetutils-syslogd status)
> -echo "$inetutils_state: inetutils-syslogd"
> +inetutils_state=" * status: running: inetutils-syslogd (to stop: 'kill -9 $(pidof syslogd)')"
> +echo "$inetutils_state"
> echo -e "$syslog_state: syslog\n"
>

what is this? Why is this needed?

why kill -9 and not proper SIGTERM?

why does this package need this message echoed when we don't warn users
from shooting themselves in the foot by uninstalling services that are
running any other place?

-nc


---
Unsubscribe:  alpine-aports+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-aports+help_at_lists.alpinelinux.org
---
Received on Fri Nov 13 2015 - 08:41:51 GMT