diff options
| author | Patrick Steinhardt <ps@pks.im> | 2025-02-26 09:22:13 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2025-02-26 09:09:35 -0800 |
| commit | ce9432889c4720345e4e1985db1b18db6add64ce (patch) | |
| tree | 7dad1a7b128b6f54fd39392a4b3045fee37242c5 | |
| parent | meson: fix OpenSSL fallback when not explicitly required (diff) | |
| download | git-ce9432889c4720345e4e1985db1b18db6add64ce.tar.gz git-ce9432889c4720345e4e1985db1b18db6add64ce.zip | |
meson: inline the static 'git' library
When setting up `libgit.a` we first create the static library itself,
and then declare it as part of a dependency such that compile arguments,
include directories and transitive dependencies get propagated to the
users of that library. As such, the static library isn't expected to be
used by anything but the declared dependency.
Inline the static library so that we don't even use a separate variable
for it. This avoids any kind of confusion that may arise and clarifies
how the library is supposed to be used.
Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | meson.build | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/meson.build b/meson.build index 83ed55c75c..a124101a73 100644 --- a/meson.build +++ b/meson.build @@ -1555,17 +1555,15 @@ libgit_version_library = static_library('git-version', include_directories: libgit_include_directories, ) -libgit_library = static_library('git', - sources: libgit_sources, - c_args: libgit_c_args, - link_with: libgit_version_library, - dependencies: libgit_dependencies, - include_directories: libgit_include_directories, -) - libgit = declare_dependency( + link_with: static_library('git', + sources: libgit_sources, + c_args: libgit_c_args, + link_with: libgit_version_library, + dependencies: libgit_dependencies, + include_directories: libgit_include_directories, + ), compile_args: libgit_c_args, - link_with: libgit_library, dependencies: libgit_dependencies, include_directories: libgit_include_directories, ) |