Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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. | ||||
* | | add copyright header | Evan Martin | 2012-06-19 | 1 | -0/+14 |
| | | |||||
* | | drop support of log formats < 4 | Evan Martin | 2012-06-18 | 2 | -23/+21 |
| | | | | | | | | | | 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 | Evan Martin | 2012-06-18 | 2 | -2/+23 |
| | | | | | | | | Fixes a warning. | ||||
* | | make canon_perftest build, add license | Evan Martin | 2012-06-18 | 1 | -0/+17 |
| | | |||||
* | | no recompacting needed when log file is empty | Peter Kuemmel | 2012-06-18 | 1 | -3/+7 |
| | | |||||
* | | Merge pull request #316 from tfarina/struct | Evan Martin | 2012-06-17 | 2 | -5/+6 |
|\ \ | | | | | | | No need to write 'struct' before type name when declaraing variables in ... | ||||
| * | | No need to write 'struct' before type name when declaraing variables in C++. | Thiago Farina | 2012-05-23 | 2 | -5/+6 |
| |/ | | | | | | | Signed-off-by: Thiago Farina <tfarina@chromium.org> | ||||
* | | improve test | Scott Graham | 2012-06-15 | 1 | -8/+7 |
| | | |||||
* | | add $in_newline | Scott Graham | 2012-06-15 | 2 | -6/+28 |
| | | |||||
* | | fix windows build | Scott Graham | 2012-06-15 | 1 | -0/+1 |
| | | |||||
* | | Add a hash collision benchmark. | Nico Weber | 2012-06-15 | 1 | -0/+43 |
| | | |||||
* | | Only store command hashes in the build log. | Nico Weber | 2012-06-15 | 8 | -17/+82 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | .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 | Evan Martin | 2012-06-15 | 1 | -0/+38 |
|\ \ | | | | | | | add canon_perftest | ||||
| * | | add canon_perftest | Nico Weber | 2012-06-03 | 1 | -0/+38 |
| | | | |||||
* | | | allow '(' and ')' in depfile paths | Evan Martin | 2012-06-12 | 3 | -7/+22 |
| | | | | | | | | | | | | Fixes issue #327. | ||||
* | | | fix a typo in a test name | Evan Martin | 2012-06-12 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #311 from nico/cleanups | Evan Martin | 2012-06-04 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | Cleanups | ||||
| * | | comment fix | Nico Weber | 2012-05-13 | 1 | -1/+1 |
| |/ | |||||
* | | Remove code that was never executed. | Nico Weber | 2012-06-02 | 1 | -9/+1 |
| | | | | | | | | | | 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 | Evan Martin | 2012-06-02 | 2 | -4/+0 |
|\ \ | | | | | | | Remove unused variable last_update_millis_. | ||||
| * | | Remove unused variable last_update_millis_. | Nico Weber | 2012-06-01 | 2 | -4/+0 |
| |/ | |||||
* | | Missing header file for "getenv". | Jonathan Sternberg | 2012-05-31 | 1 | -0/+1 |
| | | |||||
* | | Removing unused sys/termios.h header. | Jonathan Sternberg | 2012-05-30 | 1 | -1/+0 |
| | | |||||
* | | Header guards were missing from some header files. | Jonathan Sternberg | 2012-05-30 | 4 | -0/+19 |
|/ | |||||
* | Merge pull request #310 from nico/landreader | Evan Martin | 2012-05-10 | 1 | -11/+64 |
|\ | | | | | Speed up build log reading (0.5s on Win, 0.1s on Mac, 30ms on Linux) |