On 2020-01-17 05:43, Rosen Penev wrote:
> $(which) must be executed. command -v is a shell builtin.
> 
> https://github.com/koalaman/shellcheck/wiki/SC2230
> 
> Signed-off-by: Rosen Penev <ros...@gmail.com>
> ---
>  package/base-files/files/lib/functions.sh | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/package/base-files/files/lib/functions.sh 
> b/package/base-files/files/lib/functions.sh
> index 0e94162a1a..a8a4734413 100755
> --- a/package/base-files/files/lib/functions.sh
> +++ b/package/base-files/files/lib/functions.sh
> @@ -176,7 +176,7 @@ default_prerm() {
>               ret=$?
>       fi
>  
> -     local shell="$(which bash)"
> +     local shell="command -v bash"
You dropped the $() there, that looks wrong to me.

- Felix

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to