addressed pcc comments

pselect->ppoll on linux to raise the process limit roof

Merge pull request #231 from pcc/exit-cleanup-modified

Be more selective about deleting output files when interrupted

add an "urtle" tool

Be more selective about deleting output files when interrupted

Specifically, only delete if the file was modified or if the rule uses
a depfile.

Fixes issue #226.

Merge pull request #229 from sgraham/overprint

[win32] Overprint the previous line if we're attached to a console

[win32] Overprint the previous line if we're attached to a console

fix some public/private errors in rspfile patch

Merge pull request #217 from PetrWolf/master

Response files

Merge pull request #218 from tfarina/fix-graph-todo

Fix the TODO in graph.h.

Merge pull request #223 from pcc/restat-clean-manifest

Do not reload the manifest if a restat cleans it while being rebuilt

Do not reload the manifest if a restat cleans it while being rebuilt

Merge pull request #222 from polrop/test-bug-fix-62e9139740

Add a test for empty path in CanonicalizePath().

Add a test for empty path in CanonicalizePath().

This test covers bug fix introduced by 62e9139740.  However,
reverting this patch does not trigger a test failure.  Maybe, I am
not testing on the right platform (Linux).  Anyway, in all cases
I think this test deserves to be added.

fix depfile_ access

Fix the TODO in graph.h.

Signed-off-by: Thiago Farina <tfarina@chromium.org>

Merge branch 'master' of github.com:PetrWolf/ninja

Fix tabs, typos

Merge pull request #176 from pcc/exit-cleanup

Implement cleanup-on-interrupt

graphviz: don't draw edges multiple times

From a patch from Ian Godin <iangodin@gmail.com>.

Made graph horizontal (better for filenames)

Add comments, remove unwanted whitespace change

Response files

Fixes the build on OpenIndiana

Merge pull request #214 from qhuo/onpipeready-fix

Avoid using undefined value in Subprocess::OnPipeReady()

Merge pull request #213 from syntheticpp/evaluate-only-once

Evaluate command only once

don't crash when CanonicalizePath removes all path components

From a patch from Peter Kuemmel <syntheticpp@gmx.net>.

If a command fails, wait for all running commands to terminate before we do

Previously, if a command fails, the fate of the other child processes
running in parallel was inadequately controlled.  On POSIX platforms,
the processes were orphaned.  Normally they would run to completion,
but were liable to being killed by a SIGPIPE.  On Windows, the child
processes would terminate with the parent.  The cleanup-on-interrupt
patch caused the SubprocessSet and Builder destructors to clean
up after themselves by killing any running child processes and
deleting their output files, making the behaviour more predictable
and consistent across platforms.

If the build is interrupted by the user, this is correct behaviour.
But in the case where the build is stopped by a failed command, this
would be inconsistent with user expectations.  In the latter case,
we now let any remaining child processes run to completion before
leaving the main loop in Builder::Build.

Implement cleanup-on-interrupt

This causes us to clean up by deleting any output files belonging
to currently-running commands before we quit if we are interrupted
(either by Ctrl-C or by a command failing).

Fixes issue #110.

Clean up how the build line is managed at the end of the build

Avoid using undefined value in Subprocess::OnPipeReady()

compute command twice only in case of an error

Merge pull request #211 from okuoku/msvc-build-c4819

windows: Disable warning C4819

windows: Disable warning C4819

warning C4819: The file contains a character that cannot be represented
in the current code page.

Merge pull request #199 from qhuo/create-process

Mark CreateProcess "program not found" failure as non-fatal

Merge pull request #210 from syntheticpp/colon-escape

add colon escaping

parse $:

'$:' is a valid string now, it expands to ':'
update error messages and show a hint when something went wrong.

add colon escaping

Needed for Windows drive names.

For instance configure with gtest:

    python configure.py --with-gtest=c$:\gtest-1.6.0

Merge pull request #209 from scottfranklin/python3_compat

Maintain python3 compatibility for ninja_syntax.py

Explicitly turn input/output lists into list objects.

Python 3 does not support map.append/extend.

Merge pull request #208 from okuoku/cygwin-build

Fix cygwin build by including stdio.h here

Add a test, NoSuchCommand.

Fix cygwin build by including stdio.h here

rearrange query/browse output to be more sensible

Merge pull request #204 from syntheticpp/parser-crash

Fix parser crash

Merge pull request #203 from syntheticpp/win-disk-interface

Win disk interface

check access to first element of string: string could be empty

fix search&replace error on struct name

no error code about too long file names on Windows

