We're assigning the `git` variable in three places:
- In "meson.build" to store the external Git executable.
- In "meson.build" to store the compiled Git executable.
- In "Documentation/meson.build" to store the external Git executable,
a second time.
The last case is only needed because we overwrite the original variable
with the built version. Rename the variable used for the built Git
executable so that we don't have to resolve the external Git executable
multiple times.
Signed-off-by: Patrick Steinhardt <ps@xxxxxx>
---
Documentation/meson.build | 1 -
meson.build | 6 +++---
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/Documentation/meson.build b/Documentation/meson.build
index 2a26fa8a5f..6438fa6792 100644
--- a/Documentation/meson.build
+++ b/Documentation/meson.build
@@ -283,7 +283,6 @@ elif docs_backend == 'asciidoctor'
]
endif
-git = find_program('git', required: false)
xmlto = find_program('xmlto')
cmd_lists = [
diff --git a/meson.build b/meson.build
index b5e8aca288..cf3f0d59d5 100644
--- a/meson.build
+++ b/meson.build
@@ -1605,13 +1605,13 @@ libgit_commonmain = declare_dependency(
bin_wrappers = [ ]
test_dependencies = [ ]
-git = executable('git',
+git_builtin = executable('git',
sources: builtin_sources + 'git.c',
dependencies: [libgit_commonmain],
install: true,
install_dir: get_option('libexecdir') / 'git-core',
)
-bin_wrappers += git
+bin_wrappers += git_builtin
test_dependencies += executable('git-daemon',
sources: 'daemon.c',
@@ -1702,7 +1702,7 @@ test_dependencies += executable('git-imap-send',
foreach alias : [ 'git-receive-pack', 'git-upload-archive', 'git-upload-pack' ]
bin_wrappers += executable(alias,
- objects: git.extract_all_objects(recursive: false),
+ objects: git_builtin.extract_all_objects(recursive: false),
dependencies: [libgit],
)
--
2.48.1.468.gbf5f394be8.dirty
