[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 30/32] pc-bios: ensure keymaps dependencies set vnc tests
From: |
Alex Bennée |
Subject: |
[PULL 30/32] pc-bios: ensure keymaps dependencies set vnc tests |
Date: |
Fri, 10 Jan 2025 13:17:52 +0000 |
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 <pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20250108121054.1126164-31-alex.bennee@linaro.org>
diff --git a/pc-bios/keymaps/meson.build b/pc-bios/keymaps/meson.build
index 0bd8ce0077..a79a09b276 100644
--- a/pc-bios/keymaps/meson.build
+++ b/pc-bios/keymaps/meson.build
@@ -39,19 +39,18 @@ else
native_qemu_keymap = qemu_keymap
endif
+keymap_targets = []
if native_qemu_keymap.found()
- t = []
foreach km, args: keymaps
# generate with qemu-kvm
- t += custom_target(km,
- build_by_default: true,
- output: km,
- command: [native_qemu_keymap, '-f', '@OUTPUT@',
args.split()],
- install: have_system,
- install_dir: qemu_datadir / 'keymaps')
+ keymap_targets += custom_target(km,
+ build_by_default: true,
+ output: km,
+ command: [native_qemu_keymap, '-f',
'@OUTPUT@', args.split()],
+ install: have_system,
+ install_dir: qemu_datadir / 'keymaps')
endforeach
-
- alias_target('update-keymaps', t)
+ alias_target('update-keymaps', keymap_targets)
else
install_data(keymaps.keys(), install_dir: qemu_datadir / 'keymaps')
endif
diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
index c5a70021c5..f75c1057a4 100644
--- a/tests/qtest/meson.build
+++ b/tests/qtest/meson.build
@@ -383,7 +383,7 @@ qtests = {
if vnc.found()
gvnc = dependency('gvnc-1.0', method: 'pkg-config', required: false)
if gvnc.found()
- qtests += {'vnc-display-test': [gvnc]}
+ qtests += {'vnc-display-test': [gvnc, keymap_targets]}
qtests_generic += [ 'vnc-display-test' ]
endif
endif
--
2.39.5
- [PULL 32/32] MAINTAINERS: Remove myself from reviewers, (continued)
- [PULL 32/32] MAINTAINERS: Remove myself from reviewers, Alex Bennée, 2025/01/10
- [PULL 16/32] tests/qtest: remove clock_steps from virtio tests, Alex Bennée, 2025/01/10
- [PULL 20/32] tests/functional: update tuxruntest to use uncompress utility, Alex Bennée, 2025/01/10
- [PULL 19/32] tests/functional: add zstd support to uncompress utility, Alex Bennée, 2025/01/10
- [PULL 17/32] system/qtest: properly feedback results of clock_[step|set], Alex Bennée, 2025/01/10
- [PULL 26/32] tests/lcitool: remove temp workaround for debian mips64el, Alex Bennée, 2025/01/10
- [PULL 21/32] tests/functional: remove unused kernel_command_line, Alex Bennée, 2025/01/10
- [PULL 29/32] tests/vm: allow interactive login as root, Alex Bennée, 2025/01/10
- [PULL 22/32] tests/functional: bail aarch64_virt tests early if missing TCG, Alex Bennée, 2025/01/10
- [PULL 28/32] tests/vm: partially un-tabify help output, Alex Bennée, 2025/01/10
- [PULL 30/32] pc-bios: ensure keymaps dependencies set vnc tests,
Alex Bennée <=
- Re: [PULL 00/32] testing updates for 10.0 (tuxrun, libvirt, dockerfiles, misc), Stefan Hajnoczi, 2025/01/10