meson: remove config_targetos

config_targetos is now empty and can be removed; its use in sourcesets
that do not involve target-specific files can be replaced with an empty
dictionary.

In fact, at this point *all* sourcesets that do not involve
target-specific files are just glorified mutable arrays.  Enforce that
they never test for symbols in "when:" by computing the set of files
without "strict: false".

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
Paolo Bonzini 2023-08-30 11:52:43 +02:00
parent dc4954943d
commit 1220f5813a
7 changed files with 17 additions and 21 deletions

View File

@ -28,7 +28,7 @@ else
endif endif
endif endif
chardev_ss = chardev_ss.apply(config_targetos, strict: false) chardev_ss = chardev_ss.apply({})
system_ss.add(files( system_ss.add(files(
'char-hmp-cmds.c', 'char-hmp-cmds.c',

View File

@ -14,8 +14,8 @@ gdb_system_ss = ss.source_set()
gdb_user_ss.add(files('gdbstub.c', 'user.c')) gdb_user_ss.add(files('gdbstub.c', 'user.c'))
gdb_system_ss.add(files('gdbstub.c', 'system.c')) gdb_system_ss.add(files('gdbstub.c', 'system.c'))
gdb_user_ss = gdb_user_ss.apply(config_targetos, strict: false) gdb_user_ss = gdb_user_ss.apply({})
gdb_system_ss = gdb_system_ss.apply(config_targetos, strict: false) gdb_system_ss = gdb_system_ss.apply({})
libgdb_user = static_library('gdb_user', libgdb_user = static_library('gdb_user',
gdb_user_ss.sources() + genh, gdb_user_ss.sources() + genh,

View File

@ -2885,7 +2885,6 @@ endif
######################## ########################
minikconf = find_program('scripts/minikconf.py') minikconf = find_program('scripts/minikconf.py')
config_targetos = {}
config_all = {} config_all = {}
config_all_devices = {} config_all_devices = {}
@ -3084,7 +3083,6 @@ target_dirs = actual_target_dirs
# pseudo symbol replaces it. # pseudo symbol replaces it.
config_all += config_all_devices config_all += config_all_devices
config_all += config_targetos
config_all += config_all_disas config_all += config_all_disas
config_all += { config_all += {
'CONFIG_XEN': xen.found(), 'CONFIG_XEN': xen.found(),
@ -3413,7 +3411,7 @@ if enable_modules
modulecommon = declare_dependency(link_whole: libmodulecommon, compile_args: '-DBUILD_DSO') modulecommon = declare_dependency(link_whole: libmodulecommon, compile_args: '-DBUILD_DSO')
endif endif
qom_ss = qom_ss.apply(config_targetos, strict: false) qom_ss = qom_ss.apply({})
libqom = static_library('qom', qom_ss.sources() + genh, libqom = static_library('qom', qom_ss.sources() + genh,
dependencies: [qom_ss.dependencies()], dependencies: [qom_ss.dependencies()],
name_suffix: 'fa', name_suffix: 'fa',
@ -3428,10 +3426,10 @@ event_loop_base = static_library('event-loop-base',
event_loop_base = declare_dependency(link_whole: event_loop_base, event_loop_base = declare_dependency(link_whole: event_loop_base,
dependencies: [qom]) dependencies: [qom])
stub_ss = stub_ss.apply(config_all, strict: false) stub_ss = stub_ss.apply({})
util_ss.add_all(trace_ss) util_ss.add_all(trace_ss)
util_ss = util_ss.apply(config_all, strict: false) util_ss = util_ss.apply({})
libqemuutil = static_library('qemuutil', libqemuutil = static_library('qemuutil',
build_by_default: false, build_by_default: false,
sources: util_ss.sources() + stub_ss.sources() + genh, sources: util_ss.sources() + stub_ss.sources() + genh,
@ -3594,7 +3592,6 @@ foreach d, list : target_modules
foreach target : target_dirs foreach target : target_dirs
if target.endswith('-softmmu') if target.endswith('-softmmu')
config_target = config_target_mak[target] config_target = config_target_mak[target]
config_target += config_targetos
target_inc = [include_directories('target' / config_target['TARGET_BASE_ARCH'])] target_inc = [include_directories('target' / config_target['TARGET_BASE_ARCH'])]
c_args = ['-DNEED_CPU_H', c_args = ['-DNEED_CPU_H',
'-DCONFIG_TARGET="@0@-config-target.h"'.format(target), '-DCONFIG_TARGET="@0@-config-target.h"'.format(target),
@ -3655,7 +3652,7 @@ qemu_syms = custom_target('qemu.syms', output: 'qemu.syms',
capture: true, capture: true,
command: [undefsym, nm, '@INPUT@']) command: [undefsym, nm, '@INPUT@'])
authz_ss = authz_ss.apply(config_targetos, strict: false) authz_ss = authz_ss.apply({})
libauthz = static_library('authz', authz_ss.sources() + genh, libauthz = static_library('authz', authz_ss.sources() + genh,
dependencies: [authz_ss.dependencies()], dependencies: [authz_ss.dependencies()],
name_suffix: 'fa', name_suffix: 'fa',
@ -3664,7 +3661,7 @@ libauthz = static_library('authz', authz_ss.sources() + genh,
authz = declare_dependency(link_whole: libauthz, authz = declare_dependency(link_whole: libauthz,
dependencies: qom) dependencies: qom)
crypto_ss = crypto_ss.apply(config_targetos, strict: false) crypto_ss = crypto_ss.apply({})
libcrypto = static_library('crypto', crypto_ss.sources() + genh, libcrypto = static_library('crypto', crypto_ss.sources() + genh,
dependencies: [crypto_ss.dependencies()], dependencies: [crypto_ss.dependencies()],
name_suffix: 'fa', name_suffix: 'fa',
@ -3673,7 +3670,7 @@ libcrypto = static_library('crypto', crypto_ss.sources() + genh,
crypto = declare_dependency(link_whole: libcrypto, crypto = declare_dependency(link_whole: libcrypto,
dependencies: [authz, qom]) dependencies: [authz, qom])
io_ss = io_ss.apply(config_targetos, strict: false) io_ss = io_ss.apply({})
libio = static_library('io', io_ss.sources() + genh, libio = static_library('io', io_ss.sources() + genh,
dependencies: [io_ss.dependencies()], dependencies: [io_ss.dependencies()],
link_with: libqemuutil, link_with: libqemuutil,
@ -3689,7 +3686,7 @@ migration = declare_dependency(link_with: libmigration,
dependencies: [zlib, qom, io]) dependencies: [zlib, qom, io])
system_ss.add(migration) system_ss.add(migration)
block_ss = block_ss.apply(config_targetos, strict: false) block_ss = block_ss.apply({})
libblock = static_library('block', block_ss.sources() + genh, libblock = static_library('block', block_ss.sources() + genh,
dependencies: block_ss.dependencies(), dependencies: block_ss.dependencies(),
link_depends: block_syms, link_depends: block_syms,
@ -3700,7 +3697,7 @@ block = declare_dependency(link_whole: [libblock],
link_args: '@block.syms', link_args: '@block.syms',
dependencies: [crypto, io]) dependencies: [crypto, io])
blockdev_ss = blockdev_ss.apply(config_targetos, strict: false) blockdev_ss = blockdev_ss.apply({})
libblockdev = static_library('blockdev', blockdev_ss.sources() + genh, libblockdev = static_library('blockdev', blockdev_ss.sources() + genh,
dependencies: blockdev_ss.dependencies(), dependencies: blockdev_ss.dependencies(),
name_suffix: 'fa', name_suffix: 'fa',
@ -3709,7 +3706,7 @@ libblockdev = static_library('blockdev', blockdev_ss.sources() + genh,
blockdev = declare_dependency(link_whole: [libblockdev], blockdev = declare_dependency(link_whole: [libblockdev],
dependencies: [block, event_loop_base]) dependencies: [block, event_loop_base])
qmp_ss = qmp_ss.apply(config_targetos, strict: false) qmp_ss = qmp_ss.apply({})
libqmp = static_library('qmp', qmp_ss.sources() + genh, libqmp = static_library('qmp', qmp_ss.sources() + genh,
dependencies: qmp_ss.dependencies(), dependencies: qmp_ss.dependencies(),
name_suffix: 'fa', name_suffix: 'fa',
@ -3724,7 +3721,7 @@ libchardev = static_library('chardev', chardev_ss.sources() + genh,
chardev = declare_dependency(link_whole: libchardev) chardev = declare_dependency(link_whole: libchardev)
hwcore_ss = hwcore_ss.apply(config_targetos, strict: false) hwcore_ss = hwcore_ss.apply({})
libhwcore = static_library('hwcore', sources: hwcore_ss.sources() + genh, libhwcore = static_library('hwcore', sources: hwcore_ss.sources() + genh,
name_suffix: 'fa', name_suffix: 'fa',
build_by_default: false) build_by_default: false)
@ -3781,7 +3778,6 @@ foreach target : target_dirs
'-DCONFIG_DEVICES="@0@-config-devices.h"'.format(target)] '-DCONFIG_DEVICES="@0@-config-devices.h"'.format(target)]
link_args = emulator_link_args link_args = emulator_link_args
config_target += config_targetos
target_inc = [include_directories('target' / config_target['TARGET_BASE_ARCH'])] target_inc = [include_directories('target' / config_target['TARGET_BASE_ARCH'])]
if targetos == 'linux' if targetos == 'linux'
target_inc += include_directories('linux-headers', is_system: true) target_inc += include_directories('linux-headers', is_system: true)

View File

@ -87,7 +87,7 @@ else
endif endif
endif endif
qga_ss = qga_ss.apply(config_targetos, strict: false) qga_ss = qga_ss.apply({})
gen_tlb = [] gen_tlb = []
qga_libs = [] qga_libs = []

View File

@ -5,7 +5,7 @@ qsd_ss.add(blockdev, chardev, qmp, qom, qemuutil, gnutls)
subdir('qapi') subdir('qapi')
if have_tools if have_tools
qsd_ss = qsd_ss.apply(config_targetos, strict: false) qsd_ss = qsd_ss.apply({})
qsd = executable('qemu-storage-daemon', qsd = executable('qemu-storage-daemon',
qsd_ss.sources(), qsd_ss.sources(),
dependencies: qsd_ss.dependencies(), dependencies: qsd_ss.dependencies(),

View File

@ -22,7 +22,7 @@ if get_option('tcg_interpreter')
tcg_ss.add(files('tci.c')) tcg_ss.add(files('tci.c'))
endif endif
tcg_ss = tcg_ss.apply(config_targetos, strict: false) tcg_ss = tcg_ss.apply({})
libtcg_user = static_library('tcg_user', libtcg_user = static_library('tcg_user',
tcg_ss.sources() + genh, tcg_ss.sources() + genh,

View File

@ -310,7 +310,7 @@ qtests = {
'ivshmem-test': [rt, '../../contrib/ivshmem-server/ivshmem-server.c'], 'ivshmem-test': [rt, '../../contrib/ivshmem-server/ivshmem-server.c'],
'migration-test': migration_files, 'migration-test': migration_files,
'pxe-test': files('boot-sector.c'), 'pxe-test': files('boot-sector.c'),
'qos-test': [chardev, io, qos_test_ss.apply(config_targetos, strict: false).sources()], 'qos-test': [chardev, io, qos_test_ss.apply({}).sources()],
'tpm-crb-swtpm-test': [io, tpmemu_files], 'tpm-crb-swtpm-test': [io, tpmemu_files],
'tpm-crb-test': [io, tpmemu_files], 'tpm-crb-test': [io, tpmemu_files],
'tpm-tis-swtpm-test': [io, tpmemu_files, 'tpm-tis-util.c'], 'tpm-tis-swtpm-test': [io, tpmemu_files, 'tpm-tis-util.c'],