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.

Make ninja_test output nicer.

Use LinePrinter in test runner.

Make LinePrinter members private, add comments.

move printing of new lines into LinePrinter.

refactor; no intended functionality change

move single-line printing to new class

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.

sync version number in master with release

Update RELEASING with notes on how to not screw this up again.

split FakeCommandRunner out of BuildTest

Separating concerns to make a subsequent refactoring easier.

move test virtual time "now_" into VirtualFileSystem

It's the only piece that cares about the current time.

Merge pull request #527 from wang-bin/compile_fix

fix compile error about noreturn on windows

fix compile error about noreturn on windows

Merge pull request #526 from RedX2501/improved_bash_completion

Improved bash completion when using tools like '-t clean'

Improved bash completion when using tools like '-t clean'

refactor some of the output mtime-handling code

Reduces duplicated explain output.

add noreturn attr on Fatal()

decided against using XXX in the manual for unknown versions

tag a feature with the right version number

Merge pull request #525 from nico/escapedollar

Dollar signs in depfiles are escaped as "$$", not "\$".

Dollar signs in depfiles are escaped as "$$", not "\$".

See http://llvm.org/PR15642. I checked that gcc does produce depfiles
containing "$$" for files with "$" signs in their name (and as of
r178540, so does clang). I also checked that .d files that escape dollar
signs with "\$" are not read correctly by make.

allow paths with '!' in depfiles

See funny paths in
  https://github.com/google/libcxx/tree/master/test/iterators/stream.iterators/istreambuf.iterator/

Merge pull request #522 from nico/spelldebug

Add spell checking for debug flags.

Add spell checking for debug flags.

I just used `ninja -d stat` and it took me a bit to realize that I missed the
trailing 's'.

While here, move the message printing from printf() to Error(). This makes the
output consistent with other error outputs: The messages are now prefixed with
"ninja: error: " instead of just "ninja: " and they go to stderr instead of
stdout.

Merge pull request #521 from riannucci/ignore_duplicate_edges_in_shcedule_work

Fix duplicate edge Pool crash in the minimally invasive way

Add regression test

Fix debug build on linux (type strictness).

Fix Pool to use a set internally

Fix duplicate edge Pool crash in the minimally invasive way

Merge pull request #518 from syntheticpp/max-patch

Posix symbols are not enabled by default for MSVC

Posix symbols are not enabled by default for MSVC

Merge pull request #513 from fifoforlifo/master

Adding elapsed time (%e) to NINJA_STATUS.

Added %e to manual.asciidoc, fixed brace style.

Merge pull request #512 from pcc/compilation-database

Add compdb tool

Add compdb tool

This tool helps convert Ninja build files to a compilation database
of the form used by Clang tooling.

Merge pull request #499 from lht/patch-1

Fix typo in manual

Merge pull request #503 from mo/fix_typo_20130219

Fix typo, s/verison/version/

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

Made %e purely a number.

NINJA_STATUS now takes %e for elapsed time.

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.

Fix typo, s/verison/version/

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

Fix typo in manual

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