[PATCH] Improve execl* functions
Federico Terraneo
fede.tft@hotmail.it
Fri Jan 31 09:53:35 GMT 2025
On 29/01/25 12:26, Corinna Vinschen wrote:
> Your patch looks good. At first I thought it's missing the ARG_MAX
> check now, but yeah, in fact that should be tested in execve.
>
> Can you please add the rational for this change to the commit message?
> Code comments like R. Diez suggested, would be helpful as well.
>
> Also, given this adds a compile time option, it would be helpful to
> add a matching option to configure.ac and add it to the README file.
>
Hi,
I tried adding the compile-time option and re-running autoreconf, but
I'm hitting an issue. Despite using GNU Autoconf 2.69 and GNU Automake
1.15.1 as stated in the newlib/HOWTO file, running autoreconf in the
newlib directory *before* making any changes to configure.ac generates
tons of spurious changes to Makefile.in, see attached patch 0002.
Any idea what I could be doing wrong, I'm on a Debian 12.
In any case, patch 0001 and 0003 look ok, maybe it's possible to just
skip patch 0002? Even though I'd be curious to know what's the issue
with autoreconf, maybe some of you encountered it already?
Best regards,
Federico Terraneo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Lift-256-arg-limit-in-execl-execle-and-execlp.patch
Type: text/x-patch
Size: 5644 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/newlib/attachments/20250131/a76aa60d/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Despite-using-autoconf-2.69-and-automake-1.15.1-auto.patch
Type: text/x-patch
Size: 136034 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/newlib/attachments/20250131/a76aa60d/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Add-enable-newlib-use-malloc-in-execl-option-to-conf.patch
Type: text/x-patch
Size: 4487 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/newlib/attachments/20250131/a76aa60d/attachment-0005.bin>
More information about the Newlib
mailing list