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.

Fixes typo

Print output file on failure

Modify the FAILED: output to provide the output files that failed to
build, followed by the failed command on the next line.  This makes the
failure much easier to read, as you can immediately see much shorter
name of the file that failed instead of trying to parse a very long
command line.  It also makes manually re-running the failed command much
easier because you can copy the whole line without ending up with the
FAILED: prefix.

Fix typo.

Cleanup build on SIGHUP.

SIGHUP is sent when the connection hang up (i.e. when the terminal
window is closed or the ssh connection is closed).

Merge pull request #1014 from moroten/optimize-stringpiece

Optimize StringPiece hash map

Merge pull request #1018 from nico/spell

Merge LLVM r242069, make spell checking allocate less memory.

Merge LLVM r242069, make spell checking allocate less memory.

Optimize StringPiece hash map

Replace strncmp with memcmp to improve performance.

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

Describe why to use relative paths

Ninja does resolve relative paths and file system links in paths.
Therefore, such paths pointing to the same file will not match and may
lead to an invalid dependency graph.

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

Merge pull request #1009 from sgraham/fix-null.o

Don't output null.o during configure

Don't output null.o during configure

Fix getopt for AIX

AIX supplies getopt but not getopt_long.

We can't use the embedded getopt implementation, since the constness of its
arguments doesn't match the AIX system routine.

Implement GetLoadAverage on AIX using libperfstat

Fix AIX compilation error related to printf macros

On AIX, inttypes.h gets indirectly included by build_log.h.
It's easiest just to ask for the printf format macros right away.

Start AIX port

Strip tabs from od's output

/bin/od on Solaris and AIX both generate tabs.

Merge pull request #1003 from sgraham/2015-compile

Fix compile on VS2015

Remove accidental changes

Some mucking with std:: for set_terminate and terminate_handler

Set _HAS_EXCEPTIONS=0 on MSVC

Merge pull request #998 from LindleyF/stdc-format-macros

Make sure not to re-define __STDC_FORMAT_MACROS.

Make sure not to re-define __STDC_FORMAT_MACROS.

Print status when edge finishes on dumb terminals

On smart terminals ninja prints the status line both before
and after running a command, reusing the same line if possible.
On a dumb terminal that doesn't support reusing the line, it
only prints the status before starting the command, but prints
the output of the command when the command finishes, by which
point other commands may have started and printed their status
line.  This makes it impossible to determine what command
produced a line of output.

Modify BuildEdgeStarted to only print the status line if the
command is going to lock the console, or if ninja is running
on a smart terminal.  Modify BuildEdgeFinished to always
print the status line unless the command locked the console,
in which case the status was already printed and no other
command can have printed any lines.

The end result will be dumb terminal output that much more
closely matches smart terminal output.  One disadvantage is
that dumb terminals won't show anything when starting a
command, making it harder to tell what commands are currently
running, but I expect most interactive uses of ninja will use
a smart terminal.

Merge pull request #997 from mikesep/assert-arg-order

Correct order of ASSERT_EQ arguments

Correct order of ASSERT_EQ arguments

The correct order is ASSERT_EQ(expected, actual).

Merge pull request #988 from nico/slashes

win: print right slashes in 'unknown target' message

win: print right slashes in 'unknown target' message

Merge pull request #973 from sgraham/canonicalize-fix

Fix crash in attempting to canonicalize paths longer than _MAX_PATH

Merge pull request #979 from stinb/bootstrap-out-of-source

Bootstrap out of source

fix typos, punctuation, redundant words in HACKING.md

Merge pull request #984 from martine/vcvars

point people at more detailed build docs if they're on Windows

point people at more detailed build docs if they're on Windows

It's a hard balance between "most people want to run configure --bootstrap" and
"here's a massive wall of text of every detail about building".  I think the
balance in this change is better than it was before -- most people on Windows
don't want to build their own binary.

Fixes #983.

v1.6.0

mark this 1.6.0.git

Fix bootstrap from a source path containing spaces.

Search for generated headers relative to build dir.

Allow configure script to bootstrap out of source.

Merge pull request #978 from martine/revert-971-bootstrap-out-of-source

