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

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@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. """

On Thu, 22 Jul 2021 at 14:52, Marek Behún marek.behun@nic.cz wrote:
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@nic.cz
test/py/tests/test_fs/test_squashfs/sqfs_common.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
Thanks, I hit that on rpi actually.

Em qui., 22 de jul. de 2021 às 17:52, Marek Behún marek.behun@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@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@gmail.com
Thanks!

On Thu, Jul 22, 2021 at 10:52:05PM +0200, Marek Behún wrote:
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@nic.cz Reviewed-by: Simon Glass sjg@chromium.org Reviewed-by: Joao Marcos Costa jmcosta944@gmail.com
Applied to u-boot/master, thanks!
participants (4)
-
João Marcos Costa
-
Marek Behún
-
Simon Glass
-
Tom Rini