meson: Drop the .fa library suffix
authorPaolo Bonzini <pbonzini@redhat.com>
Fri, 24 May 2024 08:56:55 +0000 (10:56 +0200)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 3 Jul 2024 16:41:26 +0000 (18:41 +0200)
commit4408155ac593644f04e22db0656d79a0e198be63
treed66a720e48be921fbf803c78b4ab3768ea778d93
parent7b1070a7e17cc65c3134350728c85e1d218c3578
meson: Drop the .fa library suffix

The non-standard .fa library suffix breaks the link source
de-duplication done by Meson so drop it.

The lack of link source de-duplication causes AddressSanitizer to
complain ODR violations, and makes GNU ld abort when combined with
clang's LTO.

Fortunately, the non-standard suffix is not necessary anymore for
two reasons.

First, the non-standard suffix was necessary for fork-fuzzing.
Meson wraps all standard-suffixed libraries with --start-group and
--end-group. This made a fork-fuzz.ld linker script wrapped as well and
broke builds. Commit d2e6f9272d33 ("fuzz: remove fork-fuzzing
scaffolding") dropped fork-fuzzing so we can now restore the standard
suffix.

Second, the libraries are not even built anymore, because it is
possible to just use the object files directly via extract_all_objects().

The occurences of the suffix were detected and removed by performing
a tree-wide search with 'fa' and .fa (note the quotes and dot).

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Message-ID: <20240524-xkb-v4-4-2de564e5c859@daynix.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
.gitlab-ci.d/buildtest-template.yml
.gitlab-ci.d/buildtest.yml
docs/devel/build-system.rst
gdbstub/meson.build
meson.build
stubs/blk-exp-close-all.c
tcg/meson.build
tests/Makefile.include
tests/qtest/libqos/meson.build