summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* 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
| | |
| * | Emit "FAILED: " in red if terminal supports ANSI color outputMahmoud Al-Qudsi2018-12-051-1/+4
| |/
* | compdb: Dump every rule without any arguments, fix #1377Jan Niklas Hasse2019-08-021-16/+28
| |
* | Improve const-correctness in compdb related methodsJan Niklas Hasse2019-08-023-21/+16
| |
* | Merge pull request #1517 from chagui/1492-add-headers-to-ninja_logJan Niklas Hasse2019-07-022-5/+8
|\ \ | | | | | | #1492 add column headers to .ninja_log
| * | 1492 add column headers to .ninja_logGuilhem Charles2019-01-132-5/+8
| |/
* | Follow GCC/Clang behavior wrt depfilesPeter Wu2019-06-083-46/+157
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The option is called "depfile = gcc" and should support depfiles created by GCC. GCC does not escape backslashes and GNU Make does not try to unescape it, so neither should Ninja try to "unescape" it. Only space (' ') and hash sign ('#') are specially treated by GCC/Clang. Note that while tabs are also treated specially by GCC, Clang does not, so do not special case it (why would someone use tabs in a filename?). Support for 2N trailing backslashes in a filename is a bit questionable, but is added to be as consistent as possible with GCC/Clang. See also https://github.com/llvm-mirror/clang/blob/44c160f916a1b080098b17b466b026aa07475ec2/lib/Frontend/DependencyFile.cpp#L316 https://github.com/gcc-mirror/gcc/blob/22a8377023d59cc01ab0a84a1df56d0e1336efa3/libcpp/mkdeps.c#L47 Fixes https://github.com/ninja-build/ninja/issues/1262
* | Fix UB "member call on null pointer of type 'DepsLog'", see #1248Jan Niklas Hasse2019-05-281-1/+1
| |
* | Resurrect the 'rules' tool.Nicolas Despres2019-05-093-0/+71
| | | | | | | | | | | | | | | | | | This tool is useful for writing shell completion script for tools expecting a rule name as argument. The tool was dropped by 34b46f28c. Fix #1024.
* | query: load dyndep files for queried edgesBrad King2019-04-181-0/+7
| |
* | graph: load dyndep filesBrad King2019-04-183-1/+16
| | | | | | | | | | | | Teach the `-t graph` tool to load dyndep files because they are part of the build graph. Issue a warning when the dyndep file cannot be loaded cleanly. This will help users visualize the complete build graph.
* | clean: remove outputs specified by dyndep filesBrad King2019-04-183-0/+73
| | | | | | | | | | | | Some outputs may not be known in the main build manifest and are instead discovered through a dyndep binding. Load dyndep files that are available during cleaning so that we can clean these outputs too.
* | clean: remove unnecessary Cleaner constructor variantBrad King2019-04-183-15/+1
| | | | | | | | | | | | | | | | `Cleaner` provides two constructors that are the same except that one constructs a "real" disk interface internally and the other takes a caller-provided disk interface. A real disk interface is already available at the only call site for the former constructor. Use it directly and drop the unnecessary constructor variant.
* | Teach builder to load dyndep files when they are readyBrad King2019-04-183-7/+908
| | | | | | | | | | | | | | | | After finishing an edge that produces a dyndep file, load the file and update the build graph structure. Recompute the dirty state of all its dependents and of newly reachable portions of the graph. Add edges to the build plan that are discovered to be wanted. Finally, schedule edges that are wanted and now ready to build.
* | Teach RecomputeDirty to load dyndep files that are readyBrad King2019-04-182-0/+219
| | | | | | | | | | | | | | | | | | | | | | The full readiness of a node that has a dyndep binding cannot be known until after the dyndep file is loaded. If a dyndep file is ready while constructing the build plan it can be loaded immediately so full information can be used to decide whether anything needs to be built. If a dyndep file is not ready while constructing the build plan then the edges naming it cannot be ready either because the dyndep file is one of their inputs. In this case we defer loading the dyndep file until the build plan is being executed.
* | Teach DependencyScan to load a dyndep fileBrad King2019-04-185-2/+354
| | | | | | | | | | Add a LoadDyndeps method to load a dyndep file and update the edges that name it in their dyndep binding.
* | Add a "dyndep" reserved binding to the manifest formatBrad King2019-04-185-3/+108
| | | | | | | | | | | | Allow rules or build statements to specify one of the build statement inputs in a "dyndep" binding. This will later be used to load dependency information from the specified file.
* | Add a parser for a new "dyndep" file formatBrad King2019-04-184-0/+819
| | | | | | | | | | | | | | | | Define a file format suitable for specifying dynamically-discovered dependency information for build edges. Design a format inspired by the build manifest format and using the same lexer. Start with a required format version specification followed by "build" statements that add implicit inputs and outputs to existing edges.
* | Explicitly avoid repeat deps loadingBrad King2019-04-183-7/+13
| | | | | | | | | | | | Track for each Edge whether depfile information has been loaded using an explicit flag. This will allow RecomputeDirty to be repeated for an edge without loading deps again.
* | Make a Builder optionally available to PlanBrad King2019-04-182-3/+11
| | | | | | | | | | | | | | In order to later support dynamic updates to the build plan while building, the Plan will need access to its Builder. Since this access will be needed only for specific features we can avoid updating all Plan constructions in the test suite by making this access optional.
* | Factor out output edge ready check from Plan::NodeFinishedBrad King2019-04-182-9/+17
| | | | | | | | | | Move the logic to a new Plan::EdgeMaybeReady method so it can be re-used elsewhere.
* | Factor out edge marking logic from Plan::AddSubTargetBrad King2019-04-182-3/+9
| | | | | | | | | | Move the logic to mark edges as wanted over to a Plan::EdgeWanted method so it can be re-used elsewhere later.
* | Teach FakeCommandRunner to support multiple active commandsBrad King2019-04-181-21/+47
| | | | | | | | | | | | | | | | Replace our single active edge pointer with a vector and add a parameter that tests can set to limit the number of concurrent edges. Set the default to 1 to preserve the current behavior. Specific tests will be able to override it later to simulate concurrent builds.
* | Allow EdgeFinished and NodeFinished to fail with errorsBrad King2019-04-183-37/+75
| | | | | | | | | | | | | | Add an 'err' string argument and return a boolean for success. Update call sites to pass an 'err' string argument and check the return value. This will be useful later for adding logic to these methods that may fail.
* | Assert precondition in BuildStatus::BuildEdgeStartedBrad King2019-04-181-0/+1
| | | | | | | | | | This method should be called only with edges that have not already been started.
* | Factor out a base class of ManifestParserBrad King2019-04-184-53/+104
| | | | | | | | | | | | Create a Parser base class that holds parser functionality not specific to the build manifest file format. This will allow it to be re-used for other parsers later.
* | ManifestParser: Fix typo {expectd => expected}Brad King2019-04-181-1/+1
| |
* | Remove trailing whitespace from all filesJan Niklas Hasse2019-04-173-4/+4
| |
* | make inline.sh more portablegoshhhy2019-04-161-1/+1
| |
* | Feature/add term env dumb to win32 (#1550)jhuels2019-04-161-5/+9
| | | | | | Add reading of TERM variable for win32 dumb terminals
* | Take CPU set limitations into account when calculating processor countJan Niklas Hasse2019-02-261-0/+9
| | | | | | | | Fixes #1278.
* | mark this 1.9.0.gitJan Niklas Hasse2019-01-301-1/+1
|/
* Restore depfile toleration of multiple output paths on distinct linesBrad King2018-11-1911-17/+120
| | | | | | | | | | | | | | | | Prior to introduction of depfile parser handling of multiple rules, ninja silently accepted a depfile of the form: out: in1 in2 in3 other: otherIn1 otherIn2 otherIn3 and incorrectly treated `other` and `otherIn*` as additional inputs to `out`. Now we prefer to reject this just as we already do for a depfile specifying multiple outputs on one line. However, this can break existing cases where such a depfile was silently tolerated. Add a `-w depfilemulti={err,warn}` option to control this behavior, and make it just a warning by default.
* Fix depfile parser handling of multiple rulesBrad King2018-11-193-33/+225
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we handle Makefile rules of the form: out: in1 in2 in3 and the form: out: in1 \ in2 \ in3 Teach the depfile parser to handle the additional form: out: in1 out: in2 out: in3 This is also valid Makefile syntax and is the depfile format generated by the Intel Compiler for Windows. Note that the `gcc -MP` option adds empty phony rules to the generated Makefile fragment: out: in1 in2 in3 in1: in2: in3: Previously we tolerated these because they were treated as inputs, which was accidentally correct. Instead we must now tolerate these by ignoring targets for which no dependencies are specified.
* Fix depfile parser test case line continuationBrad King2018-11-191-4/+4
| | | | | | | Escape newlines for line continuation in the SpecialChars test case. Otherwise the test does not cover valid Makefile syntax. The missing line continuation was tolerated by our parser but is never (not) produced by a real compiler.
* Re-arrange depfile parser token processing logicBrad King2018-11-192-22/+20
| | | | | Re-arrange the existing logic to support later addition of post-token code even for empty tokens.
* Re-generate depfile parser with re2cc 1.0.1Brad King2018-11-191-3/+2
|
* Fit --help output into 80 columns and move verbose up, fix #1500Jan Niklas Hasse2018-11-161-3/+3
|
* Merge pull request #1474 from mathstuf/win32-invalid-parameter-helpJan Niklas Hasse2018-11-163-3/+11
|\ | | | | Win32 invalid parameter help
| * subprocess-win32: add hint on ERROR_INVALID_PARAMETERBen Boeckel2018-11-091-0/+4
| | | | | | | | | | This is generally associated with the command line being too long. Give a hint to this case in the error message.
| * Win32Fatal: support a "hint" for the errorBen Boeckel2018-11-092-3/+7
| | | | | | | | | | The callsite might have extra context which is helpful for interpreting the error message.
* | Merge pull request #1462 from GoaLitiuM/win10colorJan Niklas Hasse2018-11-161-0/+12
|\ \ | | | | | | Enable ANSI escape sequences on Windows 10 terminals
| * | Enable ANSI escape sequences on Windows 10 terminalsGoaLitiuM2018-11-141-0/+12
| | |
* | | Merge pull request #1439 from Lekensteyn/threaded-browse-pyNico Weber2018-11-151-1/+7
|\ \ \ | | | | | | | | browse.py: fix delay with multiple connections
| * | | browse.py: fix delay with multiple connectionsPeter Wu2018-10-251-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When Firefox opens two TCP connections and sends a request over the second connection, it is not processed until the first one is closed. Use a threaded server to solve this issue. Tested with Python 2.7.15 and 3.6.5. (Also tested with 2.6, but that failed due to lack of argparse.) Link: https://bugs.python.org/issue31639