Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Recompact the deps log when it gets too big. | Nico Weber | 2013-05-01 | 2 | -7/+22 |
| | | | | | Now that Recompact() keeps all data structures intact, it can just be called at the beginning of a build and the build will still work. | ||||
* | Keep a DepsLog's data valid after a call to DepsLog::Recompact(). | Nico Weber | 2013-05-01 | 2 | -7/+35 |
| | | | | | | Previously, a DepsLog would become invalid after Recompact() was called, due to Recompact() making all node ids refer to a temporary DepsLog object constructed in Recompact(). | ||||
* | Let DepsLog::RecordDeps() update its in-memory representation. | Nico Weber | 2013-04-30 | 2 | -8/+21 |
| | | | | | | This is a behavior change, but it should be safe: Graph only queries the deps log at startup, before running any command. Nothing else currently queries the deps log. | ||||
* | Move updating DepsLog's deps_ array into its own function. | Nico Weber | 2013-04-30 | 2 | -6/+15 |
| | | | | No functionality change. | ||||
* | Move some DepsLog::Deps initialization code into its constructor. | Nico Weber | 2013-04-30 | 2 | -5/+3 |
| | | | | No functionality change. | ||||
* | Merge pull request #559 from nico/9_9 | Nico Weber | 2013-04-29 | 1 | -1/+1 |
|\ | | | | | Fix Windows build more. | ||||
| * | Fix Windows build more. | Nico Weber | 2013-04-29 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #558 from nico/buildfix | Evan Martin | 2013-04-29 | 5 | -14/+27 |
|\ | | | | | Introduce a Truncate() function that works on POSIX and Windows. | ||||
| * | Introduce a Truncate() function that works on POSIX and Windows. | Nico Weber | 2013-04-29 | 5 | -14/+27 |
| | | | | | | | | Hopefully fixes the build on Windows. | ||||
* | | Merge pull request #556 from nico/unlink | Evan Martin | 2013-04-29 | 1 | -0/+5 |
|\ \ | | | | | | | Make sure that recompaction writes a pristine new depslog. | ||||
| * | | Make sure that recompaction writes a pristine new depslog. | Nico Weber | 2013-04-29 | 1 | -0/+5 |
| |/ | | | | | | | | | | | | | | | | | | | | | | | ...even if a prior run of DepsLog::Recompact() exited without cleaning up, for example due to a crash or because someone added a `return true;` in the middle of the function while debugging. Or because someone hits ctrl-c during deps log recompaction. No test, because I can't think of a way to trigger this scenario programmatically. Part of issue #554. | ||||
* | | Merge pull request #555 from nico/depscompactcrash | Evan Martin | 2013-04-29 | 2 | -0/+8 |
|\ \ | |/ |/| | Don't crash during deps log recompaction if there's more than one deps entry. | ||||
| * | Don't crash during deps log recompaction if there's more than one deps entry. | Nico Weber | 2013-04-29 | 2 | -0/+8 |
|/ | | | | Part of issue #554. | ||||
* | don't count eof as truncated | Evan Martin | 2013-04-27 | 1 | -1/+2 |
| | |||||
* | fix warning | Evan Martin | 2013-04-27 | 1 | -1/+1 |
| | |||||
* | deps log: recover on truncated entry | Evan Martin | 2013-04-27 | 2 | -9/+103 |
| | | | | | | If a read fails while reading an entry, truncate the log to the last successfully read entry. This prevents corruption when a subsequent run appends another entry. | ||||
* | disable SubprocessTest.SetWithLots on travis-ci | Evan Martin | 2013-04-26 | 1 | -1/+1 |
| | | | | We run into fork() limits on their VM. | ||||
* | add a test for truncated deps | Evan Martin | 2013-04-26 | 2 | -0/+61 |
| | | | | | This doesn't yet exhibit the full problems with truncation, but it's a start. | ||||
* | make DepsLogTest actually clean up on teardown | Evan Martin | 2013-04-26 | 1 | -1/+1 |
| | | | | | I think I commented this out when tracking down a bug and forgot to uncomment it. | ||||
* | travis: pull in gtest before building | Evan Martin | 2013-04-22 | 1 | -1/+4 |
| | |||||
* | add a .travis.yml, trying out travis-ci.org | Evan Martin | 2013-04-22 | 1 | -0/+5 |
| | |||||
* | Merge pull request #552 from nico/deps_dryrun | Evan Martin | 2013-04-22 | 2 | -2/+31 |
|\ | | | | | Don't record deps in dry runs. | ||||
| * | keep dry run more similar to normal run | Nico Weber | 2013-04-22 | 1 | -4/+2 |
| | | |||||
| * | Don't record deps in dry runs. | Nico Weber | 2013-04-22 | 2 | -2/+33 |
|/ | | | | | | | deps_log() is NULL during dry runs, so this fixes a crash. It also matches ninja 1.2.0's behavior as far as I can tell. Fixes issue #551. | ||||
* | adjust the wording in "multiple rules generate X" warning | Evan Martin | 2013-04-19 | 1 | -1/+2 |
| | | | | | | | | | Ideally we'd detect this at build time and only warn if your build was affected, but that's hard to do. (Really we should just abort when this scenario is detected rather than continuing, but now users are relying on it.) Hopefully improves issue #543. | ||||
* | make DiskInterfaceTest.StatBadPath quiet | Evan Martin | 2013-04-19 | 3 | -5/+17 |
| | | | | | | Add a flag to temporarily suppress error output. Fixes issue #281. | ||||
* | drop std:: qualifiers on more stl datatypes | Evan Martin | 2013-04-18 | 7 | -10/+10 |
| | | | | We "using namespace std" anywhere we need a std::string or a std::vector. | ||||
* | Merge pull request #548 from sgraham/laconic-with-repeat | Nico Weber | 2013-04-18 | 1 | -4/+17 |
|\ | | | | | reset count in LaconicPrinter for when using --gtest_repeat | ||||
| * | also print iteration for gtest_repeat=-1 | Scott Graham | 2013-04-18 | 1 | -4/+12 |
| | | |||||
| * | reset count in LaconicPrinter for when using --gtest_repeat | Scott Graham | 2013-04-17 | 1 | -0/+5 |
| | | |||||
* | | fix --profile=pprof on newer ubuntus | Evan Martin | 2013-04-18 | 1 | -1/+2 |
|/ | | | | The --as-needed default for ld would drop -lprofiler. | ||||
* | Merge pull request #547 from nico/vim_cpo | Nico Weber | 2013-04-16 | 1 | -2/+8 |
|\ | | | | | Note that version 1.3 of the vim syntax file has been sent upstream. | ||||
| * | Note that version 1.3 of the vim syntax file has been sent upstream. | Nico Weber | 2013-04-16 | 1 | -2/+8 |
|/ | | | | | | | Also fix an issue noticed by Bram during integration: Make sure to set "cpo" correctly. This matches other vim syntax files and is required to make sure that the backslash continuation works with all .vimrc files (see `:help line-continuation`). | ||||
* | Merge pull request #541 from sgraham/fix-vs2012 | Evan Martin | 2013-04-13 | 2 | -3/+4 |
|\ | | | | | try again on vs2012 build fixes | ||||
| * | try again on vs2012 build fixes | Scott Graham | 2013-04-11 | 2 | -3/+4 |
| | | |||||
* | | bootstrap: make --windows usable for cl.exe-on-wine | Evan Martin | 2013-04-11 | 1 | -11/+13 |
| | | |||||
* | | add HACKING note on cl.exe-via-wine | Evan Martin | 2013-04-11 | 1 | -0/+6 |
| | | |||||
* | | windows: always extract dependencies, even on compile failure | Evan Martin | 2013-04-11 | 1 | -5/+7 |
| | | | | | | | | We always want to filter /showIncludes output. | ||||
* | | fix test | Evan Martin | 2013-04-11 | 1 | -14/+14 |
| | | | | | | | | | | I intentionally changed the semantics of the code that affected this test, but didn't update the test. Oops. | ||||
* | | windows build fix | Evan Martin | 2013-04-11 | 1 | -1/+2 |
|/ | | | | How did this ever work? | ||||
* | build ninja itself in deps mode | Evan Martin | 2013-04-10 | 1 | -1/+1 |
| | | | | Hopefully will help flush out bugs. | ||||
* | always use output timestamp for deps | Evan Martin | 2013-04-10 | 2 | -17/+3 |
| | | | | | | | | Using the timestamp of the .d file was wrong. It can be written at a different time than the output, and it is immediately deleted after parsing; only the output remains for subsequent timestamp comparison purposes, so always use the output's timestamp. (This is how the code worked on Windows already.) | ||||
* | include mtimes in deplog explain | Evan Martin | 2013-04-10 | 1 | -1/+2 |
| | |||||
* | slightly smaller subsections | Evan Martin | 2013-04-10 | 1 | -0/+3 |
| | |||||
* | move pool docs down under "more details" | Evan Martin | 2013-04-10 | 1 | -56/+55 |
| | | | | | | The earlier section is a tutorial-style overview. The latter section is for side features like phony rules and header dependencies. Pools fit in with the latter. | ||||
* | more minor manual touchups | Evan Martin | 2013-04-10 | 1 | -5/+5 |
| | |||||
* | make unittest match module name | Evan Martin | 2013-04-10 | 1 | -0/+0 |
| | |||||
* | mention mtimes | Evan Martin | 2013-04-10 | 1 | -2/+3 |
| | |||||
* | monospace flags | Evan Martin | 2013-04-10 | 1 | -2/+3 |
| | |||||
* | more windows doc updates | Evan Martin | 2013-04-10 | 1 | -4/+11 |
| |