Merge pull request #504 from benvanik/master

Add a check for the VC++ 2012-style 64-bit path.

Merge pull request #514 from dabrahams/patch-1

Bring in declaration of atoi

Bring in declaration of atoi

./bootstrap.py fails on some platforms without this include

Merge pull request #507 from ikarienator/master

Browse command does not parse URL correctly

Merge pull request #502 from tfarina/rm-unused-fun

Removed unused UnitsWaiting() function from State class.

Browse command does not parse URL correctly

Merge pull request #506 from syntheticpp/string-find

Fix wrong usage of std::string::find

Fix wrong usage of std::string::find

Add a check for the VC++ 2012-style 64-bit path.
In VS2012 the path to the 64-bit tools has changed to
VCINSTALLDIR\bin\x86_amd64\cl.exe. This change will make the bootstrap
check there to see if it exists before falling back to the old amd64 path.

Removed unused UnitsWaiting() function from State class.

This function was added at 307f0bbd("and some basic implementation"), but nobody
calls it anymore.

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

manual: move in_newline to the reference

It's a detail, it doesn't belong in the brief tutorial overview.

add versioning notes to features in the manual

This helps guide which ninja version to depend on.

make doc xrefs link to section headers

rearrange env variable section layout

move from asciidoc html to asciidoc -> docbook -> html

As best as I can tell this is the easiest way to customize
the asciidoc HTML output.

say where the default value for -j comes from

This is a FAQ.

depfile helper: check errors on writing .d files

Closes #492 and #493.

docs for ninja_required_version

(And some extra docs for top-level variables in general.)

add syntax for checking versions

Merge pull request #498 from pck/cygwin

util.cc: Reusing windows workaround for cygwin.

util.cc: Reusing windows workaround for cygwin.

This fixes:
src/util.cc: In function 'double GetLoadAverage()':
src/util.cc:337:28: error: 'getloadavg' was not declared in this scope

mark pools as experimental in the docs

add a release checklist so I won't forget steps in the future

Merge pull request #485 from petrh/fedora-rpmbuild

RPM spec file fixed to work properly on recent Fedora releases

Merge pull request #489 from sgraham/err-message-depfile

more verbose error (including path) when depfile fails to load

more verbose error (including path) when depfile fails to load

Removed re2c build dependency from RPM spec file

Merge pull request #484 from TheOneRing/fix_mingw

if windows and not msvc, set platform to mingw

Use correct Fedora versioning scheme

RPM spec file should use correct versioning scheme and list all build dependencies

if windows and not msvc, set platform to mingw

Merge pull request #479 from nico/testfix

ManifestParser constructor accesses its first argument, don't pass NULL

ManifestParser constructor accesses its first argument, don't pass NULL

The constructor does

  env_ = &state->bindings_;

so env_ is effectively set to offsetof(ManifestParser, bindings_).  This
will blow up if env_ gets dereferenced -- this doesn't seem to happen in
these tests, but it's less confusing with this patch.  Also, passing &state is
consistent with the rest of this test.

always set GTEST_HAS_RTTI=0 in all testing code

fix build on non-Linux glibc systems

ninja-build does not build on non-Linux archs, such as
GNU/kFreeBSD and GNU/Hurd.

The problem is that the GetProcessorCount() implementation for these
architectures is the sysconf() one, but <unistd.h> has not been
included, causing sysconf() and _SC_NPROCESSORS_ONLN to not be
declared. Another solution (which is the one I chose) is to make use
of the "linux" implementation which uses get_nprocs(), which is a GNU
extension and thus available for anything using GNU libc.

drop the "rules" tool completely

I had already broken this earlier, but the refactor of variable handling
broke it completely.

update docs to clarify scoping rules

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