Sender: =?UTF-8?Q?Timo_Ter=C3=A4s?= <timo.teras@gmail.com>
Date: Tue, 25 Oct 2016 13:16:12 +0300
From: Timo Teras <timo.teras@iki.fi>
To: Joe Borg <cyborg101010@gmail.com>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [alpine-aports] [PATCH] main/nginx: pre-install to create
 /run/nginx
Message-ID: <20161025131612.1c9cb9a4@vostro.util.wtbts.net>
In-Reply-To: <20161025074054.6084-1-cyborg101010@gmail.com>
References: <20161025074054.6084-1-cyborg101010@gmail.com>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Tue, 25 Oct 2016 08:40:54 +0100
Joe Borg <cyborg101010@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
> @@ -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


Not ok.

$ mount|grep /run
tmpfs on /run type tmpfs (rw,nodev,relatime,size=768268k,mode=755)

That is /run is a tmpfs mount and gets recreated on reboot. This should
be done in the init.d script.

Thanks.


---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---