Revert "ninja_syntax.py: Fix a bug when passing variables to Writer.build()"

This reverts commit 95ab954.
The test was verifying the wrong behavior.  I reviewed too quickly. :(

windows: more build fixes

Merge pull request #184 from evanj/master

ninja_syntax.py: Fix passing variables to Writer.build()

ninja_syntax.py: Fix a bug when passing variables to Writer.build()

Add a test to cover this case.

Add a space to unexpected token errors.

Use ExitProcess instead of exit in Fatal to allow extra cleanup for MSVC

windows: make bootstrap.py/configure.py work with MSVC

Merge pull request #180 from doctorlove/int64

Int64

Add new line to new file

typedef long long for MSVC

Merge pull request #179 from polrop/fix-clang-warning

Fix warning about class previously declared as struct.

Merge pull request #177 from doctorlove/msvc_changes

Msvc changes

Fix warning about class previously declared as struct.

Reported by clang++.

std::string(NULL,0) asserts in MSVC Debug

Changes to hash_map to make it build with MSVC's version

Include <algorithm> for std::find in test.cc

allow hyphen in variable names

more private

make evalstring internals private

Merge pull request #173 from polrop/fix-typo-in-manual

Fix typo in manual.

Fix typo in manual.

build fix

switch the core ninja parser to use re2c for the lexer

- Delete the old "Tokenizer" code.
- Write separate tests for the lexer distinct from the parser.
- Switch the parser to use the new code.
- New lexer error output has file:line numbers so e.g. Emacs can
  jump your editor to the syntax error.
- The EvalEnv ($-interpolation) code is now part of the lexer as well.

minor re2c cleanups

handle nuls more carefully in re2c rules

- '.' in re2c matches anything except \n, which means it matches \000.
  Be more careful about which characters we match.
- The fallback rule [^] reads ahead another character, which means it
  can read past the trailing \000.  Add a separate rule to match it
  specifically.

This was found by Valgrind.

add script for measuring build performance

ninja_syntax: add restat attribute to rules

Patch from Ami Fischman <fischman@chromium.org>.

add bash completion based on zsh completion

move some docs to home page

link to blog post in docs

add an "all" target to build all binaries

fix parser_perftest after depfile change

de-escape backslashes in depfiles while parsing

add (failing) test involving spaces in depfiles

switch DepfileParser to take a string* to make memory clearer

Add some comments as well.

fix depfile regex to not match empty strings

From a patch from Allan Odgaard <github@simplit.com>.

adapt parser_perftest to new depfile class

remove huge indent from tools list

rearrange the new bootstrap.py

(e.g. remove tabs)

Merge branch 'master' of git://github.com/ehird/ninja

Merge pull request #166 from tfarina/fix-commit

Follow up fix to commit 32bf74f (Fix the TODO in Rule class.)

Follow up fix to commit 32bf74f (Fix the TODO in Rule class.)

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

Merge pull request #164 from tfarina/rule-private-members

Fix the TODO in Rule class.

Fix the TODO in Rule class.

Move various data members to private section and provide accessors for them.

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

windows: handle ERROR_PATH_NOT_FOUND

From Frances <frances.buontempo@gmail.com>.

Merge pull request #163 from jsternberg/master

a fix for issue #162

Matching the depfile_parser.in.cc file to properly generate the depfile_parser.cc file that's in version control.

Merge pull request #161 from nico/fix

Let ninja_syntax handle escaped spaces correctly.

Let ninja_syntax handle escaped spaces correctly.

Revert the main loop changes made in 2e7ab7514207ea7faad40faedf3fc9d72b3adf7a,
and add just a few lines to the original main loop to make '$ ' escaping work.
Add several new tests, and make the existing tests pass again.

revert part of c33b198 to fix build of ninja_test

Merge pull request #160 from tfarina/upper-case-error

Capitalize "error" to match with the other utility functions.

Capitalize "error" to match with the other utility functions.

Fatal and Warning functions already output their strings capitalized,
we were just missing the Error function. So capitalize it now.

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

Merge pull request #159 from tfarina/protect-build-log-vars

Protect BuildLog data member variables by making them private.

Protect BuildLog data member variables by making them private.

They are not accessed outside of BuildLog and there is even a SetConfig function
to set the |config_| variable.

So better to make them private to BuildLog now while nobody is using it outside.

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

Merge pull request #158 from tfarina/alphabetize

Put the list of core source files in alphabetical order.

Put the list of core source files in alphabetical order.

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

windows: use GetFileAttributesEx instead of stat

From a Hacker News comment:
"Recent finding, that sped up our systems from 15->3sec on 300,000+ files
filestamp check was to move from _stat to GetFileAttributesEx."

I do recall reading that calls to stat() on Windows were one of the
potential reasons Subversion is so slow on Windows...

gitignore exe files

factor out windows perror equivalent

inline gtest to simplify windows build

fix bad merge

fix mingw cross-compile

rearrange manual to centralize quoting rules

remove weird accidental double-return

Surprised this isn't a compiler warning.

Merge pull request #146 from nornagon/escape-spaces

Escape spaces

adjust depfile test now that parsing is more lax

handle backslashes and isolated colons in depfile parser

The logic was wrong if the input looked like
  foo : bar baz
with a space before the colon.

Test from Frances <frances.buontempo@gmail.com>.

split out depfile parser tests into new file

print 'entering directory' when -C is used

This allows Emacs to track what directory you're in.

Patch from Ami Fischman <fischman@chromium.org>.

Rewrite the bootstrap script in Python

This allows trouble-free builds on platforms with non-standard Python
executable names. As a bonus, it passes the arguments received on to
configure.py, so you don't have to reconfigure to build in a
non-standard manner.

Support non-standard Python executable names

Closes #154.

Merge pull request #153 from polrop/fix-bootstrap

Fix bootstrap

Fix 'list' tool.

Factor source file listing code in bootstrap script.

Fix bootstrap script.

- Make regexp more robust.
- Exclude .in.cc files.

Use getter instead of private member.

Fix a compilation error.

performance testing note

remove makefile parsing code, use depfile code instead

use re2c to parse depfiles

refactor EdgeEnv

make Rule::generator_ and restat_ private

make Rule::name_ private

remove unused function

make Node::out_edges_ private

make Node::in_edge_ private

make Node::dirty_ private

more whitespace

merge StatCache into State

I think I had originally imagined StatCache would contain more state,
but at this point it's clear it was just managing a single map, which
could just as well be in the already-small State object.

merge FileStat into Node

The two were always one-to-one anyway.  I started adding accessors
to FileStat and then realized most users wanted them on Node and
that forwarding them through was silly.

refactor to remove Node::ready()

add and use getter for Edge::rule_

move build/install to README

Merge pull request #152 from pcc/clean-depfiles

Have the clean tool remove depfiles

Merge pull request #151 from pcc/restat-depfiles

Use the modification time of the depfile when computing restat_mtime

Have the clean tool remove depfiles

Use the modification time of the depfile when computing restat_mtime

Fixes issue #144.

Factor out Edge::EvaluateDepFile

more whitespace in build output

remove reference to nonexistent file

add help output to clean tool