Put & operator with the type rather than the variable name.

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

doc encoding support

allow UTF-8 in rule descriptions

The lexer already mostly allowed this, except that chars >127 were
being interpreted as negative indexes into the lexer table.

Merge pull request #278 from tychoish/master

Minor Documentation Tweak

reduce custom ninja status patch in minor ways

Elsewhere in the code I avoid sstream and manual buffer management,
so switch this code to behave similarly.  Sorry for being OCD.

Merge branch 'custom_ninja_status' of git://github.com/polrop/ninja

Merge pull request #272 from nico/statstweak

Switch the order of total and avg columns in -d stats output.

Merge pull request #269 from sgraham/explain

add support for -d explain to help debug why rules are running

Merge pull request #228 from pcc/lazily-create-commandrunner

Lazily create the CommandRunner in Builder::Build

Merge branch 'dont-clean-phony' of git://github.com/pcc/ninja

Conflicts:
src/clean.cc

Merge pull request #207 from iwadon/fix-configure.py-windows

Fix configure.py for unpacked gtest on Windows platform.

Refactor and test progress status formatting.

Introduce NINJA_STATUS env var for customization.

Some people may want to display different relevant information about
the progress of the build.  Also it can be usefull to debug jobs
allocation.

Remove trailing white spaces.

docs: clarifying that there can be only one command per rule

Switch the order of total and avg columns in -d stats output.

add support for -d explain to help debug why rules are running

Merge pull request #252 from qhuo/remove-depfile-when-cleaning

Remove depfiles when running "ninja -t clean <target>" or "ninja -t clean -r <rule>"

Merge pull request #265 from mathstuf/dev/verbose-bootstrap

Allow the bootstrap to be verbose

Merge pull request #253 from qhuo/include-io.h

Fix windows build (add #include <io.h>)

Make |log_| variable private to BuildLog.

This fix the TODO in build_log.h file.

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

Merge pull request #263 from mathstuf/dev/gcc-470-build-fixes

Include unistd.h where needed

Merge pull request #260 from sgraham/rspfile_syntax

add rspfile and rspfile_content to ninja_syntax

Merge pull request #259 from usovalx/distcc_depfile_fix

Explicitly specify target name for gcc depfiles

Merge pull request #256 from iwadon/fix-sign-compare

fix warning: "comparison between signed and unsigned integer expressions"

Merge pull request #257 from PetrWolf/53598255

Add NOMINMAX macro to fix a compilation error on Win32

Allow the bootstrap to be verbose

Include unistd.h where needed

add rspfile and rspfile_content to ninja_syntax

Explicitly specify target name for gcc depfiles

Merge pull request #251 from polrop/depfile-parser-accept-tilde

Depfile parser accept tilde.

fix warning: "comparison between signed and unsigned integer expressions"

Add NOMINMAX macro to fix a compilation error on Win32

Fix windows build (add #include <io.h>)

Remove depfiles when running when running "ninja -t clean <target>"
or "ninja -t clean -r <rule>"

Depfile parser accept tilde.

Merge pull request #250 from qhuo/SetCloseOnExec-for-Windows

Add a Windows version of SetCloseOnExec()

Merge pull request #249 from qhuo/StatBadPath-Win7

Fix StatBadPath for Windows 7

Add a Windows version of SetCloseOnExec()

Fix StatBadPath for Windows 7

The StatBadPath test expects both Windows and Linux to reject a
path name with 512 characters. However, it seems that such path
is actually acceptable to Windows 7. The change constructs a
different path name that is invalid on Windows.

Merge pull request #246 from HeisSpiter/master

Fix resources leaks

Fix resources leaks

Merge pull request #245 from usovalx/unify_depfile_targets

Issue #241 - handle depfiles generated by older versions of GCC

Issue #241 - handle depfiles generated by older versions of GCC

Older versions of GCC would produce broken depfiles when -MT or -MQ is used
    gcc43 -MT foo.o -MMD -MF foo.o.d -o foo.o -c foo.c
will result in the following depfile
    foo.o foo.o: <dependencies>

Parse multiple outputs unifying duplicates and correctly report errors if
they are different.

Merge pull request #235 from fischman/ppoll

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

Merge pull request #239 from pcc/enotdir

Treat paths of the form "existing-file/something" as non-existent

Treat paths of the form "existing-file/something" as non-existent

Some people like to construct phony target names by appending a
"/something" suffix to an existing target "foo".  But if "foo" is an
existing file, stat will report ENOTDIR for this path, causing ninja
to spew errors.  Fix this by treating ENOTDIR in the same way as we
do ENOENT -- as a non-existent path.

drop rtti via -fno-rtti; we don't use it

make urtle fit on 24-line terminal

adjust rspfile docs to properly render

Merge pull request #238 from pcc/outputs-ready-no-inputs

Mark a phony target with no inputs as outputs-ready

Merge pull request #237 from pcc/ninja-clean-hint

Give a useful hint if the user runs "ninja clean" and there is no "clean...

Mark a phony target with no inputs as outputs-ready

Even if such a target is dirty (i.e. the file does not exist), it
has nothing to do, which makes it safe to mark as outputs-ready.
Without this change, ninja will print no output when rebuilding the
target (or an order-only dependency thereof), instead of reporting
it has "no work to do".

Give a useful hint if the user runs "ninja clean" and there is no "clean" target

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.

Don't delete phony targets when cleaning a specified target

Also, modify Cleaner::CleanAll to use Edge::is_phony.

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

Lazily create the CommandRunner in Builder::Build

This allows the user to interrupt the build (i.e. using Ctrl-C)
while the build plan is being computed.

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

Just add /nologo to gtest_cflags, instead of use cflags + gtest_cflags