Re: [PATCH] nfsrootfs: disable warning message if bootargs root parameter have not been defined

["Antonin Godard" <antonin.godard@bootlin.com>]

Hi,

On Tue Jun 24, 2025 at 2:14 AM CEST, Jimmy Ho wrote:
> we have case that don't define root in bootargs,
> if [ ${bootparam_root} != "/dev/nfs" ] will output warning "/init.d/85-nfsrootfs: line 4: [: !=: unary operator expected"
> let variable expension result become string to solve this problem
>
> Signed-off-by: Jimmy Ho <jimmy.ho@sifive.com>
> Reviewed-by: Antonin Godard <antonin.godard@bootlin.com>

Please don't include Reviewed-by if the person has not explicitely given their
Reviewed-by in response to the previous version of the patch.

> ---
>  meta/recipes-core/initrdscripts/initramfs-framework/nfsrootfs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/initrdscripts/initramfs-framework/nfsrootfs b/meta/recipes-core/initrdscripts/initramfs-framework/nfsrootfs
> index e67ee4c25d..30555aef55 100644
> --- a/meta/recipes-core/initrdscripts/initramfs-framework/nfsrootfs
> +++ b/meta/recipes-core/initrdscripts/initramfs-framework/nfsrootfs
> @@ -1,7 +1,7 @@
>  #!/bin/sh
>  
>  nfsrootfs_enabled() {
> -	if [ ${bootparam_root} != "/dev/nfs" ] || [ -z ${bootparam_nfsroot} ]; then
> +	if [ "${bootparam_root}" != "/dev/nfs" ] || [ -z ${bootparam_nfsroot} ]; then
>  		return 1
>  	fi
>  	return 0

Please make sure to include vX (version number) in the patch subject when
sending a new version of the patch.

See https://docs.yoctoproject.org/contributor-guide/submit-changes.html.

Antonin

-- 
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com