Merge pull request #383 from nico/fix380

Fix issue #380.

Revert "Make StringPiece data members private."

This reverts commit 904c9610fe66c4f4bd63a07d6f057c8603d24394.
The commit caused issue #380, this revert fixes it. The revert
also makes the test from the previous commit pass.

Add a regression test for issue #380 (which fails at the moment)

Merge pull request #378 from syntheticpp/msvc-fixes

also build with msvc

also build with msvc

fix a warning and leak in windows-specific console output code

fix error and prevent warning in GetTimeMillis()

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

Merge pull request #352 from tfarina/string-piece-private

Make StringPiece data members private.

rename subprocess.cc to reflex its posixness

rearrange minidump patch to match ninja code style

Merge branch 'set_terminate' of git://github.com/gmoudry/ninja into minidump

(This likely doesn't compile, just getting all the history in place.)

Conflicts:
src/util.cc

Merge pull request #351 from syntheticpp/rate

print edges per second

make the build log test match the indended build log logic

As described on https://github.com/martine/ninja/pull/349 , this test
shouldn't have been passing.  Fixing a Valgrind issue showed that the
test was wrong.  Thankfully it wasn't a critical issue.

The new code verifies that we either succeed or report an error when
parsing -- the only behavior we really care about is not crashing.

Minor: zero log buffer before reading data/parsing it

Not a real problem, mainly to make valgrind happy

mingw: specify _WIN32_WINNT also in configure.py

fix some mingw warnings in getopt.c

From a patch from Claus Klein <claus.klein@arcormail.de>.

windows: fix printf format for process id

mingw: fix quoting of -DNINJA_PYTHON

bootstrap: set _WIN32_WINNT to WinXP for mingw

From a patch from Peter Kuemmel <syntheticpp@gmx.net>.

wrap at 80 columns

Merge pull request #369 from maximuska/proposed/packaging-rpm

packaging: added basic RPM building

packaging: added basic RPM building

Merge pull request #368 from nico/toolpipe

Don't print 'Entering directory' when running tools.

Don't print 'Entering directory' when running tools.

Merge pull request #367 from tfarina/lowercase

Use lowercase consistently.

Use lowercase consistently.

https://github.com/martine/ninja/issues/360

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

Merge pull request #366 from sgraham/no-noisy-cursor

windows: use WriteConsoleOutput instead of printf to avoid moving cursor

windows: use WriteConsoleOutput instead of printf to avoid moving cursor

Merge pull request #365 from maximuska/proposed/dep-file-parsing-more-specials

dep file parsing: allow '@' and '=' in paths

dep file parsing: allow '@' and '=' in paths

disallow crlf in manifest files

It turns out to be trickier than expected to process these correctly.
It turns out to also be trickier than expected to give a nice error
message on encountering these.  But the behavior prior to this patch
would just be silent failures where we attempted to examine paths
that accidentally contained embedded \r.

For now, fix all regexes of the form [^...] to include \r in the
excluded block, then assert that we get a vague lexer error near the
problem.

In the future perhaps we can open manifest files in text mode on Windows
or just disallow Windows-style CRLF in the manual.

explain mode: show which input is more recent

Patch from Scott Graham <scottmg@chromium.org>.

windows: get correct path for ninja_syntax

Patch from Scott Graham <scottmg@chromium.org>.

gitignore unpacked gtest

Patch from Scott Graham <scottmg@chromium.org>.

subprocess_test: make SetWithLots run on all non-Windows platforms

I think it might have been an oversight that this was marked
Linux-specific.

ninja_syntax: escape colons in paths

Merge pull request #340 from sgraham/fix-win-build

fix win32 compile, fix BuildLogTest.WriteRead on 2nd run

Merge pull request #363 from usovalx/subprocess_test_build_failure

Add missing include of <sys/resouce.h>

fix manual to reflect reality of depfile cleaning

Fixes issue #362.

Merge pull request #364 from tfarina/pragma-once

Remove #pragma once from our header files.

Remove #pragma once from our header files.

https://github.com/martine/ninja/issues/358

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

Add missing include of <sys/resouce.h>

GCC 4.7 needs cstdio

print edges per second

prints the rate of finished edges per second to the console,
for instance with NINJA_STATUS="[%s/%t %o(%c)/s] ":

[132/1922 16.1(14)/s]

16.1 is the average for all processed files (here 132 since start)
14 is the average of the last n files while n is the number specifies by
-j (or its default)

Merge pull request #361 from HeisSpiter/master

Check more efficiently for emptiness

Efficiently check for emptiness

Merge pull request #359 from tedv/old-arch

Fix missing POLLRDHUP constant on older systems.

Clarify comment wording.

Fix missing POLLRDHUP constant on older systems.

Attempting to compile with g++ 4.1.2 failed because the POLLRDHUP
constant was not defined when <poll.h> is included.

Merge pull request #357 from tfarina/manifest-parser

Rename parsers.* to manifest_parser.*

Rename parsers.* to manifest_parser.*

So it matches with the class name in there.

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

Merge pull request #313 from wolfp/restat_missing_input

Do not reset restat_mtime if an input is missing

Make StringPiece data members private.

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

Merge pull request #350 from maximuska/proposed/improve-dump-edge-node

Improving Edge::Dump, introducing Node::Dump

Improving Edge::Dump, introducing Node::Dump

- Edge::Dump could crash if called while inputs_ is being extended
- Node::Dump prints Node attributes, in-edge and lists of out-edges
- Dump functions now accept "prefix" parameter, printed along with the object
  for easier orientation. For example, edge->Dump("Re-reading deps files").

Add a test for missing input files in restat rules

Do not reset restat_mtime if an input is missing (issue #295)

remove unnecessary manual Close()

don't leak file handle on fail-to-upgrade case

fix win32 compile, fix BuildLogTest.WriteRead on 2nd run

switch build log to print hashes as hex

Nico convinced me that it's safe to do this without revving the build
log format: the old decimal values will still parse as hex (just as
different numbers) and cause a superfluous rebuild.

add copyright header

add new binaries to gitignore

drop support of log formats < 4

Log format 4 was introduced in January.
There's been plenty of time to migrate.

use PRIu64 for printing 64-bit in, %d for printing int

Fixes a warning.

make canon_perftest build, add license

Merge pull request #338 from syntheticpp/empty-log-file

no recompacting needed when log file is empty

no recompacting needed when log file is empty

Merge pull request #316 from tfarina/struct

No need to write 'struct' before type name when declaraing variables in ...

fix quoting thinko

rearrange shell quoting in configure.py

Merge pull request #333 from jonforums/no-main-multi-defs

Prevent multiple definition link error

Prevent multiple definition link error

Globbed inclusion of `src/hash_collision_bench.cc` cause link errors

Merge pull request #331 from sgraham/add-in_newline

Add $in_newline

Merge pull request #330 from sgraham/win32-hash-build

fix windows build

improve test

add $in_newline

fix windows build

Merge pull request #329 from nico/hash2

Only store command hashes in the build log.

Add a hash collision benchmark.

Only store command hashes in the build log.

.build_log load time 350ms -> 17ms, filesize 197MB -> 1.6MB on
Mac. On Windows, it's 500ms -> 20ms.

Makes the build log a lot less useful for scripts, but there could
be a tool for use cases that need log information. A prototype of
such a tool is in https://github.com/nico/ninja/commit/1b243d311

The hash function is 64bit murmurhash2. Assuming that that different
commands get the same hash only by chance, it's is very unlikely
for two different commands to hash to the same value with a 64bit
hash.

Merge pull request #328 from nico/canonperf

add canon_perftest

allow '(' and ')' in depfile paths

Fixes issue #327.

add a note to HACKING about how to get gtest

fix a typo in a test name

Merge pull request #325 from luvit/fix-path-magic

bootstrap: fix path magic on windows

bootstrap: fix path magic on windows

It was reported that argv[0] doesn't get set on windows. Some research
showed that using abspath(__filename__) should accomplish this.

Merge pull request #323 from luvit/bootstrap-from-cwd

bootstrap: chdir to bootstrap's directory

bootstrap: chdir to bootstrap's directory

When integrating ninja into luvit I naively tried calling:

    ./tools/ninja/bootstrap.py

This broke because bootstrap expects you to be in the ninja directory
when you call it. This patch makes it possible to run bootstrap.py like
above and have it work.

Merge pull request #306 from nico/winunbreakconf

Strip leading whitespace from commands on Windows.

Fix configure.py on windows, which does not permit a leading space.

Merge pull request #311 from nico/cleanups

Cleanups

add canon_perftest

Merge pull request #321 from nico/diff

Fix a bug that caused total_time to always be -1.

Remove code that was never executed.

total_time was always -1, because it computed a pointer difference,
not a time difference (should've been |*end_time - *start_time|).

Merge pull request #320 from nico/cleanup

Remove unused variable last_update_millis_.

Merge pull request #319 from nico/vimp

Make `:set spell` only check spelling in comments. Patch from Thilo Six ...

Remove unused variable last_update_millis_.

Make `:set spell` only check spelling in comments. Patch from Thilo Six <T.Six@gmx.de>!