summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Recompact the deps log when it gets too big.Nico Weber2013-05-012-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 Weber2013-05-012-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 Weber2013-04-302-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 Weber2013-04-302-6/+15
| | | | No functionality change.
* Move some DepsLog::Deps initialization code into its constructor.Nico Weber2013-04-302-5/+3
| | | | No functionality change.
* Merge pull request #559 from nico/9_9Nico Weber2013-04-291-1/+1
|\ | | | | Fix Windows build more.
| * Fix Windows build more.Nico Weber2013-04-291-1/+1
|/
* Merge pull request #558 from nico/buildfixEvan Martin2013-04-295-14/+27
|\ | | | | Introduce a Truncate() function that works on POSIX and Windows.
| * Introduce a Truncate() function that works on POSIX and Windows.Nico Weber2013-04-295-14/+27
| | | | | | | | Hopefully fixes the build on Windows.
* | Merge pull request #556 from nico/unlinkEvan Martin2013-04-291-0/+5
|\ \ | | | | | | Make sure that recompaction writes a pristine new depslog.
| * | Make sure that recompaction writes a pristine new depslog.Nico Weber2013-04-291-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/depscompactcrashEvan Martin2013-04-292-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 Weber2013-04-292-0/+8
|/ | | | Part of issue #554.
* don't count eof as truncatedEvan Martin2013-04-271-1/+2
|
* fix warningEvan Martin2013-04-271-1/+1
|
* deps log: recover on truncated entryEvan Martin2013-04-272-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-ciEvan Martin2013-04-261-1/+1
| | | | We run into fork() limits on their VM.
* add a test for truncated depsEvan Martin2013-04-262-0/+61
| | | | | This doesn't yet exhibit the full problems with truncation, but it's a start.
* make DepsLogTest actually clean up on teardownEvan Martin2013-04-261-1/+1
| | | | | I think I commented this out when tracking down a bug and forgot to uncomment it.
* travis: pull in gtest before buildingEvan Martin2013-04-221-1/+4
|
* add a .travis.yml, trying out travis-ci.orgEvan Martin2013-04-221-0/+5
|
* Merge pull request #552 from nico/deps_dryrunEvan Martin2013-04-222-2/+31
|\ | | | | Don't record deps in dry runs.
| * keep dry run more similar to normal runNico Weber2013-04-221-4/+2
| |
| * Don't record deps in dry runs.Nico Weber2013-04-222-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" warningEvan Martin2013-04-191-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 quietEvan Martin2013-04-193-5/+17
| | | | | | Add a flag to temporarily suppress error output. Fixes issue #281.
* drop std:: qualifiers on more stl datatypesEvan Martin2013-04-187-10/+10
| | | | We "using namespace std" anywhere we need a std::string or a std::vector.
* Merge pull request #548 from sgraham/laconic-with-repeatNico Weber2013-04-181-4/+17
|\ | | | | reset count in LaconicPrinter for when using --gtest_repeat
| * also print iteration for gtest_repeat=-1Scott Graham2013-04-181-4/+12
| |
| * reset count in LaconicPrinter for when using --gtest_repeatScott Graham2013-04-171-0/+5
| |
* | fix --profile=pprof on newer ubuntusEvan Martin2013-04-181-1/+2
|/ | | | The --as-needed default for ld would drop -lprofiler.
* Merge pull request #547 from nico/vim_cpoNico Weber2013-04-161-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 Weber2013-04-161-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-vs2012Evan Martin2013-04-132-3/+4
|\ | | | | try again on vs2012 build fixes
| * try again on vs2012 build fixesScott Graham2013-04-112-3/+4
| |
* | bootstrap: make --windows usable for cl.exe-on-wineEvan Martin2013-04-111-11/+13
| |
* | add HACKING note on cl.exe-via-wineEvan Martin2013-04-111-0/+6
| |
* | windows: always extract dependencies, even on compile failureEvan Martin2013-04-111-5/+7
| | | | | | | | We always want to filter /showIncludes output.
* | fix testEvan Martin2013-04-111-14/+14
| | | | | | | | | | I intentionally changed the semantics of the code that affected this test, but didn't update the test. Oops.
* | windows build fixEvan Martin2013-04-111-1/+2
|/ | | | How did this ever work?
* build ninja itself in deps modeEvan Martin2013-04-101-1/+1
| | | | Hopefully will help flush out bugs.
* always use output timestamp for depsEvan Martin2013-04-102-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 explainEvan Martin2013-04-101-1/+2
|
* slightly smaller subsectionsEvan Martin2013-04-101-0/+3
|
* move pool docs down under "more details"Evan Martin2013-04-101-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 touchupsEvan Martin2013-04-101-5/+5
|
* make unittest match module nameEvan Martin2013-04-101-0/+0
|
* mention mtimesEvan Martin2013-04-101-2/+3
|
* monospace flagsEvan Martin2013-04-101-2/+3
|
* more windows doc updatesEvan Martin2013-04-101-4/+11
|