fix SEGV in graph_test

reduce doxygen output

add doxygen-compatibile comments to most classes

fix a signedness warning

Merged pull request #31 from polrop/generate-whole-graph.

The 'graph' tool now generates a graph based on all root nodes when called without any target. The build.ninja file is also adjusted so that we get the whole graph generated (including the doxygen part).

Merged pull request #30 from polrop/more-tools.

adds the -C option and the following tools: targets, rules and clean

Merged pull request #29 from polrop/minor-bug-fix.

Minor bug fix

include location of error in parse error messages in EvalEnv strings

E.g. when parsing "foo = ${bar", point at the correct location of
the missing curly brace.

Generate the entire graph not only for the 'all' target.

This way we can see the 'doxygen' target and also some object file we
forgot to add as input of the link rule.

Add the 'clean' tool.

It removes built files either by rule or target or everything.

graph: Assume all root nodes if no target given.

This way we can generate the complete graph.

Add the 'rules' tool.

It can be useful in conjunction with the 'targets' tool for writing shell
completion scripts and when debugging.

Add the 'targets' tool.

This tool list targets by depth or by rule. It can be useful:
- for shell completion script ;
- to know what are the primary targets: ninja -t targets depth 1 ;
- to know targets that are linked: ninja -t rule link ;
- when debugging.

It works by first listing the root nodes and then following the input nodes
of their in edge.

Add -C DIR option.

Can be useful when calling ninja from a script or a code editor or in many
other occasions.  It costs nothing and does not bring more complexity, so
I think we can afford it.

graph: Report errors if a target does not exist.

Do not always require a target when a tool is run.

Adjust tools options parser and documentation.

Remove no longer used current working directory.

It has been introduced by 761a3c when ManifestParser::set_root() existed.
It is no longer the case so we can save some time querying it now.

Document.

Remove useless inclusion of stdio.h.

Use Error() to report error.

This way we get the error message prefixed by the program name. It helps
while debugging.

Prefix perror(3) messages with program name.

It make it easier while debugging to know who is reporting the error:
Ninja itself or one of the command called by Ninja during the build
process or one of the generator which called Ninja.

don't close null log files

Close() and unlink() files in ways that make Windows happy

This will make the build log compaction work on Windows.
It will also make the tests no longer leave a temp log file around.

[windows] disable smart terminal support

[windows] disable browse tool on Windows for now

[windows] use local getopt.h

[windows] get processor count using Windows API

[windows] extend hash_map.h for Windows hash_map

[windows] fix class/struct confusion

[win] check in public-domain getopt implementation

use kMaxPathComponents in proper place

optimize CanonicalizePath

Null build of Chrome:
  before I added extra calls to CanonicalizePath: 1.25s.
  with extra calls to CanonicalizePath: 1.35s.
  with new CanonicalizePath: 1.05s.

And now CanonicalizePath isn't hot on profiles anymore.

fix typo in comment in bootstrap.sh

Merge branch 'doxygen'

canonicalize paths on the command-line as they're used

I'm not entirely happy with my solution.  I think Ninja should
always use canonicalized paths internally, so we should canonicalize
all paths as we receive them (from the command line and from build
files).  But there are other places where we pass paths around
(like in all the tool commands) and I don't want to add manual calls
to canonicalize in each.

Perhaps the longer-term solution is to add some sort of
GetNodeCanonicalize() to State and then make all of the functions that
work with paths as strings (like Builder::AddTarget or RunBrowsePython)
instead only accept a Node*.

use util's CanonicalizePath in parsers as well

handle foo//bar in CanonicalizePath

actually link in util_test

doxygen_mainpage is not an implicit dependency.

This way it does not appear in the $in variables.

Adjust DOXYGEN_MAINPAGE rule description.

Now it prints its output file.

Touching README and co triggers rebuild of doxygen.

Before the main page was an order-only dependency of the doxygen target.
Thus, modification in the README did not triggers rebuild of the doxygen
target as it ought to be.

We have several ways to fix this issue:

1) Make the '$builddir/doxygen_mainpage' target an explicit dependency:

build $builddir/doxygen_mainpage: doxygen_mainpage \
  README HACKING COPYING | $doxygen_mainpage_generator
build doxygen: doxygen doxygen.config $builddir/doxygen_mainpage

2) Duplicate the explicit dependencies of the '$builddir/doxygen_mainpage'
   target as implicit dependencies of the 'doxygen' target and keep the
   '$builddir/doxygen_mainpage' target as an order-only dependency:

build $builddir/doxygen_mainpage: doxygen_mainpage \
  README HACKING COPYING | $doxygen_mainpage_generator
build doxygen: doxygen doxygen.config \
  | README HACKING COPYING \
  || $builddir/doxygen_mainpage

