Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Added test of international character support | Demetri Obenour | 2014-05-08 | 2 | -29/+36 |
| | |||||
* | Added support for international characters in makefile dependency names | Demetri Obenour | 2014-05-08 | 1 | -2/+2 |
| | |||||
* | Allow paths with '{' '}' in depfiles | Maxim Kalaev | 2014-05-07 | 3 | -6/+10 |
| | |||||
* | compdb: check that inputs is not empty | Ben Boeckel | 2014-05-05 | 1 | -0/+2 |
| | |||||
* | Accept \r\n line endings in depfiles. | Nico Weber | 2014-05-01 | 3 | -8/+21 |
| | | | | Fixes #752. | ||||
* | Make manifest_parser_perftest build on Windows. | Nico Weber | 2014-04-27 | 1 | -1/+3 |
| | |||||
* | Merge pull request #714 from pcc/console-pool | Nico Weber | 2014-04-17 | 13 | -43/+196 |
|\ | | | | | Introduce the "console" pool | ||||
| * | Introduce the "console" pool | Peter Collingbourne | 2014-02-03 | 13 | -43/+196 |
| | | | | | | | | | | | | | | This is a pre-defined pool with a depth of 1. It has the special property that any task in the pool has direct access to the console. This can be useful for interactive tasks or long-running tasks which produce status updates on the console (such as test suites). | ||||
* | | Merge pull request #715 from nico/lazyenv | Nico Weber | 2014-04-17 | 1 | -4/+5 |
|\ \ | | | | | | | Allocate per-edge BindingEnvs lazily. | ||||
| * | | Allocate per-edge BindingEnvs lazily. | Nico Weber | 2014-02-11 | 1 | -4/+5 |
| |/ | | | | | | | | | | | | | In chrome, only 2000 of 22000 build edges have bindings. A BindingEnv is 64 bytes, so allocating these only when needed saves a bit over 1 MB of memory. Since env chains are shorter for lookups, builds also become a tiny bit faster. | ||||
* | | Manifest perftest: Try to make it build on Linux. | Nico Weber | 2014-04-17 | 1 | -0/+1 |
| | | |||||
* | | Manifest perftest: Pull manifest parsing into own function. | Nico Weber | 2014-04-17 | 1 | -17/+20 |
| | | |||||
* | | Manifest perftest: Also measure command evaluation time. | Nico Weber | 2014-04-17 | 1 | -5/+31 |
| | | |||||
* | | Add a simple manifest parsing perftest. | Nico Weber | 2014-04-17 | 1 | -0/+82 |
| | | |||||
* | | Rename parser_perftest to depfile_parser_perftest. | Nico Weber | 2014-04-16 | 1 | -0/+0 |
| | | |||||
* | | CLParser shouldn't read stderr | Scott Graham | 2014-04-14 | 2 | -1/+8 |
| | | |||||
* | | Merge pull request #699 from mostynb/avoid_useless_mkdir_failure | Nico Weber | 2014-04-07 | 1 | -0/+3 |
|\ \ | | | | | | | don't Fail if trying to mkdir when the dir already exists | ||||
| * | | don't Fail if trying to mkdir when the dir already exists | Mostyn Bramley-Moore | 2014-01-07 | 1 | -0/+3 |
| | | | |||||
* | | | Support both slashes on Windows when making output dirs | Scott Graham | 2014-04-03 | 2 | -6/+20 |
| |/ |/| | |||||
* | | Don’t unnecessarily escape backslashes in Win32 paths | Nicholas Hutchinson | 2014-01-08 | 2 | -1/+8 |
|/ | | | | Under ::CommandLineToArgvW() rules, the backslash character only gets special treatment if it’s immediately followed by a double quote. So, when checking to see if a string needs Win32 escaping, it’s sufficient to check for the presence of a double quote character. This allows paths like "foo\bar" to be recognised as “sensible” paths, which don’t require the full escaping. | ||||
* | Merge pull request #697 from nico/gclogs | Nico Weber | 2014-01-07 | 11 | -24/+176 |
|\ | | | | | Remove old outputs from build log and deps log during recompaction. | ||||
| * | Make BuildLogUser reference constant. | Nico Weber | 2014-01-07 | 8 | -13/+14 |
| | | |||||
| * | Add a comment. | Nico Weber | 2014-01-04 | 1 | -0/+7 |
| | | |||||
| * | Rename "IsDead" to "BuildLogUser". | Nico Weber | 2014-01-04 | 6 | -21/+30 |
| | | |||||
| * | Move duplicated code into a helper function. | Nico Weber | 2014-01-04 | 3 | -14/+20 |
| | | |||||
| * | Add a test for collection of dead log entries. | Nico Weber | 2014-01-04 | 1 | -0/+40 |
| | | |||||
| * | Add a test for collection of dead deps entries. | Nico Weber | 2014-01-03 | 2 | -1/+52 |
| | | |||||
| * | Fix DepsLogTest.Recompact by making sure outputs aren't garbag-collected. | Nico Weber | 2014-01-02 | 1 | -0/+10 |
| | | |||||
| * | Make tests compile (one fails atm). | Nico Weber | 2014-01-02 | 2 | -8/+11 |
| | | |||||
| * | Remove dead entries in .ninja_log and .ninja_deps while recompacting. | Nico Weber | 2014-01-02 | 4 | -8/+33 |
| | | | | | | | | | | | | | | | | | | | | For .ninja_deps, remove objects that have no in-edges or whose in-edges do not have a "deps" attribute. (This matches the behaviour of `-t deps`). BuildLog doesn't know about state, so let its recompact method take delegate that decides is a path is life or not, and implement it in NinjaMain. | ||||
* | | Merge pull request #690 from nickhutchinson/feature/escape-paths-properly | Nico Weber | 2014-01-06 | 6 | -12/+152 |
|\ \ | |/ |/| | More robust escaping of $in, $out paths | ||||
| * | More robust escaping of $in, $out paths | Nicholas Hutchinson | 2014-01-06 | 6 | -12/+152 |
| | | | | | | | | | | | | In summary: don’t escape if the path doesn’t contain problematic characters, otherwise: - Shell: enclose string in single quotes, escape embedded single quotes with the magic quote-backslash-quote sequence - Win32: Escape double quotes by doubling the number of consecutive backslashes that precede them (if any) and adding one more. Finally, double the number of trailing backslashes, and enclose the whole thing in double quotes. | ||||
* | | Fix additional range in character class. | Nico Weber | 2013-12-13 | 2 | -30/+43 |
|/ | | | | | | | | The range was added in 7ab6dcbdb6447861eefafc47fc3e10f3273cede2, but that change only tried to add ! to the character class. Fix by moving '-' to the end of the class. Fixes #694. | ||||
* | Add a test that shows that there is a single global namespace for rules. | Nico Weber | 2013-10-27 | 1 | -0/+15 |
| | |||||
* | Minor style fixes. No functionality change. | Nico Weber | 2013-10-19 | 4 | -8/+13 |
| | |||||
* | Merge pull request #665 from syntheticpp/localized-showinclude | Nico Weber | 2013-10-18 | 7 | -26/+41 |
|\ | | | | | add deps_prefix for localized /showIncludes' output parsing | ||||
| * | add deps_prefix for localized /showIncludes' output parsing | Peter Kümmel | 2013-10-18 | 7 | -26/+41 |
| | | |||||
* | | Fix compilation on VS2013 | Scott Graham | 2013-10-18 | 3 | -0/+4 |
|/ | |||||
* | mark this 1.4.0.git, update RELEASING | Evan Martin | 2013-09-12 | 1 | -1/+1 |
| | |||||
* | add -d keeprsp to preserve @rsp files on success on windows | Scott Graham | 2013-09-10 | 5 | -3/+12 |
| | |||||
* | Simplify. | Nico Weber | 2013-09-08 | 1 | -3/+2 |
| | |||||
* | Fix for build test for BuildWithDepsLogTest.RestatMissingDepfileDepslog | Richard Geary | 2013-09-07 | 1 | -1/+4 |
| | |||||
* | Make depslog v1->v3 message less scary. | Nico Weber | 2013-09-06 | 1 | -1/+1 |
| | | | | See the comment 5 lines up for details. | ||||
* | Simplify. | Nico Weber | 2013-09-02 | 1 | -18/+7 |
| | |||||
* | Delete a line I failed to delete in d7a46654a7be1. | Nico Weber | 2013-09-02 | 1 | -1/+0 |
| | |||||
* | Fix restat rebuild if deps are missing. | Maxim Kalaev | 2013-09-02 | 4 | -4/+126 |
| | | | | | | | | | | | | Fixes issue #603. Apparently, the problem was caused by my fix r "consider target dirty if depfile is missing" (bcc8ad1), which was not working correctly with restat rules cleaning nodes. Switching to deps only triggered an easily observable issue. Fix by setting a flag in edges with invalid deps, and not cleaning edges with that flag set. | ||||
* | Share more code between CleanNode() and RecomputeDirty(). | Maxim Kalaev | 2013-09-02 | 3 | -25/+23 |
| | | | | | | | | | Move a common loop into the new function RecomputeOutputsDirty(). Simplifies things a bit, and makes it harder for the restat path to have different behavior from the regular path. No dramatic behavior change (the restat path now also calls RestatIfNecessary()). | ||||
* | Check depslog timestamp in LoadDepsFromLog(), not in RecomputeOutputDirty(). | Maxim Kalaev | 2013-09-02 | 3 | -26/+23 |
| | | | | | | | | | | | | RecomputeOutputDirty() is called from two places: 1. RecomputeDirty(), which calls LoadDeps(). 2. CleanNode(), which always passes 0 for the deps mtime. So this is no behavior change in either case. deps_mtime was nonzero only in deps mode, and it got passed all over the place. This makes things simpler. | ||||
* | Simplify implementation of GetProcessorCount(). | Benedikt Meurer | 2013-09-01 | 1 | -22/+2 |
| | | | | | | | | | The current implementation is unnecessarily complex, because: - The BSD derived systems implement sysconf(_SC_NPROCESSORS_ONLN) in terms of sysctl({CTL_HW,HW_NCPU}). - get_nprocs() is a GNU extension, and glibc implements sysconf(_SC_NPROCESSORS_ONLN) in terms of get_nprocs(). | ||||
* | Fix an issue with more than 64k deps, spotted by maximuska. | Nico Weber | 2013-08-28 | 2 | -1/+34 |
| | | | | Also add a test for this case, which would have spotted the issue too. |