X-Original-To: alpine-aports@lists.alpinelinux.org Received: from newmail.tetrasec.net (unknown [172.21.74.12]) by lists.alpinelinux.org (Postfix) with ESMTP id B990C5C4357 for ; Tue, 25 Oct 2016 20:05:04 +0000 (GMT) Received: from ncopa-desktop.copa.dup.pw (229.63.200.37.customer.cdi.no [37.200.63.229]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: n@tanael.org) by newmail.tetrasec.net (Postfix) with ESMTPSA id 179E45A083B; Tue, 25 Oct 2016 20:05:03 +0000 (GMT) Date: Tue, 25 Oct 2016 22:04:59 +0200 From: Natanael Copa To: Joe Borg Cc: alpine-aports@lists.alpinelinux.org Subject: Re: [alpine-aports] [PATCH] main/nginx: pre-install to create /run/nginx Message-ID: <20161025220459.6088a897@ncopa-desktop.copa.dup.pw> In-Reply-To: <20161025074054.6084-1-cyborg101010@gmail.com> References: <20161025074054.6084-1-cyborg101010@gmail.com> X-Mailer: Claws Mail 3.13.2 (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 On Tue, 25 Oct 2016 08:40:54 +0100 Joe Borg 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 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@lists.alpinelinux.org Help: alpine-aports+help@lists.alpinelinux.org ---