Date: Mon, 13 Jul 2015 08:46:56 +0200
From: Natanael Copa <ncopa@alpinelinux.org>
To: Stuart Cardall <developer@it-offshore.co.uk>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [alpine-aports] [PATCH] testing/lshell: add pre-install
 user:group
Message-ID: <20150713084656.26e359c1@ncopa-desktop.alpinelinux.org>
In-Reply-To: <1436310120-99079-1-git-send-email-developer@it-offshore.co.uk>
References: <1436310120-99079-1-git-send-email-developer@it-offshore.co.uk>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Tue,  7 Jul 2015 23:02:00 +0000
Stuart Cardall <developer@it-offshore.co.uk> wrote:

> ---
>  testing/lshell/lshell.pre-install | 5 +++++
>  1 file changed, 5 insertions(+)
>  create mode 100644 testing/lshell/lshell.pre-install
> 
> diff --git a/testing/lshell/lshell.pre-install b/testing/lshell/lshell.pre-install
> new file mode 100644
> index 0000000..b8b883e
> --- /dev/null
> +++ b/testing/lshell/lshell.pre-install
> @@ -0,0 +1,5 @@
> +#!/bin/sh
> +addgroup -S lshell 2>/dev/null
> +adduser -S -H -h /var/empty -G lshell -D -s /sbin/nologin lshell 2>/dev/null
> +exit 0
> +

You need bump pkgrel or builder will not pick it up.

You need add install="$pkgname.pre-install" or the script will not be
shipped with the package.

please send an updated patch.

-nc


---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---