Improve Plan::EdgeFinished signature

Use an enumeration instead of a boolean to clarify the purpose of
arguments at call sites.

Suggested-by: Nico Weber <nicolasweber@gmx.de>

Merge pull request #1126 from bradking/pool-release-on-fail

Release the pool slot held by an edge whether it succeeds or fails

Merge pull request #1130 from JohnstonJ/fix-tests-on-vs2015

ninja_test: Fix Visual C++ 2015 warnings/errors about name hiding.

Merge pull request #1059 from bradking/avoid-double-scheduling-edge

Avoid double-scheduling build edges in another case

ninja_test: Fix Visual C++ 2015 warnings/errors about name hiding.

Visual C++ 2015 warns if a local variable hides visibility of another variable
in a higher scope.  Since this project declares warnings as errors, ninja_test
simply won't build on Visual C++ 2015.

The variables have been renamed and scope limited as appropriate, so that
ninja_test will build without error now on Visual C++ 2015.

Add another test case covering pool release on edge failure

With this build file:

    pool failpool
      depth = 1
    rule fail
      command = fail
      pool = failpool
    build out1: fail
    build out2: fail
    build out3: fail
    build final: phony out1 out2 out3

Running `ninja -k 0` should run out1..3 sequentially before failing, but
until recently we would fail after just running out1.  Add a test
covering this case.

Release the pool slot held by an edge whether it succeeds or fails

When an edge finishes building, it should be release from its pool.
Make sure that this also is the case when an edge fails to build.

The bug can be shown with a pool has size N, then `ninja -k N+1` will
still stop after N failing commands for that pool, even if there are
many more jobs to be done for that pool:

    pool mypool
      depth = 1
    rule bad_rule
      command = false
      pool = mypool
    build a : bad_rule
    build b : bad_rule

Current behaviour:

    $ ninja -k 0
    [1/2] false
    FAILED: false
    ninja: build stopped: cannot make progress due to previous errors.

Expected behaviour:

    $ ninja -k 0
    [1/2] false
    FAILED: false
    [2/2] false
    FAILED: false
    ninja: build stopped: cannot make progress due to previous errors.

Signed-off-by: Fredrik Medley <fredrik.medley@gmail.com>

Merge pull request #1125 from nico/wintools

Make deps=msvc experimentally available on non-Windows.

Make deps=msvc experimentally available on non-Windows.

This makes it possible to run most of the clparser tests on non-Windows,
and is potentially useful for cross-compiling on non-Windows hosts.
Also, the manual didn't document this as Windows-only previously.

If you use this on non-Windows, please let me know, else I might undo
this change again in the future.

Merge pull request #1118 from nico/posixspawn2

Use posix_spawn() instead of fork()/exec().

Merge pull request #1112 from Lekensteyn/browse-tool-fixes

browse tool: Allow customization of port number, Py3 fix

Use posix_spawn() instead of fork()/exec().

posix_spawn() is a syscall on OS X and Solaris and a bit faster.  It's
also easier emulate for cygwin, and the code is a bit simpler.

browse: support --port and --no-browser options

Add --port option to override the default port (8000).
Add --no-browser option to avoid opening a web browser (useful over
SSH).
Make the target name optional, using "all" if omitted.

browse.py: Python 3 compatibility

Merge pull request #1113 from nico/nosetsid

Revert #910.

Revert #910.

The change caused some issues (it makes it impossible ot use
posix_spawn() and makes it harder to suspend children on ctrl-z).  After
discussing with jln: Since it fixes a corner case that can be fixed by
explicitly running commands that need it in a wrapper that setsid()s
them, let's try reverting it for a while.  Please shout if this is a
problem for you.

See also #1097.

Merge pull request #1102 from bradking/remove-unnecessary-StatIfNecessary

Remove StatIfNecessary call that is never necessary

browse.py: allow port override via environment variable

Remove StatIfNecessary call that is never necessary

The call to StatIfNecessary in DependencyScan::RecomputeOutputsDirty was
added by commit v1.4.0^2~7^2~1 (Share more code between CleanNode() and
RecomputeDirty(), 2013-09-02) while consolidating code paths.  However,
it was needed only when called from RecomputeDirty because prior to
refactoring the CleanNode code path did not call it.

Later commit v1.6.0^2~46^2 (Let DependencyScan::RecomputeDirty() work
correclty with cyclic graphs, 2014-12-07) added back to RecomputeDirty a
loop over outputs that calls StatIfNecessary.  Therefore
RecomputeOutputsDirty no longer needs to call StatIfNecessary for either
of its own callers.

Merge pull request #1110 from G4m4/explicit_std_prefix

Fix ambiguous call to set_terminate on Windows platform

Fix ambiguous call to set_terminate on Windows platform

On Windows set_terminate() could either be the standard C++ one or (actually the same one but in the global namespace) the CRT one declared in corecrt_terminate.h
Hence this ambiguity - this patch solves it.

