Re: [PATCH v2 0/8] Some build system improvements

[Phillip Wood <phillip.wood123@gmail.com>]

On 16/03/2026 10:07, Patrick Steinhardt wrote:
> 
> Changes in v2:
>    - Turn array of precompiled headers into a simple string.
>    - Point out in the commit message that the precompiled header is
>      included implicitly.
>    - Link to v1: https://lore.kernel.org/r/20260310-b4-pks-build-infra-improvements-v1-0-ec75d0710d6a@pks.im

Thanks for expanding the commit message in patch 8, the range-diff looks 
good.

Phillip

> Thanks!
> 
> Patrick
> 
> [1]: https://lore.kernel.org/git/20250506-pks-contrib-spring-cleanup-v1-0-e6d5ddd79a72@pks.im/
> 
> ---
> Patrick Steinhardt (8):
>        Introduce new "tools/" directory
>        contrib: move "coccinelle/" directory into "tools/"
>        contrib: move "coverage-diff.sh" script into "tools/"
>        contrib: move "update-unicode.sh" script into "tools/"
>        builds: move build scripts into "tools/"
>        git-compat-util.h: move warning infra to prepare for PCHs
>        meson: compile compatibility sources separately
>        meson: precompile "git-compat-util.h"
> 
>   Makefile                                           | 76 ++++++++---------
>   ci/run-static-analysis.sh                          |  2 +-
>   config.mak.dev                                     |  2 +-
>   contrib/buildsystems/CMakeLists.txt                | 18 ++--
>   contrib/meson.build                                |  1 -
>   contrib/subtree/meson.build                        |  2 +-
>   git-compat-util.h                                  |  8 +-
>   meson.build                                        | 96 +++++++++++++---------
>   tools/README.md                                    |  7 ++
>   check-builtins.sh => tools/check-builtins.sh       |  0
>   {contrib => tools}/coccinelle/.gitignore           |  0
>   {contrib => tools}/coccinelle/README               |  2 +-
>   {contrib => tools}/coccinelle/array.cocci          |  0
>   {contrib => tools}/coccinelle/commit.cocci         |  0
>   .../coccinelle/config_fn_ctx.pending.cocci         |  0
>   {contrib => tools}/coccinelle/equals-null.cocci    |  0
>   {contrib => tools}/coccinelle/flex_alloc.cocci     |  0
>   {contrib => tools}/coccinelle/free.cocci           |  0
>   .../coccinelle/git_config_number.cocci             |  0
>   {contrib => tools}/coccinelle/hashmap.cocci        |  0
>   .../coccinelle/index-compatibility.cocci           |  0
>   {contrib => tools}/coccinelle/meson.build          |  0
>   {contrib => tools}/coccinelle/object_id.cocci      |  0
>   {contrib => tools}/coccinelle/preincr.cocci        |  0
>   {contrib => tools}/coccinelle/qsort.cocci          |  0
>   {contrib => tools}/coccinelle/refs.cocci           |  0
>   {contrib => tools}/coccinelle/spatchcache          |  6 +-
>   {contrib => tools}/coccinelle/strbuf.cocci         |  0
>   {contrib => tools}/coccinelle/swap.cocci           |  0
>   {contrib => tools}/coccinelle/tests/free.c         |  0
>   {contrib => tools}/coccinelle/tests/free.res       |  0
>   {contrib => tools}/coccinelle/the_repository.cocci |  0
>   {contrib => tools}/coccinelle/xcalloc.cocci        |  0
>   {contrib => tools}/coccinelle/xopen.cocci          |  0
>   .../coccinelle/xstrdup_or_null.cocci               |  0
>   {contrib => tools}/coccinelle/xstrncmpz.cocci      |  0
>   {contrib => tools}/coverage-diff.sh                |  0
>   detect-compiler => tools/detect-compiler           |  0
>   generate-cmdlist.sh => tools/generate-cmdlist.sh   |  0
>   .../generate-configlist.sh                         |  0
>   generate-hooklist.sh => tools/generate-hooklist.sh |  0
>   generate-perl.sh => tools/generate-perl.sh         |  0
>   generate-python.sh => tools/generate-python.sh     |  0
>   generate-script.sh => tools/generate-script.sh     |  0
>   tools/meson.build                                  |  1 +
>   tools/precompiled.h                                |  1 +
>   {contrib => tools}/update-unicode/.gitignore       |  0
>   {contrib => tools}/update-unicode/README           |  0
>   .../update-unicode/update_unicode.sh               |  0
>   49 files changed, 123 insertions(+), 99 deletions(-)
> 
> Range-diff versus v1:
> 
> 1:  224e28be31 = 1:  9d09d2c39a Introduce new "tools/" directory
> 2:  b217df51e5 = 2:  1b96bfe0f4 contrib: move "coccinelle/" directory into "tools/"
> 3:  e371b6c221 = 3:  ed6e90bd36 contrib: move "coverage-diff.sh" script into "tools/"
> 4:  3efeda9fa0 = 4:  48d8275ed1 contrib: move "update-unicode.sh" script into "tools/"
> 5:  ee074c1396 = 5:  daafeb3462 builds: move build scripts into "tools/"
> 6:  d30d4a3119 = 6:  947fc0f7b4 git-compat-util.h: move warning infra to prepare for PCHs
> 7:  ca118197a9 = 7:  6a2fb99aae meson: compile compatibility sources separately
> 8:  a865a8650b ! 8:  857b478896 meson: precompile "git-compat-util.h"
>      @@ Commit message
>           set up properly it means that the file won't need to be reprocessed.
>       
>           Set up such a precompiled header for "git-compat-util.h" and wire it up
>      -    via Meson. This leads to a significant speedup when performing full
>      -    builds:
>      +    via Meson. This causes Meson to implicitly include the precompiled
>      +    header in all compilation units. With GCC and Clang for example this is
>      +    done via the "-include" statement [1].
>      +
>      +    This leads to a significant speedup when performing full builds:
>       
>             Benchmark 1: ninja (rev = HEAD~)
>             Time (mean ± σ):     14.467 s ±  0.126 s    [User: 248.133 s, System: 31.298 s]
>      @@ Commit message
>               ninja (rev = HEAD) ran
>                 1.40 ± 0.02 times faster than ninja (rev = HEAD~)
>       
>      +    [1]: https://gcc.gnu.org/onlinedocs/gcc/Precompiled-Headers.html
>      +
>           Signed-off-by: Patrick Steinhardt <ps@pks.im>
>       
>        ## meson.build ##
>      @@ meson.build: libgit = declare_dependency(
>              c_args: libgit_c_args + [
>                '-DGIT_VERSION_H="' + version_def_h.full_path() + '"',
>              ],
>      -+      c_pch: [ 'tools/precompiled.h' ],
>      ++      c_pch: 'tools/precompiled.h',
>              dependencies: libgit_dependencies,
>              include_directories: libgit_include_directories,
>            ),
>      @@ meson.build: test_dependencies = [ ]
>        
>        git_builtin = executable('git',
>          sources: builtin_sources + 'git.c',
>      -+  c_pch: [ 'tools/precompiled.h' ],
>      ++  c_pch: 'tools/precompiled.h',
>          dependencies: [libgit_commonmain],
>          install: true,
>          install_dir: git_exec_path,
> 
> ---
> base-commit: af2c8a61818d773325ef2324dd135786a03ebca0
> change-id: 20260304-b4-pks-build-infra-improvements-cc4012c5364e
> 
>