[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 03/18] meson: cleanup common-user/ build
From: |
Paolo Bonzini |
Subject: |
[PULL 03/18] meson: cleanup common-user/ build |
Date: |
Mon, 10 Jan 2022 15:40:19 +0100 |
It is not necessary to have a separate static_library just for common_user
files; using the one that already covers the rest of common_ss is enough
unless you need to reuse some source files between emulators and tests.
Just place common files for all user-mode emulators in common_ss,
similar to what is already done for softmmu_ss in full system emulators.
The only disadvantage is that the include_directories under bsd-user/include/
and linux-user/include/ are now enabled for all targets rather than only
user mode emulators. This however is not different from how include/sysemu/
is available when building user mode emulators.
Tested-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
common-user/meson.build | 2 +-
meson.build | 13 +------------
2 files changed, 2 insertions(+), 13 deletions(-)
diff --git a/common-user/meson.build b/common-user/meson.build
index 5cb42bc664..26212dda5c 100644
--- a/common-user/meson.build
+++ b/common-user/meson.build
@@ -1,6 +1,6 @@
common_user_inc += include_directories('host/' / host_arch)
-common_user_ss.add(files(
+user_ss.add(files(
'safe-syscall.S',
'safe-syscall-error.c',
))
diff --git a/meson.build b/meson.build
index de111d6fa1..5f0b6300b4 100644
--- a/meson.build
+++ b/meson.build
@@ -2399,7 +2399,6 @@ blockdev_ss = ss.source_set()
block_ss = ss.source_set()
chardev_ss = ss.source_set()
common_ss = ss.source_set()
-common_user_ss = ss.source_set()
crypto_ss = ss.source_set()
hwcore_ss = ss.source_set()
io_ss = ss.source_set()
@@ -2651,17 +2650,6 @@ subdir('common-user')
subdir('bsd-user')
subdir('linux-user')
-common_user_ss = common_user_ss.apply(config_all, strict: false)
-common_user = static_library('common-user',
- sources: common_user_ss.sources(),
- dependencies: common_user_ss.dependencies(),
- include_directories: common_user_inc,
- name_suffix: 'fa',
- build_by_default: false)
-common_user = declare_dependency(link_with: common_user)
-
-user_ss.add(common_user)
-
# needed for fuzzing binaries
subdir('tests/qtest/libqos')
subdir('tests/qtest/fuzz')
@@ -2879,6 +2867,7 @@ common_all = common_ss.apply(config_all, strict: false)
common_all = static_library('common',
build_by_default: false,
sources: common_all.sources() + genh,
+ include_directories: common_user_inc,
implicit_include_directories: false,
dependencies: common_all.dependencies(),
name_suffix: 'fa')
--
2.33.1
- [PULL v4 00/18] Build system and KVM changes for 2021-12-23, Paolo Bonzini, 2022/01/10
- [PULL 01/18] meson: reuse common_user_inc when building files specific to user-mode emulators, Paolo Bonzini, 2022/01/10
- [PULL 02/18] user: move common-user includes to a subdirectory of {bsd, linux}-user/, Paolo Bonzini, 2022/01/10
- [PULL 03/18] meson: cleanup common-user/ build,
Paolo Bonzini <=
- [PULL 04/18] block/file-posix: Simplify the XFS_IOC_DIOINFO handling, Paolo Bonzini, 2022/01/10
- [PULL 07/18] configure, makefile: remove traces of really old files, Paolo Bonzini, 2022/01/10
- [PULL 05/18] configure: simplify creation of plugin symbol list, Paolo Bonzini, 2022/01/10
- [PULL 06/18] configure: do not set bsd_user/linux_user early, Paolo Bonzini, 2022/01/10
- [PULL 09/18] configure: move non-command-line variables away from command-line parsing section, Paolo Bonzini, 2022/01/10
- [PULL 08/18] configure: parse --enable/--disable-strip automatically, flip default, Paolo Bonzini, 2022/01/10
- [PULL 10/18] meson: build contrib/ executables after generated headers, Paolo Bonzini, 2022/01/10
- [PULL 11/18] configure, meson: move config-poison.h to meson, Paolo Bonzini, 2022/01/10
- [PULL 12/18] meson: add comments in the target-specific flags section, Paolo Bonzini, 2022/01/10
- [PULL 13/18] KVM: use KVM_{GET|SET}_SREGS2 when supported., Paolo Bonzini, 2022/01/10