Merge pull request #95 from qhuo/depfile-for-command-with-multiple-outputs

Add depfile support to build command with multiple outputs (Fixes: #61)

link gtest code with -lpthread

I broke this in a refactoring.

Merge pull request #99 from tfarina/eval-env-test

Move EvalString tests to eval_env_test.cc

Move EvalString tests to eval_env_test.cc

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

Merge pull request #98 from tfarina/fix-win-build

Fix windows build by moving mkdtemp() implementation from ninja_test.cc t

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

Conflicts:
configure.py

fix redundant manifest rebuild

Don't rebuild the manifest when it's already up to date.
The underlying problem was that Builder::Build has a confusing API;
split the API so it's more clear for callers what the return values
mean.

Fix windows build by moving mkdtemp() implementation from ninja_test.cc to disk_interface_test.cc

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

allow --with-gtest flag to use unpacked gtest dir

Useful for building on Macs.

Merge pull request #96 from tfarina/disk-interface-test

Move DiskInterfaceTest into its own source file.

Move DiskInterfaceTest into its own source file.

TEST=ninja_test --gtest_filter=DiskInterface*

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

Move a passing test from Errors to MultipleOutputs

preserve configure.py params across re-runs

Merge pull request #91 from pcc/reload-manifest

Support for rebuilding and reloading manifest files

build only the main binary by default

add 'default' keyword to python module

don't use "all" in the phony example, because we now have "default"

link to github in docs

Add depfile support to build command with multiple outputs (Fixes: #61)

parsers.cpp: allow depfile used at build command with multiple outputs.
graph.cpp: allow depfile used at build command with multiple outputs.
parsers_test.cpp: make the test pass.

As before, the depfile itself can only mention one target, which must
be the first of a build command with multiple outpus.

[There is really no need to mention all the output in the depfile,
because all targets should depend on exactly the same files anyway,
because these targets are built by a single build command.]

update docs on other build system integration with recent news

make --profile take an argument to use either gmon or google-pprof

Merge pull request #94 from qhuo/named-pipe

subprocess-win32.cc: change named pipe names to contain process ID and subprocess object address.

subprocess-win32.cc: change named pipe names to contain process ID and subprocess object address.

Support for rebuilding and reloading manifest files

This introduces support for rebuilding the top-level manifest file
using a provided build statement, and reloading it before building
the user-requested targets.

Merge pull request #89 from tfarina/state

Factor out State struct from ninja_jumble.cc into its header/source files

Factor out State struct from ninja_jumble.cc into its header/source files.

This was a TODO in src/ninja_jumble.cc. Now this task is completed.

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

Merge pull request #92 from pcc/fix-warning

Fix a Clang self-assign warning by casting result to void

Merge pull request #90 from pcc/default-target-statements

Implement default target statements

Fix a Clang self-assign warning by casting result to void

Implement default target statements

This introduces a new directive, the default target statement, which
may be used to control the list of targets built by default (i.e. if
no target is named on the command line).

ninja_syntax.py: add include and subninja

Merge pull request #87 from tfarina/fix-ninja-compilation

Fix ninja compilation by including a missing disk_interface.h.

Fix ninja compilation by including a missing disk_interface.h.

When running ./ninja the compilation fails because test.h uses DiskInterface but
doesn't include disk_interface.h to get the declaration.

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

Merge pull request #86 from tfarina/real-disk-interface

Move RealDiskInterface class to disk_interface.h.

Move RealDiskInterface class to disk_interface.h.

This is a TODO in src/ninja_jumble.cc

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

semantic change: allow reaching into parent directories in paths

This allows generating build files in a subdirectory of your source tree.
- Change CanonicalizePath to accept this.
- CanonicalizePath no longer has an error condition, so change it to a void
  function.

I profiled the result against Chrome and it might be ~100ms slower, but that
might just be Chrome's size working against me.  In any case I think there
are lower-hanging performance fruit elsewhere.

Merge pull request #82 from tfarina/disk-interface

Factor out DiskInterface class into its own source/header files.

Merge pull request #83 from rlarocque2/master

Update configure.py following ninja.py rename

Update configure.py for ninja module rename

rename ninja module to ninja_syntax

add escaping function to python module

Factor out DiskInterface class into its own source/header files.

This is a TODO in src/ninja_jumble.cc

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

Merge pull request #80 from tfarina/master

fix trivial style issue.

Fix trivial style issues with Cleaner class.

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

fix double-indenting

ninja.py: fix the new test and pass the test

From a patch from Elazar Leibovich <elazarl@gmail.com>.

unit test for line-wrapping in ninja.py

From a patch from Elazar Leibovich <elazarl@gmail.com>.

don't track line/column until you encounter an error, then re-parse

This speeds up the common case (where you don't need a line number)
at the small expense of the uncommon case (for error messages, you
do need a line number).  And it's less code.

add a --debug option to configure

don't leak the build_log file descriptor into subprocesses

Closes issue #74.

clean: don't delete phony targets

Based on a patch from John Birtley.

Merge pull request #73 from qhuo/setvbuf

Set stdout to line-buffered.

Set stdout to line-buffered.

Merge pull request #71 from jbirtley/master

Fix for long command lines in Win32

Fix for win32 long command lines:

    - remove 'cmd /c' prepend from SubProcess::Start()

    - fix the subprocess tests to prepend 'cmd /c' explicitly

Fix for 'pipe has ended' errors on Win32 distributed builds

Patch from John Birtley.

Merge pull request #69 from Roorback/master

Path to python interpreter (on FreeBSD)

Merge pull request #70 from scottfranklin/master

Add tab-completion for zsh.

add docs on variable expansion

mention python module in the docs

Adding tab-completion function for zsh.

Respect $CXX, fallback to 'g++' in bootstrap.sh

use /usr/bin/env to find python interpreter

Merge pull request #62 from qhuo/multiple-outputs-fix

Add a test for the multiple outputs fix

Add a test for the multiple outputs fix

Merge pull request #59 from qhuo/multiple-outputs-fix

Multiple outputs fix: make $out include all outptus

src/graph.cc: fix EdgeEnv::LookupVariable

Make it return the correct value for "out" when there are multiple outputs.
It used to return only the first target file.

build_log_test.cc: add 3 #include for stat(2) on Linux

remove +x bit from source

On Windows, ninja didn't create needed paths first

Now it does. Still works as it should on linux too.

The canonical example that now works on Windows is:

builddir = build

rule copy
command = copy $in $out

build $builddir\fred\test.out: copy test.in

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

ninja: Split StatCache struct into its own header file.

Note: This is a TODO in ninja_jumble.cc.

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

Merge pull request #51 from philipcraig/master

mingw ninja_test links and passes. (as do linux and native VS2010)

mingw ninja_test links and passes. (as do linux and native VS2010)

Use chsize instead of truncate for mingw compatibility
SubprocessTest uses DoWork to force Subprocesses to execute
Drop GoodCommandStdOut test as it is tested in SetWithSingle

Merge pull request #50 from philipcraig/master

Fixes to let native builds on Visual Studio 2010 build, pass tests

Tests now build on a native Windows build (tested with VS2010)
All tests except SubProcess pass on a native Windows build
Tests continue not to build on a platform=mingw build

use 0 for NULL in test to silence gcc 4.5 warning

expand $$ as $

switch to $ as the line continuation char

This means that backslashes are passed through without interpretation,
allowing us to support Windows paths without worrying about escaping.

perftest: print summary info at end

add --profile configure flag

add more test coverage for makefile parsing

add a test program for evaluating depfile parse speed

move GetTimeMillis to util

move ReadFile into util

refactor let parsing, passing another test

refactor

show correct location for unexpected var error

refactor parser, check in some failing tests

emacs: highlight variable expansions too

merge two line continuation codepaths

include filename in subninja load err message

update obsolete HACKING notes

don't recompact log often for small programs

make the phony depfile edge order-only

I'm not sure this is semantically different but it's consistent with
how I think about these phony edges.

don't mangle absolute paths in the canonicalizer

We frequently do use absolute paths when depfiles refer to
e.g. /usr/include/stdio.h.

add extra check for no error in build test

update instructions on using gyp; mention cmake

simplify CommandRunner, simplifying users too

After much staring at this I think I found the more clear way to
express what it's doing.

rearrange build loop, pass new test