Signed-off-by: g4m4 <misept.dieseneuf@gmail.com>

Merge pull request #1107 from nicolasdespres/ninja_syntax_implicit_outputs

Add support for implicit outputs in ninja_syntax.py.

Merge pull request #1108 from edbaunton/patch-1

Add link to mailing list

Add link to mailing list

As a convenience to new users, provide a link to the mailing list.

Add support for implicit outputs in ninja_syntax.py.

Merge pull request #1106 from nico/winfix

win: Only drop /showIncludes in bootstrap builds.

win: Only drop /showIncludes in bootstrap builds.
\vFixes #1103.

Merge pull request #1101 from fbenkstein/vs2010

disable warning about conditional expressions

MSVC: disable warning about constant conditional expressions

The standard headers for Visual Studio 2008 generate a warning about
constant conditional expressions when compiled with exception support
disabled.  This is caused by the _CATCH_ALL macro in xstddef which is
defined thusly:

  #ifdef _HAS_EXCEPTIONS
  #define _CATCH_ALL } catch (...) {
  #else
  #define _CATCH_ALL } if (0) {
  #endif

Merge pull request #1100 from nico/doc

Minor updates to the manual.

Minor updates to the manual.

* Update link to Chromium's ninja docs (fixes #1038)
* Update cmake URL to what it redirects to, and mention that ninja
  is well-supported on all platforms in newer CMake versions.
* Let "others" link to the wiki page listing generators.

Avoid double-scheduling build edges in another case

The change in commit v1.2.0~3^2~3^2~3 (Fix duplicate edge Pool crash in
the minimally invasive way, 2013-03-18) avoids double-scheduling in a
case involving duplicate out edges.  However, double-scheduling may also
occur on a consistent graph when an edge and one of its dependencies
share an order-only input:

    $ cat build.ninja
    ...
    build c: touch
    build b: touch || c
    build a: touch | b || c
    $ ninja a
    $ rm a c
    $ ninja a

In this case 'c' will build first.  When NodeFinished('c') loops over
the out edges it will find AllInputsReady is true for 'b' and call
EdgeFinished('b') since it is not wanted (up to date).  This will
call NodeFinished('b') which will loop over its out edges, find
AllInputsReady is true for 'a', and call ScheduleEdge('a').  When
we eventually return to the loop in NodeFinished('c') it will move
on to its second output and find that AllInputsReady is true for
'a' and call ScheduleEdge('a') again.

Teach ScheduleEdge to tolerate duplicate calls for an edge that has
already been scheduled.  Avoid calling EdgeScheduled more than once
for the same edge.

Merge pull request #1033 from colincross/failed

Print output file on failure

Merge pull request #989 from bradking/implicit-outputs

Add support for build statement implicit outputs

Merge pull request #1060 from bradking/deduplicate-disk-interface

Deduplicate disk abstraction infrastructure

Merge pull request #1099 from bradking/fix-perftest-compilation

perftest: fix compilation after dupe_edge_should_err change

Replace ManifestParser::FileReader with general FileReader

Avoid having two separate filesystem interfaces.  Simplify test
infrastructure by avoiding custom `ManifestParser::FileReader`
implementations.

Expose more details in FileReader::ReadFile signature

Return a status so callers can distinguish a missing file from an empty
file.  This allows our VirtualFileSystem test infrastructure to report
as missing any file for which it has no entry.

Factor a `FileReader` base class out of `DiskInterface`

Some clients will need only the ability to read files, so provide this
as a more narrow interface than the full disk interface.

Add support for build statement implicit outputs

Some build rules produce outputs that are not mentioned on the command
line but that should be part of the build graph.  Such outputs should
not be named in the `$out` variable.  Extend the build statement syntax
to support specification of implicit outputs using the syntax
`| out1 out2` after the explicit outputs and before the `:`.

For example, compilation of a Fortran source file `foo.f90` that defines
`MODULE FOO` may now be specified as:

    rule fc
      command = f95 -c $in -o $out
    build foo.o | foo.mod: fc foo.f90

The `foo.mod` file is an implicit output generated by the compiler based
on the content of the source file and not mentioned on the command line.

Merge pull request #1076 from colincross/canonicalize

Canonicalize "." to "."

perftest: fix compilation after dupe_edge_should_err change

Fix some ManifestParser constructor calls missed by commit 56bab441b7
(dupe_edge_should_err from bool to enum, 2016-01-27).

Merge pull request #1096 from sgraham/dupe-edge-action

dupe_edge_should_err from bool to enum

Merge pull request #1098 from shinh/keepdep

Add -d keepdepfile to preserve depfiles

Add -d keepdepfile to preserve depfiles

This is useful when you are developing a tool which generates
GCC-style depfiles.

dupe_edge_should_err from bool to enum

Merge pull request #1095 from sgraham/dupbuild-err

Make dupbuild=err work in subninja

Make dupbuild=err work in subninja

Merge pull request #1094 from sphawk/master

fix for localized version of cl.exe

