X-Original-To: alpine-aports@mail.alpinelinux.org Delivered-To: alpine-aports@mail.alpinelinux.org Received: from mail.alpinelinux.org (dallas-a1.alpinelinux.org [127.0.0.1]) by mail.alpinelinux.org (Postfix) with ESMTP id E61D8DC010D for ; Fri, 13 Nov 2015 07:41:54 +0000 (UTC) Received: from newmail.tetrasec.net (unknown [74.117.189.116]) by mail.alpinelinux.org (Postfix) with ESMTP id C4EFCDC00CA for ; Fri, 13 Nov 2015 07:41:54 +0000 (UTC) Received: from ncopa-desktop.alpinelinux.org (unknown [79.160.13.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: n@tanael.org) by newmail.tetrasec.net (Postfix) with ESMTPSA id 5D2815A12EF; Fri, 13 Nov 2015 07:31:18 +0000 (GMT) Date: Fri, 13 Nov 2015 08:41:51 +0100 From: Natanael Copa To: Stuart Cardall Cc: alpine-aports@lists.alpinelinux.org Subject: Re: [alpine-aports] [PATCH] testing/inetutils-syslogd: fix deinstall script Message-ID: <20151113084151.555750e9@ncopa-desktop.alpinelinux.org> In-Reply-To: <1447359857-285-1-git-send-email-developer@it-offshore.co.uk> References: <1447359857-285-1-git-send-email-developer@it-offshore.co.uk> X-Mailer: Claws Mail 3.13.0 (GTK+ 2.24.28; x86_64-alpine-linux-musl) X-Mailinglist: alpine-aports Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV using ClamSMTP On Thu, 12 Nov 2015 20:24:17 +0000 Stuart Cardall 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 > @@ -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@lists.alpinelinux.org Help: alpine-aports+help@lists.alpinelinux.org ---