summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* wrap at 80 columnsEvan Martin2012-07-271-2/+5
|
* Don't print 'Entering directory' when running tools.Nico Weber2012-07-251-1/+4
|
* Use lowercase consistently.Thiago Farina2012-07-201-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-cursorEvan Martin2012-07-201-6/+20
|\ | | | | windows: use WriteConsoleOutput instead of printf to avoid moving cursor
| * windows: use WriteConsoleOutput instead of printf to avoid moving cursorScott Graham2012-07-191-6/+20
| |
* | dep file parsing: allow '@' and '=' in pathsMaxim Kalaev2012-07-193-17/+30
|/
* disallow crlf in manifest filesEvan Martin2012-07-183-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 recentEvan Martin2012-07-171-1/+4
| | | | Patch from Scott Graham <scottmg@chromium.org>.
* subprocess_test: make SetWithLots run on all non-Windows platformsEvan Martin2012-07-171-4/+6
| | | | | I think it might have been an oversight that this was marked Linux-specific.
* Merge pull request #340 from sgraham/fix-win-buildEvan Martin2012-07-172-0/+3
|\ | | | | fix win32 compile, fix BuildLogTest.WriteRead on 2nd run
| * remove unnecessary manual Close()Scott Graham2012-06-191-1/+0
| |
| * don't leak file handle on fail-to-upgrade caseScott Graham2012-06-191-0/+1
| |
| * fix win32 compile, fix BuildLogTest.WriteRead on 2nd runScott Graham2012-06-192-0/+3
| |
* | Merge pull request #363 from usovalx/subprocess_test_build_failureEvan Martin2012-07-171-0/+5
|\ \ | | | | | | Add missing include of <sys/resouce.h>
| * | Add missing include of <sys/resouce.h>Oleksandr Usov2012-07-161-0/+5
| | |
* | | Remove #pragma once from our header files.Thiago Farina2012-07-174-4/+0
|/ / | | | | | | | | | | https://github.com/martine/ninja/issues/358 Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | Efficiently check for emptinessPierre Schweitzer2012-07-141-1/+1
| |
* | Clarify comment wording.Ted Vessenes2012-07-111-3/+3
| |
* | Fix missing POLLRDHUP constant on older systems.Ted Vessenes2012-07-111-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 Farina2012-07-097-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_inputEvan Martin2012-07-032-7/+55
|\ \ | | | | | | Do not reset restat_mtime if an input is missing
| * | Add a test for missing input files in restat ruleswolfp2012-06-271-0/+54
| | |
| * | Do not reset restat_mtime if an input is missing (issue #295)wolfp2012-06-271-7/+1
| | |
* | | Improving Edge::Dump, introducing Node::DumpMaxim Kalaev2012-06-282-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 hexEvan Martin2012-06-192-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 headerEvan Martin2012-06-191-0/+14
| |
* | drop support of log formats < 4Evan Martin2012-06-182-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 intEvan Martin2012-06-182-2/+23
| | | | | | | | Fixes a warning.
* | make canon_perftest build, add licenseEvan Martin2012-06-181-0/+17
| |
* | no recompacting needed when log file is emptyPeter Kuemmel2012-06-181-3/+7
| |
* | Merge pull request #316 from tfarina/structEvan Martin2012-06-172-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 Farina2012-05-232-5/+6
| |/ | | | | | | Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | improve testScott Graham2012-06-151-8/+7
| |
* | add $in_newlineScott Graham2012-06-152-6/+28
| |
* | fix windows buildScott Graham2012-06-151-0/+1
| |
* | Add a hash collision benchmark.Nico Weber2012-06-151-0/+43
| |
* | Only store command hashes in the build log.Nico Weber2012-06-158-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/canonperfEvan Martin2012-06-151-0/+38
|\ \ | | | | | | add canon_perftest
| * | add canon_perftestNico Weber2012-06-031-0/+38
| | |
* | | allow '(' and ')' in depfile pathsEvan Martin2012-06-123-7/+22
| | | | | | | | | | | | Fixes issue #327.
* | | fix a typo in a test nameEvan Martin2012-06-121-1/+1
| | |
* | | Merge pull request #311 from nico/cleanupsEvan Martin2012-06-041-1/+1
|\ \ \ | |/ / |/| | Cleanups
| * | comment fixNico Weber2012-05-131-1/+1
| |/
* | Remove code that was never executed.Nico Weber2012-06-021-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/cleanupEvan Martin2012-06-022-4/+0
|\ \ | | | | | | Remove unused variable last_update_millis_.
| * | Remove unused variable last_update_millis_.Nico Weber2012-06-012-4/+0
| |/
* | Missing header file for "getenv".Jonathan Sternberg2012-05-311-0/+1
| |
* | Removing unused sys/termios.h header.Jonathan Sternberg2012-05-301-1/+0
| |
* | Header guards were missing from some header files.Jonathan Sternberg2012-05-304-0/+19
|/
* Merge pull request #310 from nico/landreaderEvan Martin2012-05-101-11/+64
|\ | | | | Speed up build log reading (0.5s on Win, 0.1s on Mac, 30ms on Linux)