summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | | | | | | | | Ensure tests do not leave stale files in current directory.David 'Digit' Turner2023-10-064-42/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes a minor but annoying issue during development where some tests would leave stale files in the current directory. + Introduce new ScopedFilePath class to perform remove-on-scope-exit of a given file path. Fixes #1583
| * | | | | | | | | | Merge pull request #2338 from ninja-build/remove-appveyor-msvcJan Niklas Hasse2023-10-061-19/+0
| |\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / | |/| | | | | | | | | Remove MSVC build from AppVeyor
| | * | | | | | | | | Remove MSVC build from AppVeyorJan Niklas Hasse2023-10-061-19/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have GitHub Actions now
| * | | | | | | | | | Merge pull request #2335 from ↵Jan Niklas Hasse2023-10-063-4/+4
| |\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ninja-build/dependabot/github_actions/actions/upload-artifact-3 Bump actions/upload-artifact from 1 to 3
| | * | | | | | | | | Bump actions/upload-artifact from 1 to 3dependabot[bot]2023-10-063-4/+4
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 1 to 3. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](https://github.com/actions/upload-artifact/compare/v1...v3) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
| * | | | | | | | | Merge pull request #2334 from EwoutH/patch-2Jan Niklas Hasse2023-10-061-0/+7
| |\ \ \ \ \ \ \ \ \ | | |_|_|_|/ / / / / | |/| | | | | | | | Add Dependabot configuration for GitHub Actions updates
| | * | | | | | | | Add Dependabot configuration for GitHub Actions updatesEwout ter Hoeven2023-10-061-0/+7
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a Dependabot configuration that checks once a week if the GitHub Actions are still using the latest version. If not, it opens a PR to update them. It will open few PRs, since only major versions are specified (like v3), so only on a major release (like v4) it will update and open a PR. This way it helps actively keep GitHub Actions workflows up to date and secure, while minimizing maintenance burden. See https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot
| * | | | | | | | Merge pull request #2329 from digit-google/fix-mingw-compilationJan Niklas Hasse2023-09-251-7/+13
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix Mingw cross-compilation.
| | * | | | | | | | Fix Mingw cross-compilation.David 'Digit' Turner2023-09-251-7/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous pull request broke compilation with the Mingw toolchain, which does not provide <atlcore.h>. Moreover, this massive header was only used to expand a macro that generates a static volatile variable under the hood (which is not necessary here). Replace it with with a simpler and more portable code path to fix the build.
| * | | | | | | | | Merge pull request #2316 from gerioldman/TaskingRSPJan Niklas Hasse2023-09-241-2/+11
| |\ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | | | AddTasking RSP syntax to printCompdb functionality
| | * | | | | | | | Correct formattinggerioldman2023-09-061-4/+8
| | | | | | | | | |
| | * | | | | | | | AddTasking RSP syntax to printCompdb functionalitygerioldman2023-08-271-2/+7
| | | | | | | | | |
| * | | | | | | | | Merge pull request #2321 from brondani/fix-check-filename-lengthJan Niklas Hasse2023-09-194-7/+52
| |\ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | | | Fix check of filename length
| | * | | | | | | | Fix check of filename lengthDaniel Brondani2023-09-074-7/+52
| |/ / / / / / / /
| * | | | | | | | Merge pull request #2311 from alcroito/fix_browse_python_3_11Jan Niklas Hasse2023-08-231-2/+7
| |\ \ \ \ \ \ \ \ | | |_|_|_|/ / / / | |/| | | | | | | Fix browse to work with python 3.11
| | * | | | | | | Fix browse to work with Python 3.11+Alexandru Croitor2023-07-311-2/+7
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python sources should not contain null bytes, so don't pass the final string null terminator character to the python interpreter. Fixes: #2310
| * | | | | | | Merge pull request #2291 from digit-google/allow-duplicate-variable-expansionJan Niklas Hasse2023-05-292-6/+56
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Allow duplicate rule variable usage.
| | * | | | | | | Allow duplicate rule variable usage.David 'Digit' Turner2023-04-272-6/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes #1966 by removing the variable name from the lookups stack once the recursive lookup call has been performed. Without this, any previously expanded variable could no longer be referenced in the command, as Ninja would (incorrectly) complain about a cyclical dependency.
| * | | | | | | | Merge pull request #2293 from digit-google/fix-ppoll-auto-detectionJan Niklas Hasse2023-05-111-2/+10
| |\ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | CMakeLists: Fix ppoll() feature detection.
| | * | | | | | | CMakeLists: Fix ppoll() feature detection.David 'Digit' Turner2023-05-101-2/+10
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use check_cxx_symbol_exists() instead of check_symbol_exists() for the subtle reason explained in the comment added in this patch. Change-Id: Id949b8f2c7af39eeef3a848d3bdbc7639432f38a
| * | | | | | | Merge pull request #2281 from Hi-Angel/improve-ninja-modeJan Niklas Hasse2023-04-211-6/+31
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | emacs: improve ninja-mode
| | * | | | | | | ninja-mode.el: add support for indentationKonstantin Kharlamov2023-03-281-5/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We default `ninja-indent-offset` to `2` because Meson and CMake use `1` and `2` values accordingly, but `1` seems like too little, so use `2`. The correctness was tested in particular on two `build.ninja` files, one generated by Meson (1790 lines), and another by CMake (7777 lines). After setting `ninja-indent-offset` to the expected by the file value and re-indenting whole file the buffer was left unchanged, i.e. the calculated offset matched the ones already used.
| | * | | | | | | ninja-mode.el: fix variables highlightingKonstantin Kharlamov2023-03-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The code previously did not take into account that between the name and assignment may be any number of space including no space at all. It was also incorrectly highlighting in a code like foo = bar = buzz the `bar` as a variable, even though the `bar = buzz` is just a text that gets assigned to `foo`, i.e. `bar` is not a variable. Fix that.
| * | | | | | | | Merge pull request #2282 from arxanas/patch-4Jan Niklas Hasse2023-04-211-6/+5
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix formatting for `msvc` tool documentation
| | * | | | | | | | Fix formatting for `msvc` tool documentationWaleed Khan2023-03-311-6/+5
| | |/ / / / / / /
| * | | | | | | | Merge pull request #2290 from digit-google/use-ppollJan Niklas Hasse2023-04-211-0/+11
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Enable ppoll() usage when available.
| | * | | | | | | | Enable ppoll() usage when available.David 'Digit' Turner2023-04-211-0/+11
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch modifies the CMakeLists.txt file to probe for ppoll() on the target system, and define -DUSE_PPOLL=1 if it is available. This can be disabled by setting -DNINJA_FORCE_PSELECT=ON when invoking CMake. This matches the default behavior of the configure.py script (and its `--force-pselect` option). Note that there is no noticeable performance difference before build commands are launched, so this change is very hard to benchmark properly. Fix for https://github.com/ninja-build/ninja/issues/1821
| * | | | | | | | Increment ninja_log version (#2240)Junji Watanabe2023-04-182-25/+33
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Increment ninjg_log version * fix test * remove code for ninjg_log version 4 * line length * split error messages
| * | | | | | | Clean up Windows workflow and add an x86 build (#2251)Nikhil Dabas2023-02-071-44/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Clean up Windows workflow, add support for x86 * Sensible naming for artifacts No suffix for the x64 version, as that was the default already. * Remove x86 build from Windows workflow
| * | | | | | | Merge pull request #2245 from scivision/cimacJan Niklas Hasse2023-01-311-2/+2
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | CI:macos package with macos 12 for macos >= 10.15
| | * | | | | | | ci:macos package with macos 12 for macos >= 10.15scivision2023-01-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when this parameter was added, it was also for the last version that recently went end of life
| * | | | | | | | Merge pull request #2250 from tocic/fix/docs_typosJan Niklas Hasse2023-01-312-11/+11
| |\ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | Fix typos in docs
| | * | | | | | | Fix typos in docstocic2023-01-272-11/+11
| |/ / / / / / /
| * | | | | | | Merge pull request #2235 from orgads/no-auto-ptrJan Niklas Hasse2023-01-151-5/+1
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove auto_ptr
| | * | | | | | | Remove auto_ptrOrgad Shaneh2023-01-141-5/+1
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | C++11 is now a hard requirement.
| * | | | | | | Use python3 in all run linesNico Weber2023-01-105-11/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We already did this in some, this converts the rest. Also chmod +x on write_fake_manifests.py while here.
| * | | | | | | Merge pull request #2143 from orgads/flushJan Niklas Hasse2022-12-262-0/+2
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Flush output after each line
| | * | | | | | | Flush output after each lineOrgad Shaneh2022-06-072-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The output is not flushed automatically after \n, at least on Windows 10.
| * | | | | | | | Merge pull request #2225 from cristianadam/longpathJan Niklas Hasse2022-12-151-0/+1
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Add longPathAware manifest to enable long paths on Windows
| | * | | | | | | | Add longPathAware manifest to enable long paths on WindowsCristian Adam2022-12-131-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #1900
| * | | | | | | | | Add overridable parameter for name of Python binary (#2224)William Kent2022-12-131-1/+3
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add overridable parameter for name of Python binary On macOS, python is not an available command any more. python3 is. * Use option() as requested * Store variable in different way
| * | | | | | | | Merge pull request #2160 from zmodem/fix_FindFirstFileExANico Weber2022-12-092-1/+13
| |\ \ \ \ \ \ \ \ | | |_|_|_|/ / / / | |/| | | | | | | Handle ERROR_DIRECTORY when calling FindFirstFileExA
| | * | | | | | | Handle ERROR_DIRECTORY when calling FindFirstFileExAHans Wennborg2022-07-122-1/+13
| | | |_|_|_|_|/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a directory referenced by .ninja_deps has changed to a regular file since the last build, FindFirstFileExA will return ERROR_DIRECTORY. Fixes #2159
| * | | | | | | Merge pull request #2208 from ThadHouse/winarmciJan Niklas Hasse2022-11-091-0/+40
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Add build job for Windows Arm64
| | * | | | | | | Add build job for Windows Arm64Thad House2022-10-311-0/+40
| |/ / / / / / /
| * | | | | | | Merge pull request #2020 from randomascii/steady_clock_now_goodJan Niklas Hasse2022-10-312-49/+36
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Make ScopedMetric portable, accurate, and efficient
| | * | | | | | | Revert configure.py changeBruce Dawson2022-10-251-0/+0
| | | | | | | | |
| | * | | | | | | Remove bad merge text from .md fileBruce Dawson2022-10-241-3/+0
| | | | | | | | |
| | * | | | | | | Merge branch 'master' into steady_clock_now_goodBruce Dawson2022-10-2431-223/+981
| | |\ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | |
| * | | | | | | | Merge pull request #2086 from randomascii/compiler_detectJan Niklas Hasse2022-10-231-0/+9
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Detect whether VC++ compiler can be found in configure.py