[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] iotests: Skip 233 if certtool not installed
From: |
Eric Blake |
Subject: |
[Qemu-devel] [PATCH] iotests: Skip 233 if certtool not installed |
Date: |
Tue, 20 Nov 2018 16:52:41 -0600 |
The use of TLS while building qemu is optional. While the
'certtool' binary should be available on every platform that
supports building against TLS, that does not imply that the
developer has installed it. Make the test gracefully skip
in that case.
Reported-by: Kevin Wolf <address@hidden>
Signed-off-by: Eric Blake <address@hidden>
---
On Fedora, libvirt requires libtls-utils to be present, but not qemu.
I'm fine if Kevin wants to pick this up in a pull request related
to iotests in general; if not, I'll do a pull request through my
NBD tree in time for -rc3.
tests/qemu-iotests/common.tls | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/qemu-iotests/common.tls b/tests/qemu-iotests/common.tls
index 39f17c1b999..eae81789bbc 100644
--- a/tests/qemu-iotests/common.tls
+++ b/tests/qemu-iotests/common.tls
@@ -31,6 +31,9 @@ tls_x509_cleanup()
tls_x509_init()
{
+ (certtool --help) >/dev/null 2>&1 || \
+ _notrun "certtool utility not found, skipping test"
+
mkdir -p "${tls_dir}"
# use a fixed key so we don't waste system entropy on
--
2.17.2
- [Qemu-devel] [PATCH] iotests: Skip 233 if certtool not installed,
Eric Blake <=
- Re: [Qemu-devel] [PATCH] iotests: Skip 233 if certtool not installed, Eric Blake, 2018/11/20
- Re: [Qemu-devel] [Qemu-block] [PATCH] iotests: Skip 233 if certtool not installed, John Snow, 2018/11/20
- Re: [Qemu-devel] [PATCH] iotests: Skip 233 if certtool not installed, Kevin Wolf, 2018/11/21
- Re: [Qemu-devel] [PATCH] iotests: Skip 233 if certtool not installed, Daniel P . Berrangé, 2018/11/21
- Re: [Qemu-devel] [PATCH] iotests: Skip 233 if certtool not installed, Wainer dos Santos Moschetta, 2018/11/21