]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test: drop the trailing whitespace from the QEMU version check
authorFrantisek Sumsal <frantisek@sumsal.cz>
Wed, 16 Dec 2020 15:33:50 +0000 (16:33 +0100)
committerFrantisek Sumsal <frantisek@sumsal.cz>
Wed, 16 Dec 2020 15:38:26 +0000 (16:38 +0100)
I suspect the original version of the regex was written on a system,
which prints both the QEMU version and the QEMU package version in the
--version output, like Fedora:

$ /bin/qemu-system-x86_64 --version
QEMU emulator version 4.2.1 (qemu-4.2.1-1.fc32)
Copyright (c) 2003-2019 Fabrice Bellard and the QEMU Project developers

However, Arch Linux prints only the QEMU version:
$ /bin/qemu-system-x86_64 --version
QEMU emulator version 5.2.0
Copyright (c) 2003-2020 Fabrice Bellard and the QEMU Project developers

This causes the awk regex to not match the version string, since there's
no whitespace after it, causing the version check to fail (as well as the
TEST-36-NUMAPOLICY) as well.

Follow-up for 43b49470d1f2808555c07f64cd0a1529b7ddd559.

test/test-functions

index 39f2baea01902f00d79da40865dca85ae33a8cf2..ce2c42b954b4f5bb703eefbe0c5de20e8066d405 100644 (file)
@@ -235,7 +235,7 @@ function qemu_min_version() {
     find_qemu_bin || return 2
 
     # get version from binary
-    qemu_ver=$($QEMU_BIN --version | awk '/^QEMU emulator version ([0-9]*\.[0-9]*\.[0-9]*) / {print $4}')
+    qemu_ver=$($QEMU_BIN --version | awk '/^QEMU emulator version ([0-9]*\.[0-9]*\.[0-9]*)/ {print $4}')
 
     # Check version string format
     echo "$qemu_ver" | grep -q '^[0-9]*\.[0-9]*\.[0-9]*$' || return 2