Merge branch 'master' into release

call this version 1.3.0

delete obsolete todo

Merge pull request #572 from nico/incless

Remove a few unused includes.

Remove a few unused includes.

Merge pull request #568 from riannucci/fix_test

Fix pool tests (and provide utility to fix other similar issues as well)

Merge pull request #570 from nico/fixdarwin

Fix bootstrap on OS X.

Fix bootstrap on OS X.

This was broken in 4c552c2c3cbc07acce9c1a379fee054a3f680100.

Fix Pool tests.

At any phase in the test where multiple edges are ready
simultaneously, acquire all edges and sort them into a predictable
order. This allows the test to execute deterministically regardless
of the order of edge allocation.

Merge pull request #565 from mdempsky/master

Add support for OpenBSD.

Don't poll pipes using POLLRDHUP

POLLRDHUP is Linux-specific, and isn't necessary for polling pipes
anyway.  Linux and OpenBSD both return POLLHUP if no process has the
pipe open for writing.

Merge pull request #566 from nico/tweak

minor formatting change

minor formatting change

Merge pull request #563 from nico/recompact-deps

Implement deps log recompaction.

Add support for OpenBSD.

Use ppoll() on OpenBSD.  Also, fix interrupt handling to recognize
that on FreeBSD and OpenBSD, an interrupt might have been delivered
even if pselect()/ppoll() don't return -1/EINTR.

Recompact the deps log when it gets too big.

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().

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.

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.

No functionality change.

Move some DepsLog::Deps initialization code into its constructor.

No functionality change.

share platform support between configure/bootstrap

Merge pull request #562 from TheOneRing/fixed-include

added missing windows include for Truncate

added missing windows include

Merge pull request #559 from nico/9_9

Fix Windows build more.

Fix Windows build more.

Merge pull request #558 from nico/buildfix

Introduce a Truncate() function that works on POSIX and Windows.

Introduce a Truncate() function that works on POSIX and Windows.

Hopefully fixes the build on Windows.

Merge pull request #556 from nico/unlink

Make sure that recompaction writes a pristine new depslog.

Merge pull request #555 from nico/depscompactcrash

Don't crash during deps log recompaction if there's more than one deps entry.

Make sure that recompaction writes a pristine new depslog.

...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.

Don't crash during deps log recompaction if there's more than one deps entry.

Part of issue #554.

don't count eof as truncated

fix warning

deps log: recover on truncated entry

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

We run into fork() limits on their VM.

add a test for truncated deps

This doesn't yet exhibit the full problems with truncation, but it's
a start.

make DepsLogTest actually clean up on teardown

I think I commented this out when tracking down a bug and forgot
to uncomment it.

travis: pull in gtest before building

add a .travis.yml, trying out travis-ci.org

Merge pull request #552 from nico/deps_dryrun

Don't record deps in dry runs.

keep dry run more similar to normal run

Don't record deps in dry runs.

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

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

Add a flag to temporarily suppress error output.

Fixes issue #281.

drop std:: qualifiers on more stl datatypes

We "using namespace std" anywhere we need a std::string or a std::vector.

Merge pull request #548 from sgraham/laconic-with-repeat

reset count in LaconicPrinter for when using --gtest_repeat

also print iteration for gtest_repeat=-1

fix --profile=pprof on newer ubuntus

The --as-needed default for ld would drop -lprofiler.

reset count in LaconicPrinter for when using --gtest_repeat

Merge pull request #547 from nico/vim_cpo

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.

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

try again on vs2012 build fixes

bootstrap: make --windows usable for cl.exe-on-wine

add HACKING note on cl.exe-via-wine

windows: always extract dependencies, even on compile failure

We always want to filter /showIncludes output.

fix test

I intentionally changed the semantics of the code that affected this
test, but didn't update the test.  Oops.

windows build fix

How did this ever work?

try again on vs2012 build fixes

build ninja itself in deps mode

Hopefully will help flush out bugs.

always use output timestamp for deps

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

slightly smaller subsections

move pool docs down under "more details"

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

make unittest match module name

mention mtimes

monospace flags

more windows doc updates

update some windows references in the docs

Drop the timing info, it's too variable to commit to a manual.

bigger manual

Merge pull request #539 from nico/typo

fix typo in comment

fix typo in comment

Merge pull request #537 from yannicklm/master

    ninja.vim: add deps as keyword

ninja.vim: add deps as keyword

new in ninja 1.3

Merge pull request #535 from sgraham/fix-windows-depslog

fix windows build after depslog

fix test

Merge pull request #534 from nico/invalid_log

On invalid depslog header, restart build instead of showing an error.

add ifdef around console_

fix windows build after depslog

On invalid depslog header, restart build instead of showing an error.

Also add tests for invalid headers.

document deps

Merge branch 'dep-pipeless'

This merges a new mechanism for tracking "depfile" dependencies that
is faster on all platforms but dramatically so on Windows.

Merge pull request #532 from nico/sshhh_gtest

Make test runner less chatty.

Try to fix build on Linux more.

Try to fix build on Linux.

add a straightforward deps log test, fix the other one

The first test I wrote was wrong; write a simpler test that exercises
the "no failures" code paths, then fix the second test and the bugs it
exposed.

Make ninja_test output nicer.

Use LinePrinter in test runner.

Make LinePrinter members private, add comments.

move printing of new lines into LinePrinter.

Write the depslog version in binary instead of text.

This way, it doubles as a byte-order marker. The header is now exactly one
line in a hex editor, and it's still relatively easy to look at the version
in a text editor.

refactor; no intended functionality change

move single-line printing to new class

Make deps=gcc without depfile an error.

When I first played with depslog, I accidentally removed the depfile attribute
on my cc edges, which had the effect of ninja silently ignoring all depfiles.
Instead, let ninja complain about edges that have deps set to gcc and depfile
set to nothing.

This is done at edge build time, instead of at mainfest load time, because
adding this check to ParseEdge() regressed empty build time for target 'chrome'
by 30ms. The check is only useful for generator authors, regular users should
never see this.

add a test for the "deps out of date" case

It touched the various remaining XXXes in the code, hooray.

Make gtest output more silent, ninja issue #528.

This is just a proof-of-concept. The terminal printing logic should be
extracted from src/build.cc and then reused here.

add a test verifying build failure on bad deps

make it an error for now to have multiple outputs with depslog

only write deps log if edge is in deps log mode

use logged deps mtime in dirty calculation

The idea here is that it's possible for a build to complete (writing
its output) but then for Ninja to get interrupted before writing out
the updated dependency information.  In that case the mtime stored in
the deps log (if any) will match the previous output, and we'll know
we need to rebuild the output just to get the deps updated.