Revert "Bootstrap out of source"

Revert "Bootstrap out of source"

Merge pull request #975 from nico/docs

Document the change from #921 in the manual.

Document the change from #921 in the manual.

Merge pull request #971 from stinb/bootstrap-out-of-source

Bootstrap out of source

Merge pull request #970 from nico/explain

Add a missing EXPLAIN() call.

don't alias input/output in ExtractDeps (i.e. actually works now)

propagate include normalization failure to caller instead

Error+exit -> Fatal

Merge pull request #974 from bradking/add-missing-initializer

Add missing member initializer in Edge constructor

Add missing member initializer in Edge constructor

In commit v1.1.0^2~15^2~20 (stub out an api and de-constify Pool,
2012-10-03) the Edge "pool_" member was added as a raw pointer but the
initializer was accidentally left out of the constructor.  Add it now.

don't abort()

push error to caller and abort on failure

Fix crash in attempting to canonicalize paths longer than _MAX_PATH

Search for generated headers relative to build dir.

Allow configure script to bootstrap out of source.

Add a missing EXPLAIN() call.

Fix typo in comment.

Merge pull request #962 from sgraham/pool-use-fix

Fix pool use count going unbalanced

Merge pull request #743 from nicolasdespres/sigterm

Handle SIGTERM properly.

simplify & inline

avoid calling ResumeDelayedJobs instead

add comment

Forward interruption signal to child processes.

Allow SIGTERM for interruption.

Default signal sent by many other programs (mainly kill(1)) to gently
terminates another one is SIGTERM.

Fix pool use count going unbalanced

Merge pull request #958 from rui314/master

Run more than 34 processes on Win32 if we have 32+ cores.

Run more than 34 processes on Win32 if we have 32+ cores.

For compatiblity reason, dwNumberOfProcessors in Win32 is capped at 32.
So even if your machine has more than 32 cores, Ninja spawns at most 34
subprocesses. This patch fixes the issue by using GetNativeSystemInfo,
which returns the system info from Wow64 point of view, instead of
GetSystemInfo.

Merge pull request #954 from nico/fix

Fix an assert (and tests in --debug mode) after #921.

Fix an assert (and tests in --debug mode) after #921.

Merge pull request #953 from Sushisource/graphivz_fix

Fix backslashes in graphviz causing incorrect rendering on windows.

Fix backslashes in graphviz causing incorrect rendering on windows.

Merge pull request #951 from nico/cyclefix2

Don't get stuck on cyclic graphs where one edge has multiple outputs.

Don't get stuck on cyclic graphs where one edge has multiple outputs.

Fixes #934. Plan::AddSubTarget() tracks in want_ if each edge has been
visited and visits every edge only once.  But Plan::CheckDependencyCycle()
worked on nodes however, so if a cycle was entered through an edge with
multiple outputs, ninja would fail to detect that cycle.

Move cycle detection to look for duplicate edges instead of nodes to fix
this.  The extra jump makes CheckDependencyCycle() a bit slower: for a
synthetic build file with 10000 serial build steps, `ninja -n` now takes
0.32s instead of 0.26s before on my laptop.  In practice, projects have
a dependency change length on the order of 50, so there shouldn't be a
noticeable slowdown in practice.  (If this does end up being a problem:
CheckDependencyCycle() currently does O(n) work and is called O(m) times
from AddSubTarget(), so I think cycle checking is O(n^2) in the build
depth.  So instead of worrying about constant overhead, we could use
a set<> instead of a stack<>.  But it doesn't seem to be a problem in
practice.)

Merge pull request #950 from nico/cleanup

Simplify Plan's cycle detection code. No behavior change.

Cleanup: Don't search stack for cycle elements twice.

The common case is that there is no cycle.  In that case,
CheckDependencyCycle() searched the stack for a dupe from the back,
wouldn't find one, and return false.

If there was a cycle, it would then search again from the front
(probably because the push_back() that used to be here would invalidate
the ri iterator).

Since the push_back() is gone, just search from the front once. No
intended behavior change.

Cleanup: Make stack a const ref now that it's no longer modified.

Cleanup: Don't modify  during cycle checking.