rearrange handling of builtin bindings to make rules simpler

Now, a 'build' block can override any special binding like 'command'
or 'description' if it needs to.

wrap some overlong lines

fix all "class" -> "struct"

add a TODO from a pull request

Merge pull request #455 from azuk/master

Make a native Windows build possible when using Cygwin Python.

Merge pull request #467 from ppuryear/browse-error

browse: Read ninja's error text from stderr.

ninja_syntax.py: don't add trailing space when build has no inputs

Seen in Ninja's configure.py where it creates an "rpmbuild" command.

fix test build under clang/system gtest

Tests always need GTEST_HAS_RTTI=0 set, but the code was only
setting it in the --with-gtest branch.  Instead always use the
test-specific cflags when compiling test code.

wrap test in anon namespace

correctly open /dev/null in subprocesses

Fixes issue #468.

update a comment

Merge pull request #474 from yannicklm/master

  NINJA_STATUS: add support of `%p` for percentage

Merge pull request #475 from nico/vim

Update version number in ninja.vim after recent changes.

Update version number in ninja.vim after recent changes.

NINJA_STATUS: add support of `%p` for percentage

Merge pull request #471 from riannucci/fix_vim_syntax

Fix vim syntax

Merge pull request #472 from riannucci/add_python_pool_syntax

Add python ninja_syntax.py support for pool

Merge pull request #473 from riannucci/fix_emacs

Fix emacs ninja-mode

Add missing default keyword

Add pool

Add missing rspfile and rspfile_content

Add ninja.vim syntax for pool

Add python ninja_syntax.py support for pool

Merge pull request #461 from riannucci/global_section

Resource pools for ninja

browse: Read ninja's error text from stderr.

Remove unnecessary parameter from ShouldDelayEdge

Rename isValid

Fix formatting

Improve comments for src/state.h

Doc improvements

Make Edge->pool() a pointer like it should have been

Improve the manual documentation

Merge pull request #464 from tfarina/clang-warning

Fix clang warning.

Fix clang warning.

The return type of CollectTarget() is Node, so we should return NULL in the
failure case instead of false.

src/ninja.cc:188:16: warning: initialization of pointer of type 'Node *' to null from a constant boolean expression [-Wbool-conversion]
        return false;
               ^~~~~

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

Merge pull request #459 from maximuska/proposed/refactor-collect-targets

refactoring: decompose CollectTargetsFromArgs

Merge pull request #443 from tfarina/disk-interface-alloc

Allocate disk_interface near where it's needed.

Merge pull request #454 from nico/tweak

Fix a doxygen bug found by clang's -Wdocumentation.

Revert "Dump state for debugging"

This reverts commit 9b196dc806e57cefd88bbbacd12286447dbf9ad9.

Uninitialized variable! There is always one...

Dump state for debugging

Make edge dump pool name, and skip default pool

cover the nulled pool case

add docs

Add some tests!

begin rationalizing platform for both parsers

block parse method done

all building and tests passing

Dump pools with State

Merge pull request #458 from rgeary1/minor

Minor improvements to ninja : Different exit code if user presses Ctrl-C, add eclipse to .gitignore, Improve an error message

refactoring: decompose CollectTargetsFromArgs

Exit status = 2 if user presses ctrl-c

Change-Id: I7be958e18eb2e434e78afb6e03b332281a651957

Improved error message with more information

Change-Id: Idb1ce67a320a9819de262d83b498ee10eb362ed2

.gitignore Eclipse project files

Change-Id: I7cfe7cee92e1800284829f099fe6a44f212b527f

Add --windows option to bootstrap.py

Makes it possible to make a native Windows build when using Cygwin Python.

Fix a bug... now chrome build works O_O

and some basic implementation

stub out an api and de-constify Pool

Const ref FTW

Pull graph.cc too

Pull out base changes to state

Fix a doxygen bug found by clang's -Wdocumentation.

(That found one more issue, but I think that might be a bug in
-Wdocumentation, http://llvm.org/PR14295.)

refactor repeated code in cleaner

delete some obsolete TODOs

trailing whitespace

Merge pull request #453 from curinir/bash_completion

Add support for reading directory for bash completion.

Add support for reading directory for bash completion.

Currently ninja bash completion only works in the directory where the
build.ninja files is located. This adds support for using the -C
argument to ninja.

Merge pull request #451 from ppuryear/browse-python3

browse.py: Support Python 3.

browse.py: Don't truncate error message if ninja -t query fails.

Signed-off-by: Philip Puryear <philippuryear@gmail.com>

browse.py: Fix truncation with an unknown target.

Signed-off-by: Philip Puryear <philippuryear@gmail.com>

browse.py: Python 3 support.

Signed-off-by: Philip Puryear <philippuryear@gmail.com>

fix bad merge

mingw helper: drop unused -r flag

update mingw docs for ubuntu precise

fix whitespace

bootstrap: fail more gracefully if gcc isn't available

Merge pull request #450 from zchothia/zc/py3k

Add support for Python 3

Python scripts: use built-in print function.

Add support for Python 3

Merge pull request #441 from tfarina/missing-virtual

Add missing 'virtual' annotation to ReadFile() override.

Add missing 'virtual' annotation to ReadFile() override.

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

Merge pull request #448 from maximuska/proposed/buildlog-mini-refactor2

Proposed/buildlog mini refactor2

build log: fixing parameter names

Merge pull request #446 from maximuska/proposed/start-end-edge-metrics

build metrics: StartEdge and EndEdge

build metrics: StartEdge and EndEdge

Total build time is also relevant.
The costs of StartEdge and EndEdge turned out to be quite surprising.
Especially if multiple targets are specified on edges.
 (I've tried declaring depfiles targets explicitly)

build log: mini-refactoring to use constructors to initialize entries

build log: moving HashCommand() calculation out of targets loop

build log: Adding test for multiple target edges

Allocate disk_interface near where it's needed.

This avoids allocating disk_interface unnecessarily.

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

Merge pull request #439 from wolfp/metric-recompact

Add metric for log recompacting

Add metric for log recompacting

Merge pull request #438 from wolfp/clean-efficiency

Improve the efficiency of -t clean

Improve the efficiency of -t clean

Prevent each node from being examined for cleaning multiple times, if it
is used in several other nodes

Merge pull request #437 from mo/master

[PATCH] Fixed two spelling errors in ninja

Fix two spelling errors

Merge pull request #436 from tfarina/dry-run-unnamed-namespace

Move DryRunCommandRunner into a unnamed namespace.

Move DryRunCommandRunner into a unnamed namespace.

No functional change.

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

Merge pull request #433 from polrop/fix-help-option

Don't say -h is invalid whereas it is supported.

Don't say -h is invalid whereas it is supported.

getopt_long(3) was reporting "ninja: invalid option -- h" when "ninja -h" was
called.

Regression most probably introduced by 5fdb12ed5cec4e1c853c64026142d088ff5519e1

Merge pull request #428 from maximuska/proposed/rpm-build-fixed

packaging: refactored rpm building

packaging: refactored rpm building

Now passing through a valid SRPM, working in a standard way.