Re: [PATCH v2 1/5] tools/nolibc: add support for openat(2)

2025-03-11 Thread Louis Taylor
On Thu Mar 6, 2025 at 5:10 PM GMT, Thomas Weißschuh wrote: > On 2025-03-04 07:58:15+, Louis Taylor wrote: > > openat is useful to avoid needing to construct relative paths, so expose > > a wrapper for using it directly. > > > > Signed-off-by: Louis Taylor > > Looks good. I have some tiny nitp

Re: [PATCH v2 1/5] tools/nolibc: add support for openat(2)

2025-03-06 Thread Thomas Weißschuh
On 2025-03-04 07:58:15+, Louis Taylor wrote: > openat is useful to avoid needing to construct relative paths, so expose > a wrapper for using it directly. > > Signed-off-by: Louis Taylor Looks good. I have some tiny nitpicks inline, but if you prefer I can also pick it up as-is. Acked-by: T

Re: [PATCH v2 1/5] tools/nolibc: add support for openat(2)

2025-03-06 Thread Willy Tarreau
Hi Thomas, On Thu, Mar 06, 2025 at 06:22:45PM +0100, Thomas Weißschuh wrote: > Hi Willy, > > On 2025-03-04 09:11:16+0100, Willy Tarreau wrote: > > On Tue, Mar 04, 2025 at 07:58:15AM +, Louis Taylor wrote: > > > openat is useful to avoid needing to construct relative paths, so expose > > > a w

Re: [PATCH v2 1/5] tools/nolibc: add support for openat(2)

2025-03-06 Thread Thomas Weißschuh
Hi Willy, On 2025-03-04 09:11:16+0100, Willy Tarreau wrote: > On Tue, Mar 04, 2025 at 07:58:15AM +, Louis Taylor wrote: > > openat is useful to avoid needing to construct relative paths, so expose > > a wrapper for using it directly. > > Reviewed the whole series, no comments from me. Let's w

Re: [PATCH v2 1/5] tools/nolibc: add support for openat(2)

2025-03-04 Thread Willy Tarreau
Hi Louis, On Tue, Mar 04, 2025 at 07:58:15AM +, Louis Taylor wrote: > openat is useful to avoid needing to construct relative paths, so expose > a wrapper for using it directly. Reviewed the whole series, no comments from me. Let's wait for Thomas to double-check but for me it's OK: Acked-by