Merge pull request #1085 from tetsuok/remove-unnecessary-std

Remove unnecessary `std::`

fix for localized version of cl.exe

Merge pull request #1086 from tetsuok/use-container-based-infra-for-ci

Use container-based infrastructure for travis-ci.

Use container-based infrastructure for travis-ci.

To start up faster.

Remove unnecessary `std::`

Merge pull request #1073 from ninja-build/manual-font

use the default font size for manual headings

Merge pull request #1074 from ninja-build/manual-command

add a section to the manual discussion the command= variable

Merge pull request #1082 from nicolasdespres/zshcomp-intermediary-targets

Teach zsh to complete intermediary targets.

Teach zsh to complete intermediary targets.

Bash completion script uses "-t targets all" to list the target which
is faster than "-t targets" and reports intermediary targets
(see the manual entry for the 'targets' tool).

See commit fc135c45.

Canonicalize "." to "."

"." is a legal path, if the string is empty after canonicalization
return ".".

add a section to the manual discussing the command= variable

This includes a mention of using cmd /c on Windows.
This would have helped on issue #1070 for example.

use the default font size for manual headings

The third-level subsection headings were almost indistinguishable
from the second-level ones.  Fix this by just using the default
styling.

Merge pull request #1069 from nicolasdespres/dry_manifest_regen

Do not attempt to rebuild infinitely in dry-run mode.

Do not attempt to rebuild infinitely in dry-run mode.

Merge pull request #1042 from avallee/master

Fix wrapping file paths in Writer.comment

Merge pull request #1062 from tfarina/bindings-typedef

make use of Bindings typedef

Merge pull request #1063 from ptzz/master

Fix typo

Merge pull request #1064 from NiklasRosenstein/1043-msvc_deps_prefix-per-rule

add "msvc_deps_prefix" to Rule::IsReservedBinding()

add "msvc_deps_prefix" to Rule::IsReservedBinding()

fixes the error about an unexpected variable for a rule that
declares the msvc_deps_prefix. The manual suggests that this
should work since Ninja 1.5
(https://ninja-build.org/manual.html#ref_rule).

Closes #1043

Fix typo

make use of Bindings typedef

Looks like we declared this typedef but never used it.

It seems we just forgot to use it so this patch makes uses of it now.
Otherwise we could just delete it.

Disable long word wrapping entirely in comments.

As pointed out by nico, we should unconditionally disable breaking of
long words in comments. It is unlikely long words that are in comments
should be split (like pathnames).

prefer https in readme

Merge pull request #1053 from ninja-build/configure

Minor tweaks to the configure script

when building with cwd = source root, use relative paths

The pull request that introduced building from a directory other
than the source dir, https://github.com/ninja-build/ninja/pull/979,
made it so all source paths are effectively absolute paths.  This
change restores the old behavior in the case when you are building
in the source.  See the comments there.

rename the build-file variable $sourcedir to $root

Confusingly, $sourcedir is the path to the root of the repository,
*not* the path to the src/ directory.

put warning flags together

Merge pull request #1052 from ninja-build/docs

Minor doc fixes

drop a [horizontal] definition table

The two columns of the table run together, making it hard to read.

improve docs generation, including PDF output

- Fix the manual build rules (missing the .xsl as an input).
- Add a README describing how the docs build works.
- Add rules that generate PDF, just 'cause we can.

provide a better title for the manual

In particular, this helps web search engines index it.

Merge pull request #1047 from pykello/master

Speed-up bash target auto-complete.

Merge pull request #1046 from zed0/master

Speed up zsh target completion

Fix indent in pull request #1042.

Merge pull request #1007 from mikesep/aix

Support for AIX

Merge pull request #999 from colincross/dumbstatus

Print status when edge finishes on dumb terminals

Merge pull request #964 from nicolasdespres/sighup

Cleanup build on SIGHUP.

Merge pull request #763 from drbo/master

Allow paths with international characters in depfiles

Try to fix #685

drop more references to github/martine

Update link to point to ninja-build.org

Merge pull request #1049 from nico/update-links

Make links point to org page

Make links point to org page

test commit

Speed-up bash target auto-complete.

These speed-ups include:

* Let compgen do the command substitution. Similar to
  https://lists.gnu.org/archive/html/bug-bash/2012-03/msg00115.html
* Use "cut" instead of "awk" for separating fields.

Speed up zsh target completion

Fix wrapping file paths in Writer.comment

Long file names, especially with hyphens will get incorrectly wrapped by
the comment method. Pass has_path=True to prevent this type of wrapping.

This is mainly so that longer path names can show up in comments on
their on line without breaking them up.

Separate bootstrapped build from final build

AIX does not support rebuilding ninja in-place from the bootstrapped ninja.

Merge pull request #1015 from moroten/docs-relpath

Describe why to use relative paths

Merge pull request #1040 from peter1000/patch-1

Fixes typo

Merge pull request #1029 from nicolasdespres/fix-doc-typo

Fix typo.