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

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.

Merge pull request #427 from jonforums/jf/mingw-n-msvc

fix mingw build fail - redux

Merge pull request #426 from sgraham/spaces-in-headers-2

fix spaces in headers for -t msvc

Always include stdio.h

less random comment

review fixes

Silence bothersome warning from -Wextra

Struct initializations such as those in `CLWrapper::Run` of file
`src/msvc_helper-win32.cc` causes MinGW GCC to spew warnings.

Give MinGW builds MSVC build helper superpowers

Note: _WIN32 is used instead of WIN32 to enable builds with MSVC
IDE, Windows SDK non-IDE command line tools, and mingw/mingw-w64
based toolchains

fix spaces in headers for -t msvc

Merge pull request #419 from syntheticpp/rate-buffer-size

Buffer size passed to snprintf is wrong in rate code

Fix subtile buffer size error

The deduced type was char* with size 4 and not char[32] with size 32.
This removes strange output characters on Windows.

build with msvc2012

remove some code duplication

Merge pull request #422 from sgraham/fix-dupe-includes

don't emit duplicate headers for msvc helper

don't emit duplicate headers for msvc helper

Merge pull request #416 from nico/slide

Change rate measurement code.

Merge pull request #415 from nico/getopt

Clean up getopt_long call.

Change rate measurement code.

For %o, remove a superfluous + 0.5: snprintf("%f") rounds already.
Remove some unnecessary code.

For %c, fix a TODO to add a sliding window and update after every
completed edge. Else, with -j50 and several files that take 3s to
compile each, this number would only update every 150s. Also give
the number one decimal place so that this can measure steps slower
than 1s.

Clean up getopt_long call.

Remove now-unimplemented 'V' from getopt_long. Remove 'h', since it's
included in the long options. Order switch cases in the same order as in
the getopt_long argument.

mark msvc-helper as experimental

windows: merge msvc-helper into ninja.exe itself

Now "ninja -t msvc ..." passes the "..." to the msvc helper main.
This drastically simplifies bootstrap and makes ninja a single
binary again.

Merge pull request #414 from riannucci/master

Fix re2c detection in bootstrap which breaks win32 w/ gnuwin32

Fix re2c detection which breaks win32 w/ gnuwin32

scoping workaround for gcc on Windows

From https://github.com/martine/ninja/issues/410.

include termios for solaris

add solaris platform definitions

disable a new warning that popped up on Windows

This pattern is safe as long as you're careful; we don't use it
very much.

add GetProcessorCount() implementation for Solaris

drop special case for msvc bucket count

Calling bucket_count() works locally with MSVC.  I wonder if some
other change in the code fixed this.

Merge pull request #408 from nico/fix

Make sure no stale test file exists before running build log tests.

allow tools to specify when they run (before/after build.ninja load)

This will be necessary to inline msvc-helper as well as so -t graph
can get depfiles.

factor out metrics dump from ninja main

factor out build log load from main

rearrange tool-picking logic

Now "ninja -t list" works from any directory.

show all in graph

reduce indent

move BuildConfig out of globals

remove DiskInterface from globals

remove unfortunate header dependency

This was temporarily added, and now it can be removed.

clarify setter

move BuildLog to DependencyScan

The build log is needed in computing whether an edge is
dirty, so I think it belongs here.  (It's a bit weird
that Builder needs to reach into it to record completed
commands, maybe it will become cleaner with more thought.)

fix --debug for glibc pedantic mode

pass Builder as arg to build-running functions

Reducing use of globals.

remove config from BuildLog, rename members

remove a redundant arg to RecomputeOutputDirty

split out dirty recomputation logic from Edge class

Rather than passing States and DiskInterfaces through all the calls,
put the necessary ambient information in a new DependencyScan object
and move the code accordingly.

Note: I didn't move the source location of the functions to preserve
history, though this does result in a sort of weird order for the
functions in graph.cc.

Make sure no stale test file exists before running build log tests.

Merge pull request #407 from maximuska/proposed/dirty-on-missing-depfile

safer build: consider target dirty if depfile is missing

safer build: consider target dirty if depfile is missing

Merge pull request #403 from nico/everylineweighsyoudown

Remove unused macro NINJA_UNUSED_ARG.

Remove unused macro NINJA_UNUSED_ARG.

point to HACKING.md in README

drop DepfileParserTest.Tilde, as it's covered by .SpecialChars

The SpecialChars test covers a bunch of different special characters,
including tilde.

link directly to gtest zip

update old test to cover newer rule attributes

add test that checks attributes on rules

windows: pass /Zi to gtest compile as well

remove crlfs from .gitignore

These were introduced by a change of mine on Windows, whoops.

fix test broken in 697350d

That's what I get for making last-second adjustments before checking in!

make it more explicit that a bad build log causes us to rebuild

(Committing this on top of b56fe80 since they're related,
but I may end up reverting both.)

Merge pull request #401 from syntheticpp/win-network-path

on windows a network path starts with two backslashes

on windows a network path starts with two backslashes

drop HACKING from doxygen

I tried just fixing the code to pull in HACKING.md but it didn't show
up in the doxygen output; it's maybe too long to include anyway.

use 4 space tabs in configure.py, warn on re2c missing

many updates for HACKING.md

In particular, describe a policy for good patches.

Merge pull request #400 from nico/dynre2c

Only write re2c rules if a re2c binary is found in the PATH.

Only write re2c rules if a re2c binary is found in the PATH.

Merge pull request #399 from nico/hackless

Remove -fcolor-diagnostics reference from HACKING

Remove -fcolor-diagnostics reference from HACKING

configure.py adds that flag automatically if CXX is set to clang.

convert HACKING to markdown, add MSVC section

if a file is missing in the log, count it as dirty

This could cause overbuilding (if the log is missing an entry and
the right file is already in place) but is otherwise necessary
for correctness (if a file is already in place but we don't have
a log entry for it).

add a helper binary for wrapping cl.exe

Modify bootstrap etc. to make use of this binary.

pass env block to cl helper

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

rename -V to --version and clean up --help output

create phony rules for all binaries

msvc helper: drop system includes

Drop any #includes that look like they're referencing system headers.
This reduces the dependency information considerably.

includes_normalize: also lowercase cross-drive includes

It seems to me inconsistent to normalize one but not the other.

add functions for normalizing win32 include paths

(Note from Evan: this is landing Scott's code more or less verbatim
without a lot of analysis; it could maybe be simplified and reduced,
but it's only intended to be used in the MSVC helper so it's fine
to be experimental.)

add .ilk (incremental linker) to .gitignore

msvc helper: attempt to filter out when it prints the input filename

This is a heuristic but it appears to work for the Chrome build.

add subprocess-spawning to msvc_helper

Rather than using subprocess.h, reimplement the subprocess code.
This allows:
1) using anonymous (instead of named) pipes
2) not using all the completion port craziness
3) printing the output as it happens
4) further variation, like adjusting the environment (in a forthcoming
   change) without affecting the main subprocess code

doc some cl.exe flags in the configure script

move Win32Fatal into util

add a module for working with MSVC (cl.exe) behavior

This will be needed for performant builds on Windows.

use correct path separator for --with-gtest source