Merge pull request #201 from syntheticpp/mingw-msvc

Some workarounds are only for msvc.

only msvc needs these workarounds

windows: use _WIN32 define everywhere

Rather than mixing use of WIN32 and _WIN32.

Mark CreateProcess "program not found" failure as non-fatal

Merge pull request #198 from nico/strip0

Strip ansi escape sequences from subcommand output when not writing to a...

Strip ansi escape sequences from subcommand output when not writing to a smart terminal.

Merge pull request #197 from doctorlove/simple_dry_run

Simple dry run

Add a test that dry run shows all commands that could be run (none
cleaned) and a fix for this

Stop warning about chdir etc which are treated as errors on MSVC with the
current settings

windows: always generate PDB

enable libstd++ debugging in debug mode

Merge pull request #196 from evanj/master

Fix MSVC invalid string problem

BuildLog: Use Log::insert(Log::value_type()) to avoid invalid strings.

The MSVC std::string implementation copies strings, so using make_pair
resulted in a pointer to invalid memory. This ensures the insert uses a
StringPiece without an intermediate std::string copy, so the correct pointer
ends up in the hash_map.

windows: make ninja_test build

Merge branch 'stringpiece-hash'

This branch changed the fundamental key used in hashes in Ninja from
char* to StringPiece.  Using StringPiece allows us to eliminate some
copies and null termination in various places.

While I was at it, I switched the hash function to the Murmur hash.

Seems to be ~100ms faster:

evmar:/work/chrome/src$ ~/projects/ninja/misc/measure.py ~/projects/ninja/ninja-trunk -C out/Debug chrome
sampling: 994ms 1010ms 1000ms 999ms 1019ms 994ms 1002ms 993ms 997ms 1040ms
estimate: 993ms (mean err 11.6ms)
evmar:/work/chrome/src$ ~/projects/ninja/misc/measure.py ~/projects/ninja/ninja -C out/Debug chrome
sampling: 888ms 895ms 891ms 886ms 896ms 888ms 891ms 903ms 895ms 888ms
estimate: 886ms (mean err 5.7ms)

decide on murmur hash, delete stl hash

A nice thing about working on Google: a C++ expert who happened
to be writing a proposal on hashing for the next C++ version
wandered into my office.  He seemed to think using Murmur here
is fine.

build gtest with -fvisibility=hidden as well

Fixes a warning on Mac.

make new hash work on windows

windows: hash_map bucket count has different getter

GetTempPath() needs windows.h

back to murmur hash

remove string copy while loading depfiles

split canonicalize

more stringpiece

murmur -> stl hash

switch node lookup to StringPiece

convert ExternalStringHash to use StringPiece

include main hash load in -d stats output

configure.py: make CC='command with args' work

dynamic metrics report width

track canonicalize metric

migrate tempdir code to test.cc

adjust restat behavior around missing outputs

If a restat rule claims to write an output but doesn't, consider it
"no change" (in the restat sense) if the output didn't exist beforehand.
I.e. if the output didn't exist before and the output doesn't exist after,
we don't need to run dependent rules.

add a '-d stats' flag for detailed timings

1) Add a system for recording detailed timing info of functions.
2) Add a -d flag for requesting debug info at runtime, with the
   above as the first user.

windows: don't use \e in string

drop some std:: prefixes

convert all time_t to a new TimeStamp type

factor out the main builder call

Merge pull request #194 from evanj/newline_spaces

Lexer: include leading spaces in the newline token.

Merge pull request #193 from evanj/reparse_error

ninja.cc: Clear the rules/variables/state when reloading build.ninja

Merge pull request #192 from evanj/master

Fix ninja_test by adding the change to lexer.cc to lexer.in.cc

Lexer: include leading spaces in the newline token.

This means that indented blank lines are skipped without causing errors.

ninja.cc: Clear the rules/variables/state when reloading build.ninja

This error was introduced in commit 4f6f015b. Previously, the reload goto
label created a new State. This now does the same, manually.

Lexer: Remove the trailing newline from lexer.in.cc not just lexer.cc

This probably should have been part of commit eeed7241

test error message of ManifestParser::Load

Fixes the other half of issue #187.

make Lexer::Error not emit trailing newline

Now it's consistent with other errors.
Fixes part of issue #187.

make my last change actually work

remove build_log v1/v2 code

warn on accidental args to configure

Lexer: include leading whitespace in the comment token.

Indented comments are ignored rather than causing errors.

Merge pull request #188 from nico/spacelog

Switch build log to use tabs as field separators, to support outputs wit...