[PATCH] test/py: Improve check for mksquashfs version

João Marcos Costa jmcosta944 at gmail.com
Fri Jul 23 15:00:13 CEST 2021


Em qui., 22 de jul. de 2021 às 17:52, Marek Behún <marek.behun at nic.cz> escreveu:
>
> Some builds of squashfs-tools append version string with "-git" or
> similar. The float() conversion will fail in this case.
>
> Improve the code to only convert to float() the string before the '-'
> character.
>
> Signed-off-by: Marek Behún <marek.behun at nic.cz>
> ---
>  test/py/tests/test_fs/test_squashfs/sqfs_common.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/py/tests/test_fs/test_squashfs/sqfs_common.py b/test/py/tests/test_fs/test_squashfs/sqfs_common.py
> index 267c4b57d1..8b84c2cdca 100644
> --- a/test/py/tests/test_fs/test_squashfs/sqfs_common.py
> +++ b/test/py/tests/test_fs/test_squashfs/sqfs_common.py
> @@ -146,7 +146,7 @@ def get_mksquashfs_version():
>      out = subprocess.run(['mksquashfs -version'], shell=True, check=True,
>                           capture_output=True, text=True)
>      # 'out' is: mksquashfs version X (yyyy/mm/dd) ...
> -    return float(out.stdout.split()[2])
> +    return float(out.stdout.split()[2].split('-')[0])
>
>  def check_mksquashfs_version():
>      """ Checks if mksquashfs meets the required version. """
> --
> 2.31.1
>

Reviewed-by: Joao Marcos Costa <jmcosta944 at gmail.com>

Thanks!

-- 
Atenciosamente,
João Marcos Costa

www.linkedin.com/in/jmarcoscosta/
https://github.com/jmarcoscosta


More information about the U-Boot mailing list