From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnout Vandecappelle Date: Fri, 28 May 2021 18:05:54 +0200 Subject: [Buildroot] [PATCH] package/nginx: fix NGINX pidfile handling systemd In-Reply-To: <87lf7y274w.fsf@dell.be.48ers.dk> References: <20210521144149.35725-1-matthew.weber@collins.com> <20210526204856.GP3208066@scaer> <20210528113959.GB2788252@scaer> <4f98da29-4eb3-e5af-2139-a55d6defa12e@mind.be> <87lf7y274w.fsf@dell.be.48ers.dk> Message-ID: <3ebb96db-8047-2fa0-b18e-50f58c459f0e@mind.be> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On 28/05/2021 17:46, Peter Korsgaard wrote: >>>>>> "Arnout" == Arnout Vandecappelle writes: > > Hi, > > > Unfortunately, not so easy in practice. "daemon off" means a change in the > > configuration file, which is typically going to be modified by the user. Also, > > it's something that comes from nginx itself, so we'd have to patch the one that > > gets installed by nginx (and that one will probably be overwritten anyway in an > > overlay). > > Notice that you can override the configuration on the command line with > the -g option (-g 'daemon off;') I didn't realize that was possible! Then we can do it from the service file directly, and it becomes trivial. Matt? Regards, Arnout > like it is typically done in > Dockerfiles: > > https://github.com/nginxinc/docker-nginx/blob/master/Dockerfile-debian.template#L106 >