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.

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.

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.

Fixes typo

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.

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