Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | move processor-count code to util.cc | Evan Martin | 2012-07-28 | 3 | -25/+35 |
| | |||||
* | note that -l doesn't work on windows | Evan Martin | 2012-07-28 | 2 | -0/+4 |
| | | | | Filed issue #386 about it too. | ||||
* | simplify load-average code | Evan Martin | 2012-07-28 | 1 | -15/+3 |
| | |||||
* | delete vestigal declarations | Evan Martin | 2012-07-28 | 1 | -7/+0 |
| | |||||
* | Disable SubprocessTest.TestWithLots on OS X. | Nico Weber | 2012-07-28 | 1 | -2/+4 |
| | | | | It fails on OS X, see issue #384. | ||||
* | Revert "Make StringPiece data members private." | Nico Weber | 2012-07-28 | 8 | -19/+14 |
| | | | | | | 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) | Nico Weber | 2012-07-28 | 1 | -0/+19 |
| | |||||
* | also build with msvc | Peter Kümmel | 2012-07-27 | 3 | -5/+10 |
| | |||||
* | fix a warning and leak in windows-specific console output code | Evan Martin | 2012-07-27 | 1 | -2/+2 |
| | |||||
* | fix error and prevent warning in GetTimeMillis() | Claus Klein | 2012-07-27 | 2 | -2/+2 |
| | |||||
* | Merge branch 'master' of github.com:martine/ninja | Evan Martin | 2012-07-27 | 8 | -14/+19 |
|\ | |||||
| * | Merge pull request #352 from tfarina/string-piece-private | Evan Martin | 2012-07-27 | 8 | -14/+19 |
| |\ | | | | | | | Make StringPiece data members private. | ||||
| | * | Make StringPiece data members private. | Thiago Farina | 2012-06-30 | 8 | -14/+19 |
| | | | | | | | | | | | | Signed-off-by: Thiago Farina <tfarina@chromium.org> | ||||
* | | | rename subprocess.cc to reflex its posixness | Evan Martin | 2012-07-27 | 1 | -0/+0 |
| | | | |||||
* | | | rearrange minidump patch to match ninja code style | Evan Martin | 2012-07-27 | 3 | -113/+132 |
| | | | |||||
* | | | Merge branch 'set_terminate' of git://github.com/gmoudry/ninja into minidump | Evan Martin | 2012-07-27 | 3 | -0/+121 |
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | (This likely doesn't compile, just getting all the history in place.) Conflicts: src/util.cc | ||||
| * | | Changed #ifdef _WIN32 to #ifdef _MSC_VER to make sure that MinGW compiles ↵ | Jiri Moudry | 2012-04-26 | 2 | -3/+3 |
| | | | | | | | | | | | | happily | ||||
| * | | Improve handling of fatal errors on Windows, support creation of minidumps | Jiri Moudry | 2012-03-28 | 3 | -0/+120 |
| | | | |||||
* | | | Merge pull request #351 from syntheticpp/rate | Evan Martin | 2012-07-27 | 9 | -17/+101 |
|\ \ \ | | | | | | | | | print edges per second | ||||
| * | | | GCC 4.7 needs cstdio | Peter Kümmel | 2012-07-15 | 1 | -1/+1 |
| | | | | |||||
| * | | | print edges per second | Peter Kuemmel | 2012-07-15 | 9 | -16/+100 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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) | ||||
* | | | | make the build log test match the indended build log logic | Evan Martin | 2012-07-27 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | Maxim Kalaev | 2012-07-27 | 1 | -1/+3 |
| | | | | | | | | | | | | | | | | Not a real problem, mainly to make valgrind happy | ||||
* | | | | fix some mingw warnings in getopt.c | Evan Martin | 2012-07-27 | 1 | -6/+2 |
| | | | | | | | | | | | | | | | | From a patch from Claus Klein <claus.klein@arcormail.de>. | ||||
* | | | | windows: fix printf format for process id | Evan Martin | 2012-07-27 | 1 | -1/+1 |
| | | | | |||||
* | | | | wrap at 80 columns | Evan Martin | 2012-07-27 | 1 | -2/+5 |
| | | | | |||||
* | | | | Don't print 'Entering directory' when running tools. | Nico Weber | 2012-07-25 | 1 | -1/+4 |
| | | | | |||||
* | | | | Use lowercase consistently. | Thiago Farina | 2012-07-20 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/martine/ninja/issues/360 Signed-off-by: Thiago Farina <tfarina@chromium.org> | ||||
* | | | | Merge pull request #366 from sgraham/no-noisy-cursor | Evan Martin | 2012-07-20 | 1 | -6/+20 |
|\ \ \ \ | | | | | | | | | | | windows: use WriteConsoleOutput instead of printf to avoid moving cursor | ||||
| * | | | | windows: use WriteConsoleOutput instead of printf to avoid moving cursor | Scott Graham | 2012-07-19 | 1 | -6/+20 |
| | | | | | |||||
* | | | | | dep file parsing: allow '@' and '=' in paths | Maxim Kalaev | 2012-07-19 | 3 | -17/+30 |
|/ / / / | |||||
* | | | | disallow crlf in manifest files | Evan Martin | 2012-07-18 | 3 | -53/+81 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | Evan Martin | 2012-07-17 | 1 | -1/+4 |
| | | | | | | | | | | | | | | | | Patch from Scott Graham <scottmg@chromium.org>. | ||||
* | | | | subprocess_test: make SetWithLots run on all non-Windows platforms | Evan Martin | 2012-07-17 | 1 | -4/+6 |
| | | | | | | | | | | | | | | | | | | | | I think it might have been an oversight that this was marked Linux-specific. | ||||
* | | | | Merge pull request #340 from sgraham/fix-win-build | Evan Martin | 2012-07-17 | 2 | -0/+3 |
|\ \ \ \ | | | | | | | | | | | fix win32 compile, fix BuildLogTest.WriteRead on 2nd run | ||||
| * | | | | remove unnecessary manual Close() | Scott Graham | 2012-06-19 | 1 | -1/+0 |
| | | | | | |||||
| * | | | | don't leak file handle on fail-to-upgrade case | Scott Graham | 2012-06-19 | 1 | -0/+1 |
| | | | | | |||||
| * | | | | fix win32 compile, fix BuildLogTest.WriteRead on 2nd run | Scott Graham | 2012-06-19 | 2 | -0/+3 |
| | | | | | |||||
* | | | | | Merge pull request #363 from usovalx/subprocess_test_build_failure | Evan Martin | 2012-07-17 | 1 | -0/+5 |
|\ \ \ \ \ | | | | | | | | | | | | | Add missing include of <sys/resouce.h> | ||||
| * | | | | | Add missing include of <sys/resouce.h> | Oleksandr Usov | 2012-07-16 | 1 | -0/+5 |
| | |/ / / | |/| | | | |||||
* | | | | | Remove #pragma once from our header files. | Thiago Farina | 2012-07-17 | 4 | -4/+0 |
|/ / / / | | | | | | | | | | | | | | | | | | | | | https://github.com/martine/ninja/issues/358 Signed-off-by: Thiago Farina <tfarina@chromium.org> | ||||
* | | | | Efficiently check for emptiness | Pierre Schweitzer | 2012-07-14 | 1 | -1/+1 |
| | | | | |||||
* | | | | Clarify comment wording. | Ted Vessenes | 2012-07-11 | 1 | -3/+3 |
| | | | | |||||
* | | | | Fix missing POLLRDHUP constant on older systems. | Ted Vessenes | 2012-07-11 | 1 | -0/+7 |
| | | | | | | | | | | | | | | | | | | | | Attempting to compile with g++ 4.1.2 failed because the POLLRDHUP constant was not defined when <poll.h> is included. | ||||
* | | | | Rename parsers.* to manifest_parser.* | Thiago Farina | 2012-07-09 | 7 | -9/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | 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 | Evan Martin | 2012-07-03 | 2 | -7/+55 |
|\ \ \ \ | |_|_|/ |/| | | | Do not reset restat_mtime if an input is missing | ||||
| * | | | Add a test for missing input files in restat rules | wolfp | 2012-06-27 | 1 | -0/+54 |
| | | | | |||||
| * | | | Do not reset restat_mtime if an input is missing (issue #295) | wolfp | 2012-06-27 | 1 | -7/+1 |
| | | | | |||||
* | | | | Improving Edge::Dump, introducing Node::Dump | Maxim Kalaev | 2012-06-28 | 2 | -6/+27 |
| |/ / |/| | | | | | | | | | | | | | | | | | - 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"). | ||||
* | | | switch build log to print hashes as hex | Evan Martin | 2012-06-19 | 2 | -2/+3 |
| | | | | | | | | | | | | | | | | | | 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. |