main/nginx: pre-install to create /run/nginx v1 PROPOSED

Joe Borg: 1
 main/nginx: pre-install to create /run/nginx

 1 files changed, 4 insertions(+), 0 deletions(-)

[alpine-aports] [PATCH] main/nginx: pre-install to create /run/nginx Export this patch

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

Timo Teras <timo.teras@iki.fi>

7 years ago

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
---

Natanael Copa <ncopa@alpinelinux.org>

7 years ago

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
---

-- 
2.10.1



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---