commit | 9bae73f36a4689eb46a60cc7d6863f6017e431f2 | [log] [tgz] |
---|---|---|
author | David 'Digit' Turner <digit@google.com> | Thu May 12 18:18:16 2022 +0200 |
committer | David 'Digit' Turner <digit@google.com> | Mon May 15 13:29:10 2023 +0200 |
tree | f7c45e9de4bd3d4c3ab03cf2d691509ba92245a0 | |
parent | eca1641ad0e6ff9540c9673589a480b481006f55 [diff] |
Fix .ninja parse time reported by `-d stats`. Because Parser::Load() is called recursively during Ninja manifest parsing, the call to METRIC_RECORD() in this function used to over-count the total parsing time (for example, by a factor of 2 for the Fuchsia build). This fixes the problem by introducing a new RECORD_METRIC_IF() macro, which only records anything if a given condition is true. This ensures that metric collection only starts and stops with the outer Parser::Load() call, and none of its recursive sub-calls. The effect on the output of `-d stats`is, for a Fuchsia build plan where `ninja -d stats nothing` takes a bit more than 5s: BEFORE: metric count avg (us) total (ms) .ninja parse 27304 372.6 10172.2 AFTER: metric count avg (us) total (ms) .ninja parse 1 4165297.0 4165.3 Note that |count| went to 1, since there is only one top-level Parser::Load() operation in this build. It would be more if dyndeps files were loaded, which does not happen in this build plan. Upstream-Pull-Request: https://github.com/ninja-build/ninja/pull/2292 Original-Change-Id: Id68874d56fa6899f77d0c1872cc8b7701fe8cf6e Change-Id: I8dc2d8a86822759b8a52072d955f8830097f6ab6
Ninja is a small build system with a focus on speed. https://ninja-build.org/
See the manual or doc/manual.asciidoc
included in the distribution for background and more details.
Binaries for Linux, Mac and Windows are available on GitHub. Run ./ninja -h
for Ninja help.
Installation is not necessary because the only required file is the resulting ninja binary. However, to enable features like Bash completion and Emacs and Vim editing modes, some files in misc/ must be copied to appropriate locations.
If you're interested in making changes to Ninja, read CONTRIBUTING.md first.
You can either build Ninja via the custom generator script written in Python or via CMake. For more details see the wiki.
./configure.py --bootstrap
This will generate the ninja
binary and a build.ninja
file you can now use to build Ninja with itself.
cmake -Bbuild-cmake cmake --build build-cmake
The ninja
binary will now be inside the build-cmake
directory (you can choose any other name you like).
To run the unit tests:
./build-cmake/ninja_test