Re: PATCH: unify VAR!=cmd and other command execution

2023-08-28 Thread Theo Buehler
On Mon, Aug 28, 2023 at 09:18:40PM +0200, Marc Espie wrote: > On Mon, Aug 28, 2023 at 08:42:50PM +0200, Marc Espie wrote: > > Turns out the exec in cmd_exec.c has absolutely zero reason to be > > different from what engine does. > > > > This small patch moves a bit of code around, so that all exec

Re: PATCH: unify VAR!=cmd and other command execution

2023-08-28 Thread Marc Espie
On Mon, Aug 28, 2023 at 08:42:50PM +0200, Marc Espie wrote: > Turns out the exec in cmd_exec.c has absolutely zero reason to be > different from what engine does. > > This small patch moves a bit of code around, so that all execv() consumers > benefit from the same optimisation (namely, no extra s

PATCH: unify VAR!=cmd and other command execution

2023-08-28 Thread Marc Espie
Turns out the exec in cmd_exec.c has absolutely zero reason to be different from what engine does. This small patch moves a bit of code around, so that all execv() consumers benefit from the same optimisation (namely, no extra shell when not needed). The only visible change should be that now, VA