Re: bug: shell variable quoting in config.guess

[Dmitry V. Levin]

On Tue, Jan 19, 2021 at 08:49:34AM -0500, mlevins--- via Patches for the 
config.guess and config.sub scripts wrote:
> After commit 827c77253b396c07306927b2a4b794a3251c48eb in the GNU config
> repository, word splitting is no longer performed on the $sysctl shell
> variable in config.guess:
> 
>       sysctl="sysctl -n hw.machine_arch"
>       UNAME_MACHINE_ARCH=$( (uname -p 2>/dev/null || \
>           "/sbin/$sysctl" 2>/dev/null || \
>           "/usr/sbin/$sysctl" 2>/dev/null || \
>           echo unknown))
> 
> This would attempt to run an executable named "sysctl -n hw.machine_arch"
> (instead of an executable named "sysctl").
> 
> It could be fixed by removing the quotes again, or by spelling out the full
> command twice instead of using a shell variable at all. Two alternative
> patches are attached.

Applied one of them, thanks!


-- 
ldv