summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Include unistd.h in files that use unlink() on OS X.Nico Weber2012-08-052-3/+5
| | | | | | | | `man unlink` says this is necessary, and according to a report by Claus Klein, omitting them breaks the build on OS X 10.5 with gcc 4.7 (see issue #396). (On Windows, ninja's util.h includes a define for unlink.)
* reject tabs (and CRs) in input files more aggressivelyEvan Martin2012-08-026-41/+57
|
* Merge branch 'master' of github.com:martine/ninjaEvan Martin2012-08-011-3/+3
|\
| * fix hash_map StringPiece comparatorScott Graham2012-07-311-3/+3
| |
* | Merge branch 'factor-elide-middle' of git://github.com/polrop/ninjaEvan Martin2012-08-014-13/+29
|\ \ | |/ |/| | | | | Conflicts: src/util.cc
| * Re-factor elide code and test it.Nicolas Despres2012-07-314-13/+30
| |
* | use chdir() define from util.hEvan Martin2012-07-281-4/+0
| |
* | move processor-count code to util.ccEvan Martin2012-07-283-25/+35
| |
* | note that -l doesn't work on windowsEvan Martin2012-07-282-0/+4
| | | | | | | | Filed issue #386 about it too.
* | simplify load-average codeEvan Martin2012-07-281-15/+3
| |
* | delete vestigal declarationsEvan Martin2012-07-281-7/+0
| |
* | Disable SubprocessTest.TestWithLots on OS X.Nico Weber2012-07-281-2/+4
| | | | | | | | It fails on OS X, see issue #384.
* | Revert "Make StringPiece data members private."Nico Weber2012-07-288-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 Weber2012-07-281-0/+19
|/
* also build with msvcPeter Kümmel2012-07-273-5/+10
|
* fix a warning and leak in windows-specific console output codeEvan Martin2012-07-271-2/+2
|
* fix error and prevent warning in GetTimeMillis()Claus Klein2012-07-272-2/+2
|
* Merge branch 'master' of github.com:martine/ninjaEvan Martin2012-07-278-14/+19
|\
| * Merge pull request #352 from tfarina/string-piece-privateEvan Martin2012-07-278-14/+19
| |\ | | | | | | Make StringPiece data members private.
| | * Make StringPiece data members private.Thiago Farina2012-06-308-14/+19
| | | | | | | | | | | | Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | | rename subprocess.cc to reflex its posixnessEvan Martin2012-07-271-0/+0
| | |
* | | rearrange minidump patch to match ninja code styleEvan Martin2012-07-273-113/+132
| | |
* | | Merge branch 'set_terminate' of git://github.com/gmoudry/ninja into minidumpEvan Martin2012-07-273-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 Moudry2012-04-262-3/+3
| | | | | | | | | | | | happily
| * | Improve handling of fatal errors on Windows, support creation of minidumpsJiri Moudry2012-03-283-0/+120
| | |
* | | Merge pull request #351 from syntheticpp/rateEvan Martin2012-07-279-17/+101
|\ \ \ | | | | | | | | print edges per second
| * | | GCC 4.7 needs cstdioPeter Kümmel2012-07-151-1/+1
| | | |
| * | | print edges per secondPeter Kuemmel2012-07-159-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 logicEvan Martin2012-07-271-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 itMaxim Kalaev2012-07-271-1/+3
| | | | | | | | | | | | | | | | Not a real problem, mainly to make valgrind happy
* | | | fix some mingw warnings in getopt.cEvan Martin2012-07-271-6/+2
| | | | | | | | | | | | | | | | From a patch from Claus Klein <claus.klein@arcormail.de>.
* | | | windows: fix printf format for process idEvan Martin2012-07-271-1/+1
| | | |
* | | | 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
| | | |