Skip to content

Commit 1b1e1b0

Browse files
committed
pc-bios: ensure keymaps dependencies set vnc tests
I was seeing failures on vnc-display-test on FreeBSD: make vm-build-freebsd V=1 TARGET_LIST=aarch64-softmmu BUILD_TARGET=check-qtest QEMU_LOCAL=1 DEBUG=1 Leads to: qemu-system-aarch64: -vnc none: could not read keymap file: 'en-us' Broken pipe ../src/tests/qtest/libqtest.c:196: kill_qemu() tried to terminate QEMU process but encountered exit status 1 (expected 0) which was as far as I could tell because we don't populate the $BLD/pc-bios/keymaps (although scripts/symlink-install-tree.py attempts to symlink qemu-bundle/usr/local/share/qemu/keymaps/ to that dir). Reviewed-by: Paolo Bonzini <[email protected]> Signed-off-by: Alex Bennée <[email protected]> Message-Id: <[email protected]>
1 parent 376c490 commit 1b1e1b0

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

pc-bios/keymaps/meson.build

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,19 +39,18 @@ else
3939
native_qemu_keymap = qemu_keymap
4040
endif
4141

42+
keymap_targets = []
4243
if native_qemu_keymap.found()
43-
t = []
4444
foreach km, args: keymaps
4545
# generate with qemu-kvm
46-
t += custom_target(km,
47-
build_by_default: true,
48-
output: km,
49-
command: [native_qemu_keymap, '-f', '@OUTPUT@', args.split()],
50-
install: have_system,
51-
install_dir: qemu_datadir / 'keymaps')
46+
keymap_targets += custom_target(km,
47+
build_by_default: true,
48+
output: km,
49+
command: [native_qemu_keymap, '-f', '@OUTPUT@', args.split()],
50+
install: have_system,
51+
install_dir: qemu_datadir / 'keymaps')
5252
endforeach
53-
54-
alias_target('update-keymaps', t)
53+
alias_target('update-keymaps', keymap_targets)
5554
else
5655
install_data(keymaps.keys(), install_dir: qemu_datadir / 'keymaps')
5756
endif

tests/qtest/meson.build

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -383,7 +383,7 @@ qtests = {
383383
if vnc.found()
384384
gvnc = dependency('gvnc-1.0', method: 'pkg-config', required: false)
385385
if gvnc.found()
386-
qtests += {'vnc-display-test': [gvnc]}
386+
qtests += {'vnc-display-test': [gvnc, keymap_targets]}
387387
qtests_generic += [ 'vnc-display-test' ]
388388
endif
389389
endif

0 commit comments

Comments
 (0)