summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Delay actually opening log files until the first write, fix #1724Jan Niklas Hasse2020-05-234-47/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling DepsLog/BuildLog::OpenForWrite will now only save the file path. The actual opening of the file (moved to OpenForWriteIfNeeded) happens right before the first write attempt. This is needed so that the files aren't held open when the generator runs (i.e. RebuildManifest) as it may call tools like recompact which won't be able to open the file on Windows. The disadvantage is that now the error reporting happens at a later time and will be reported as a failed write, not a failed open.
* | | | | | Fix typosDimitris Apostolou2020-07-072-3/+3
| | | | | |
* | | | | | Merge pull request #1804 from ninja-build/nico-patch-2Nico Weber2020-06-211-1/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Include sys/select.h in subprocess-posix.cc
| * | | | | | Include sys/select.h in subprocess-posix.ccNico Weber2020-06-211-1/+6
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | pselect() is in sys/select.h in "newer" (2001) versions of posix, so add an include for it. While here, only include poll.h if USE_PPOLL is defined.
* | | | | | Include unistd.h in disk_interface.ccNico Weber2020-06-211-0/+2
|/ / / / / | | | | | | | | | | | | | | | stat() needs unistd.h in addition to sys/stat.h and sys/types.h per POSIX. At least one (hobby) OS does need unistd.h, so add an include for it.
* | | | | [clang-tidy] check empty instead of size (#1784)Rosen Penev2020-06-044-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | Found with readability-container-size-empty Signed-off-by: Rosen Penev <rosenp@gmail.com>
* | | | | [clang-tidy] remove pointless c_str() (#1785)Rosen Penev2020-06-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | Found with readability-redundant-string-cstr Signed-off-by: Rosen Penev <rosenp@gmail.com>
* | | | | bugfix: Process escaped colon in GCC depfiles. (#1774)zero91782020-05-203-24/+121
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Added ability to parse escaped colons in GCC Dep files enabling ninja to parse dep files of GCC 10 on Windows * Added generated depfile_parser.cc * Addressed formatting * Added extra tests with real world examples of paths produced by both GCC 10 and Clang and GCC pre 10. Adjusted one test so it doesn't fail * Adjusted regular expression to not match \: if the character following the : is either EOF or whitespace * Fixed typo in regex (should be 0x20 for space not 0xa) * Changed regular expression form using lookahead to instead matching a separate expression. This was needed as re2c pre version 1.17 is broken when using lookaheads. Also added tests for \: followed by whitespace * Addressed formatting * Forgot a missing std:: * Fixed formatting for spaces after , as well as respecting column width
* | | | MinGW now has _mktemp_s, use overload with size parameterJan Niklas Hasse2020-05-181-11/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MSVC also used the custom implementation, since the function isn't a macro and therefore #ifndef _mktemp_s didn't work as intended.
* | | | Null terminate the out2 string passed as filter2 to log.Restatyydevelop2020-04-151-1/+1
| | | |
* | | | Merge pull request #1637 from beevik/windows-create-process-fixJan Niklas Hasse2020-03-261-5/+13
|\ \ \ \ | | | | | | | | | | Improve error messages when ninja commands fail on Windows.
| * | | | Improve error messages when ninja commands fail on Windows.Brett Vickers2019-08-281-5/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a call to the Win32 API CreateProcessA fails, ninja now outputs the exact command string that caused it to fail. It also detects when the command contained leading whitespace and outputs a hint that lets the user know why the command failed.
* | | | | browse.py: use html.escape for python3Michael Hirsch, Ph.D2020-02-191-2/+5
| |_|_|/ |/| | | | | | | | | | | | | | | fixes #1741 fixes #1736
* | | | mark this 1.10.0.gitJan Niklas Hasse2020-01-271-1/+1
| | | |
* | | | restat: No need for loading build.ninjaJan Niklas Hasse2020-01-161-1/+1
| | | | | | | | | | | | | | | | | | | | This results in a huge speed up for large builds (e.g. Chromium). See #1718.
* | | | restat: Accept list of outputs to restat (none means all)Jan Niklas Hasse2020-01-164-8/+43
| | | | | | | | | | | | | | | | | | | | This will become handy when the generator only wants to restat the build.ninja entry in the log. See #1718.
* | | | Merge branch 'master' into clean_dead_outputsJan Niklas Hasse2020-01-0424-310/+868
|\ \ \ \
| * \ \ \ Merge pull request #1534 from mathstuf/remove-depslog-restrictionJan Niklas Hasse2020-01-0410-141/+378
| |\ \ \ \ | | | | | | | | | | | | manifest_parser: remove multi-output depslog restriction
| | * | | | depfile_parser_test: test buggy -MP behaviorBen Boeckel2019-11-203-2/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This ensures the current behavior of rejecting this case due to `x` being reused as an input.
| | * | | | depfile_parser: remove restriction on multiple outputsBen Boeckel2019-11-2010-138/+344
| | | | | |
| | * | | | build: add to deps log for each edge outputBen Boeckel2019-11-201-7/+9
| | | | | |
| * | | | | Merge pull request #1685 from jhasse/restatJan Niklas Hasse2020-01-047-24/+197
| |\ \ \ \ \ | | | | | | | | | | | | | | Add restat tool which recalculates all mtimes in the build log
| | * | | | | Ignore nonexistent .ninja_log/.ninja_deps for restat and recompactJan Niklas Hasse2019-12-237-26/+66
| | | | | | |
| | * | | | | Add restat tool which recalculates all mtimes in the build logJan Niklas Hasse2019-12-234-0/+133
| | | | | | |
| * | | | | | Fix logic-error in IsPathDead, see #1432Jan Niklas Hasse2019-12-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The conditional `(!n || !n->in_edge()) && ` was moved up. It now needs to be inversed because there's a `return false;`. See https://github.com/ninja-build/ninja/commit/3beebde51a2089ecb01820f1428efe0263deaeea#diff-78294872cbf9d32f4f972288561fa718R146 and https://github.com/ninja-build/ninja/pull/1432#discussion_r321827528
| * | | | | | Fix error handling for getcwdAndreas Kempf2019-12-271-2/+4
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Quoting from the Linux man page for errno, "The value in errno is significant only when the return value of the call indicated an error (i.e., -1 from most system calls; -1 or NULL from most library functions); a function that succeeds is allowed to change errno. The value of errno is never set to zero by any system call or library function." Successful calls to getcwd are allowed to set errno causing the compilation database not to be written. Spurious failures of this nature were observed on AIX. Adjust the error handling for getcwd so that errno is only checked if the call returned NULL.
| * | | | | build.cc: constify BuildStatusKonstantin Kharlamov2019-11-202-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>
| * | | | | build.cc: constify a map in BuildStatusKonstantin Kharlamov2019-11-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Modifying a key in C++ associative containers is UB. Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>
| * | | | | graph.cc: constify DependencyScanKonstantin Kharlamov2019-11-202-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>
| * | | | | build.cc: constify a few Plan functionsKonstantin Kharlamov2019-11-202-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>
| * | | | | build.cc: constify a map key in RealCommandRunnerKonstantin Kharlamov2019-11-201-3/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Modifying a key in C++ associative containers is UB. Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>
| * | | | Merge pull request #1680 from jonesmz/patch-1Jan Niklas Hasse2019-11-162-2/+2
| |\ \ \ \ | | | | | | | | | | | | Fix invalid preprocessor #if
| | * | | | Update graph_test.ccMichael Jones2019-11-141-1/+1
| | | | | |
| | * | | | Fix invalid preprocessor #ifMichael Jones2019-11-141-1/+1
| | | |/ / | | |/| |
| * | | | Fix minor typo of return valuexianglin10062019-11-151-2/+2
| |/ / / | | | | | | | | Return value of ManifestParser::ParseEdge shoule be boolean
| * | | Add Bionic tweak for getloadavgjoakim-noah2019-11-091-1/+1
| | | |
| * | | Merge pull request #1654 from BourgondAries/masterJan Niklas Hasse2019-10-302-0/+10
| |\ \ \ | | | | | | | | | | Ensure substring indexing never goes negative
| | * | | Ensure substring indexing never goes negativeKevin Robert Stravers2019-09-282-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With widths lower than 4, the ElideMiddle function would crash because its substring access would wrap around and attempt to access the max size_t value. This patch fixes that.
| * | | | Revert "1492 add column headers to .ninja_log"Jan Niklas Hasse2019-10-282-8/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | This reverts commit cfd0bd3007b291df505f8c45083453310142d681. See #1662.
| * | | Fix crash when using MSVC in debug modeJan Niklas Hasse2019-09-211-0/+4
| | | | | | | | | | | | | | | | | | | | Accessing inputs_[0] when it's empty results in an assert when running in debug. Avoid it by using data() if available.
| * | | Small constifications (#1647)Konstantin Kharlamov2019-09-195-15/+15
| |/ / | | | | | | | | | | | | | | | | | | * build: constify EdgeWanted() * build: constify a bit of CommandRunner * graph: constify functions of struct Edge Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>
| * | Enable build on IBM i platform (#1630)ThePrez2019-08-091-1/+5
| | | | | | | | | | | | * No perfstat for IBM i (OS400) AIX variant * Allow for future IBM i to identify as 'os400'
| * | Merge pull request #1604 from jessemckenna/correct-cpu-count-over-64Jan Niklas Hasse2019-08-081-3/+1
| |\ \ | | | | | | | | Enable ninja to use > 64 CPUs on Windows
| | * | Make GetProcessorCount() count processors across all processor groupsJesse McKenna2019-07-121-3/+1
| | | |
| * | | Merge pull request #1513 from nykula/masterJan Niklas Hasse2019-08-061-10/+4
| |\ \ \ | | | | | | | | | | Use st_mtim if st_mtime is macro, fix #1510
| | * | | Use st_mtim if st_mtime is macro, fix #1510makepost2018-12-241-10/+4
| | | | | | | | | | | | | | | | | | | | | | | | | In POSIX.1-2008, sys_stat has a st_mtim member and a st_mtime backward compatibility macro. Should help avoid hardcoding platform detection.
| * | | | fix building ninja_test on AIX 6.1ky0ko2019-08-051-0/+9
| | | | |
| * | | | Merge pull request #1508 from mqudsi/colored_failJan Niklas Hasse2019-08-022-2/+8
| |\ \ \ \ | | | | | | | | | | | | Emit "FAILED: " in red if terminal supports ANSI color output
| | * | | | Add braces to clarify conditional scopeMahmoud Al-Qudsi2019-02-041-2/+3
| | | | | |
| | * | | | Unset suports_color_ if SetConsoleMode fails on WIN32Mahmoud Al-Qudsi2018-12-101-1/+3
| | | | | |
generated by cgit v0.12 at 2025-04-05 00:06:05 (GMT)