Andrew Clayton
Andrew Clayton
Hi @RobbieMcKinstry We are planning to do a 1.33.0 release real soon now(tm) and I'm planning on including this patch (attributed to yourself) once the language module has been updated...
Yes, unfortunately just too late for the s-o-b, but you are down as the `Author`. You did track down and suggest exactly what was missing!
> 2024/09/13 16:06:51 [alert] 1#1 bind(6, unix:/var/run/control.unit.sock.tmp) failed (13: Permission denied) You will need to specify `--control` to point to someplace your user has permissions to read/write. Probably the same...
> It seems that although unit is running as unit (uid:999), the log file is created as root: This is due to ``` root 1 0.0 0.0 10084 4324 ?...
> The first one was introduced at [caa0588](https://github.com/nginx/unit/commit/caa05887ff70bbd6338b129959a234ef56f1a287). And two ways to fix it. > > > ``` > diff -r 0e6d01d0c23b src/nxt_sprintf.c > --- a/src/nxt_sprintf.c Thu Sep 28 15:14:21...
On Mon, 09 Oct 2023 02:47:47 -0700 hongzhidao ***@***.***> wrote: > > I'd be inclined not to bother about this. > > Do you mean to keep them as is?...
A patch addressing the first warning has been [merged](https://github.com/nginx/unit/commit/9c8b9a46a42d47b29453eefc4c5dd325f59e46b4)
With current GCC (14..1.1) the second warning is gone.
@thresheek @andrey-zelenkov To recap, this would be to provide coverage of building Unit under musl libc + clang.
For ``` tools/openapi: update OpenAPI references ``` I would make the prefix `docs/openapi` as `docs/unit-openapi.yaml` is the source of truth.