summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* ManifestParser constructor accesses its first argument, don't pass NULLNico Weber2013-01-031-3/+6
| | | | | | | | | | | The constructor does env_ = &state->bindings_; so env_ is effectively set to offsetof(ManifestParser, bindings_). This will blow up if env_ gets dereferenced -- this doesn't seem to happen in these tests, but it's less confusing with this patch. Also, passing &state is consistent with the rest of this test.
* fix build on non-Linux glibc systemsPino Toscano2012-12-291-2/+2
| | | | | | | | | | | | ninja-build does not build on non-Linux archs, such as GNU/kFreeBSD and GNU/Hurd. The problem is that the GetProcessorCount() implementation for these architectures is the sysconf() one, but <unistd.h> has not been included, causing sysconf() and _SC_NPROCESSORS_ONLN to not be declared. Another solution (which is the one I chose) is to make use of the "linux" implementation which uses get_nprocs(), which is a GNU extension and thus available for anything using GNU libc.
* drop the "rules" tool completelyEvan Martin2012-12-291-19/+0
| | | | | I had already broken this earlier, but the refactor of variable handling broke it completely.
* rearrange handling of builtin bindings to make rules simplerEvan Martin2012-12-2912-154/+183
| | | | | Now, a 'build' block can override any special binding like 'command' or 'description' if it needs to.
* wrap some overlong linesEvan Martin2012-12-2912-27/+53
|
* fix all "class" -> "struct"Evan Martin2012-12-295-11/+9
|
* Merge pull request #467 from ppuryear/browse-errorEvan Martin2012-12-291-5/+6
|\ | | | | browse: Read ninja's error text from stderr.
| * browse: Read ninja's error text from stderr.Philip Puryear2012-12-081-5/+6
| |
* | wrap test in anon namespaceEvan Martin2012-12-281-0/+4
| |
* | correctly open /dev/null in subprocessesEvan Martin2012-12-212-1/+16
| | | | | | | | Fixes issue #468.
* | update a commentEvan Martin2012-12-211-6/+3
| |
* | NINJA_STATUS: add support of `%p` for percentageyannicklm2012-12-141-0/+8
| |
* | Merge pull request #461 from riannucci/global_sectionEvan Martin2012-12-1315-202/+581
|\ \ | |/ |/| Resource pools for ninja
| * Remove unnecessary parameter from ShouldDelayEdgeRobert Iannucci2012-11-302-2/+2
| |
| * Rename isValidRobert Iannucci2012-11-301-1/+1
| |
| * Fix formattingRobert Iannucci2012-11-304-13/+13
| |
| * Improve comments for src/state.hRobert Iannucci2012-11-301-3/+12
| |
| * Doc improvementsRobert Iannucci2012-11-301-0/+8
| |
| * Make Edge->pool() a pointer like it should have beenRobert Iannucci2012-11-302-8/+8
| |
| * Revert "Dump state for debugging"Robert Iannucci2012-11-101-4/+0
| | | | | | | | This reverts commit 9b196dc806e57cefd88bbbacd12286447dbf9ad9.
| * Uninitialized variable! There is always one...Robert Iannucci2012-11-101-1/+1
| |
| * Dump state for debuggingRobert Iannucci2012-11-101-0/+4
| |
| * Make edge dump pool name, and skip default poolRobert Iannucci2012-11-102-1/+10
| |
| * cover the nulled pool caseRobert Iannucci2012-11-101-19/+20
| |
| * Add some tests!Robert Iannucci2012-11-101-3/+129
| |
| * begin rationalizing platform for both parsersRobert Iannucci2012-11-101-7/+5
| |
| * block parse method doneRobert Iannucci2012-11-106-185/+277
| |
| * all building and tests passingRobert Iannucci2012-11-092-2/+2
| |
| * Dump pools with StateRobert Iannucci2012-11-093-6/+26
| |
| * Fix a bug... now chrome build works O_ORobert Iannucci2012-11-091-1/+1
| |
| * and some basic implementationRobert Iannucci2012-11-094-15/+46
| |
| * stub out an api and de-constify PoolRobert Iannucci2012-11-096-17/+63
| |
| * Const ref FTWRobert Iannucci2012-11-091-1/+1
| |
| * Pull graph.cc tooRobert Iannucci2012-11-091-1/+2
| |
| * Pull out base changes to stateRobert Iannucci2012-11-092-2/+40
| |
* | Fix clang warning.Thiago Farina2012-11-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | The return type of CollectTarget() is Node, so we should return NULL in the failure case instead of false. src/ninja.cc:188:16: warning: initialization of pointer of type 'Node *' to null from a constant boolean expression [-Wbool-conversion] return false; ^~~~~ Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | Merge pull request #459 from maximuska/proposed/refactor-collect-targetsEvan Martin2012-11-121-49/+51
|\ \ | | | | | | refactoring: decompose CollectTargetsFromArgs
| * | refactoring: decompose CollectTargetsFromArgsMaxim Kalaev2012-11-091-49/+51
| | |
* | | Merge pull request #443 from tfarina/disk-interface-allocEvan Martin2012-11-121-1/+1
|\ \ \ | | | | | | | | Allocate disk_interface near where it's needed.
| * | | Allocate disk_interface near where it's needed.Thiago Farina2012-10-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This avoids allocating disk_interface unnecessarily. Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | | | Merge pull request #454 from nico/tweakEvan Martin2012-11-121-1/+1
|\ \ \ \ | | | | | | | | | | Fix a doxygen bug found by clang's -Wdocumentation.
| * | | | Fix a doxygen bug found by clang's -Wdocumentation.Nico Weber2012-11-081-1/+1
| | |_|/ | |/| | | | | | | | | | | | | | (That found one more issue, but I think that might be a bug in -Wdocumentation, http://llvm.org/PR14295.)
* | | | Merge pull request #458 from rgeary1/minorEvan Martin2012-11-092-1/+4
|\ \ \ \ | |/ / / |/| | | Minor improvements to ninja : Different exit code if user presses Ctrl-C, add eclipse to .gitignore, Improve an error message
| * | | Exit status = 2 if user presses ctrl-cRichard Geary2012-11-091-0/+3
| | | | | | | | | | | | | | | | Change-Id: I7be958e18eb2e434e78afb6e03b332281a651957
| * | | Improved error message with more informationRichard Geary2012-11-091-1/+1
| | | | | | | | | | | | | | | | Change-Id: Idb1ce67a320a9819de262d83b498ee10eb362ed2
* | | | refactor repeated code in cleanerEvan Martin2012-10-312-14/+18
| | | |
* | | | delete some obsolete TODOsEvan Martin2012-10-311-2/+1
| | | |
* | | | trailing whitespaceEvan Martin2012-10-311-1/+1
| |_|/ |/| |
* | | browse.py: Don't truncate error message if ninja -t query fails.Philip Puryear2012-10-261-10/+15
| | | | | | | | | | | | Signed-off-by: Philip Puryear <philippuryear@gmail.com>
* | | browse.py: Fix truncation with an unknown target.Philip Puryear2012-10-261-1/+3
| | | | | | | | | | | | Signed-off-by: Philip Puryear <philippuryear@gmail.com>
generated by cgit v0.12 at 2024-09-07 10:16:58 (GMT)