Skip to content

Commit cc1838c

Browse files
committed
storage-daemon: Add missing build dependency to the vhost-user-blk-test
vhost-user-blk-test needs the qemu-storage-daemon, otherwise it currently hangs. So make sure that we build the daemon before running the tests. Message-Id: <[email protected]> Tested-by: Alexander Bulekov <[email protected]> Signed-off-by: Thomas Huth <[email protected]>
1 parent a1f0f36 commit cc1838c

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

storage-daemon/meson.build

+4-4
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@ subdir('qapi')
66

77
if have_tools
88
qsd_ss = qsd_ss.apply(config_host, strict: false)
9-
executable('qemu-storage-daemon',
10-
qsd_ss.sources(),
11-
dependencies: qsd_ss.dependencies(),
12-
install: true)
9+
qsd = executable('qemu-storage-daemon',
10+
qsd_ss.sources(),
11+
dependencies: qsd_ss.dependencies(),
12+
install: true)
1313
endif

tests/qtest/meson.build

+5-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,11 @@ foreach dir : target_dirs
276276
endif
277277
qtest_env.set('G_TEST_DBUS_DAEMON', meson.source_root() / 'tests/dbus-vmstate-daemon.sh')
278278
qtest_env.set('QTEST_QEMU_BINARY', './qemu-system-' + target_base)
279-
qtest_env.set('QTEST_QEMU_STORAGE_DAEMON_BINARY', './storage-daemon/qemu-storage-daemon')
280-
279+
if have_tools and have_vhost_user_blk_server
280+
qtest_env.set('QTEST_QEMU_STORAGE_DAEMON_BINARY', './storage-daemon/qemu-storage-daemon')
281+
test_deps += [qsd]
282+
endif
283+
281284
foreach test : target_qtests
282285
# Executables are shared across targets, declare them only the first time we
283286
# encounter them

0 commit comments

Comments
 (0)