I chose the first option since the doxygen executable only cares about its
first argument.  But it would not be the case for all commands.  So I
think introducing "slice" support for the $in and $out variables would be
great.  This way I could rewrite the 'doxygen' rule this way:

rule doxygen
  command = doxygen $in[0]
  description = DOXYGEN $in[0]

Note that using variables to avoid duplication does not work since the
three files are seen as a single one:

doxygen_deps = README HACKING COPYING

build $builddir/doxygen_mainpage: doxygen_mainpage \
  $doxygen_deps | $doxygen_mainpage_generator
build doxygen: doxygen doxygen.config \
  | $doxygen_deps \
  || $builddir/doxygen_mainpage

Maybe Ninja's philosophy expects the generators to generate the second
option.

Prefix error messages by script name.

It helps when debugging compilation output to know who is speaking even if
Ninja buffers command output contrary to make.

Make include_file() never returns 1.

Otherwise the script is stopped because errexit option is set.

Call error() not fatal() if file is not found.

Add missing Apache license header.

Reviewed by Evan Martin.

Fix gitignore line for Doxygen.

Reviewed by Evan Martin.

Merge branch 'term-fix' of https://github.com/SByer/ninja

Add doxygen support.

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

Conflicts:
src/util.cc
src/util.h

Remove the declaration of DumpBacktrace.

This function was removed on 01880fb3a2a13f9071e9729c3a13752846828ed2.

Move CanonicalizePath into util.h so it can be shared by the other modules.

Also add util_test.cc and move the CanonicalizePathTest into there.

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

Fix an exception when terminal is narrow or set for unlimited width

Merge branch 'errors-reporting' of https://github.com/polrop/ninja

Add missing CPP guards.

Prefix error messages with program name.

It make it easier while debugging to know who is reporting the error:
Ninja itself or one of the command called by Ninja during the build
process or one of the generator which called Ninja.

Add link to mailing list to README.

src/inline.sh: fix typo

I (Evan) got this wrong in the previous commit.

inline.sh: BSD / OS X fixes

fix test that relied on pointer values

make bad build tests fail earlier

fix terrible bug in gyp patch

fix bootstrap when build dir already exists

move the implementation of GraphViz to source file

update gyp patch

fix bootstrap by running python inliner too

inline.sh: rewrite for better portability

handle variable expansion with upper-case characters

use a shell script to inline browse.py

Relying on the inline-assembly trick was cute but it didn't work on
other platforms; relying on xxd will make us depend on xxd being
available.  Instead, inline browse.py into a header using a shell
script.

(Making this work required fixing multiple bugs in ninja...)

files that have both implicit and explicit edges should be implicit

This is just deleting some code that was written in anticipation
of staying memory-resident; when loading a depfile, we don't need
to attempt to update existing entries in the dependency list, we
just need to blindly add them.

canonicalize paths loaded from depfiles

If a C file #includes "../foo.cc", then gcc will emit paths like
"bar/../foo.cc" into the dependency file; canonicalize these when we
load the file.

Add a test module for testing the graph dirty recomputation directly,
without all the build classes around it.

more comments

move VirtualFileSystem into test.*

move test support into test.*

don't rely on /proc for 'browse' mode

Instead, pass the script code via stdin.
We end up with a zombie (the child process that passes the script code)
because we can't ignore SIGCHLD in the parent (the Python code relies on
being able to wait for children), but the zombie dies with the outer
process.  This probably could be avoided by double-forking or other
approaches, but it doesn't seem worth the effort.

factor browse out into its own file

chdir into a temp dir before disk tests

drop use of ar's T flag

This produces smaller archives, which is important for large projects
to not clobber your disk cache.  But we are small (the resulting .a is
~3.5mb for debug) and it's not supported on older operating systems.

limit output width to prevent line-wrapping

linux: simpler way to get number of processors

From a conversation with Anatol Pomazau <anatol@google.com>.

print short name of command before dumping command stdout/stderr

Previously we showed the full command line, but with this change
we only show the full command line in verbose mode or when the
command fails.

drop DumpBacktrace

It wasn't that helpful; simplifies other platforms to just not use it.

update gyp patch

(Needed to fix Chrome build.)

get the number of CPUs on FreeBSD and OS X via sysctl()

From a patch from asmodai@in-nomine.org and dmitry@codingrobots.com.

bootstrap.sh: FreeBSD support

doc phony actions

check off todo

Fix various typos.

Merge branch 'master' of https://github.com/geophree/ninja

Merge branch 'master' of neugierig.org:projects/git/ninja

quick-fix gyp.patch for chrome change

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

Fix a typo in the manual.

add copyrights

clean up bootstrap

create builddir on start; fixes bootstrap

use -f for input file, matching make

check in demo file

check off todo item

include command name when printing output, add newline

fancier output