Mail archive
alpine-aports

Re: [alpine-aports] [PATCH] main/nginx: pre-install to create /run/nginx

From: Natanael Copa <ncopa_at_alpinelinux.org>
Date: Tue, 25 Oct 2016 22:04:59 +0200

On Tue, 25 Oct 2016 08:40:54 +0100
Joe Borg <cyborg101010_at_gmail.com> wrote:

> Nginx throws an error, the first time you run it, because the /run/nginx
> directory does not exist. Have added to the pre-install script to create this,
> if it doesn't already exist.
>
> http://bugs.alpinelinux.org/issues/6380
> ---
> main/nginx/nginx.pre-install | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/main/nginx/nginx.pre-install b/main/nginx/nginx.pre-install
> index ae5b685..e30aa2f 100644
> --- a/main/nginx/nginx.pre-install
> +++ b/main/nginx/nginx.pre-install
> _at_@ -5,4 +5,8 @@ addgroup -S nginx 2>/dev/null
> adduser -S -D -H -h /var/lib/nginx -s /sbin/nologin -G nginx -g nginx nginx 2>/dev/null
> addgroup nginx www-data 2>/dev/null
>
> +if [ ! -d /run/nginx ]; then
> + mkdir /run/nginx
> +fi
> +
> exit 0

I think it needs to be done from the startup script. /run is normally a
tmpfs so the dir will be gone next reboot. This means that we cannot
rely on something that creates the dir at package install time.


-nc


---
Unsubscribe:  alpine-aports+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-aports+help_at_lists.alpinelinux.org
---
Received on Tue Oct 25 2016 - 22:04:59 GMT