Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | make Node::dirty_ private | Evan Martin | 2011-12-07 | 6 | -43/+49 |
| | |||||
* | more whitespace | Evan Martin | 2011-12-07 | 1 | -0/+4 |
| | |||||
* | merge StatCache into State | Evan Martin | 2011-12-07 | 5 | -120/+49 |
| | | | | | | I think I had originally imagined StatCache would contain more state, but at this point it's clear it was just managing a single map, which could just as well be in the already-small State object. | ||||
* | merge FileStat into Node | Evan Martin | 2011-12-07 | 14 | -109/+119 |
| | | | | | | The two were always one-to-one anyway. I started adding accessors to FileStat and then realized most users wanted them on Node and that forwarding them through was silly. | ||||
* | refactor to remove Node::ready() | Evan Martin | 2011-12-07 | 3 | -5/+14 |
| | |||||
* | add and use getter for Edge::rule_ | Evan Martin | 2011-12-07 | 4 | -10/+11 |
| | |||||
* | move build/install to README | Evan Martin | 2011-12-06 | 2 | -24/+12 |
| | |||||
* | Merge pull request #152 from pcc/clean-depfiles | Evan Martin | 2011-12-06 | 2 | -0/+17 |
|\ | | | | | Have the clean tool remove depfiles | ||||
| * | Have the clean tool remove depfiles | Peter Collingbourne | 2011-12-06 | 2 | -0/+17 |
| | | |||||
* | | Merge pull request #151 from pcc/restat-depfiles | Evan Martin | 2011-12-06 | 3 | -4/+16 |
|\ \ | | | | | | | Use the modification time of the depfile when computing restat_mtime | ||||
| * | | Use the modification time of the depfile when computing restat_mtime | Peter Collingbourne | 2011-12-06 | 1 | -1/+9 |
| |/ | | | | | | | Fixes issue #144. | ||||
| * | Factor out Edge::EvaluateDepFile | Peter Collingbourne | 2011-12-06 | 2 | -3/+7 |
|/ | |||||
* | more whitespace in build output | Evan Martin | 2011-12-05 | 1 | -0/+1 |
| | |||||
* | remove reference to nonexistent file | Evan Martin | 2011-12-05 | 1 | -2/+0 |
| | |||||
* | add help output to clean tool | Evan Martin | 2011-12-05 | 1 | -10/+16 |
| | |||||
* | prettify manual for tools | Evan Martin | 2011-12-05 | 1 | -21/+31 |
| | |||||
* | also spell-check the 'clean' tool; abort on misspelling | Evan Martin | 2011-12-05 | 1 | -2/+3 |
| | |||||
* | minor manual touchups | Evan Martin | 2011-12-05 | 1 | -13/+15 |
| | |||||
* | split out tool list into a separate subcommand | Evan Martin | 2011-12-05 | 1 | -14/+27 |
| | | | | | Fits the -h output on one screen again, and allows more whitespace in both the -h output and the tool list. | ||||
* | disable the 'unused parameter' warning | Evan Martin | 2011-12-05 | 9 | -12/+15 |
| | | | | It was firing too often, and hadn't uncovered any bugs. | ||||
* | Merge pull request #150 from luislavena/mingw-no-visibility | Evan Martin | 2011-12-05 | 1 | -0/+1 |
|\ | | | | | visibility=hidden should not be used for Windows | ||||
| * | visibility=hidden should not be used for Windows | Luis Lavena | 2011-12-05 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | | | | | | | While GCC 4.x supports the usage of -fvisibility=hidden and the related pragmas, usage of it under MinGW is a noop and should not be used. Instead, __declspec(dllexport/dllimport) should be used instead to indicate exportable functions. Since Ninja builds a static library, no functions are exported and visibility, at least under MinGW, is not an issue. | ||||
* | | Merge pull request #149 from luislavena/mingw-static | Evan Martin | 2011-12-05 | 1 | -1/+1 |
|\ \ | |/ |/| | Use g++'s -static option instead of specific libs | ||||
| * | Use g++'s -static option instead of specific libs | Luis Lavena | 2011-12-05 | 1 | -1/+1 |
|/ | | | | | | | | MinGW g++ recognizes -static as option to avoid the need to specify individual libraries to be statically linked into the executable. It also solves the warning of -static-libstdc++ not being recorgnized by GCC 4.5.2 (TDM build) | ||||
* | fix some minor style issues in RecomputeDirty | Evan Martin | 2011-12-04 | 1 | -11/+11 |
| | |||||
* | when an edge is dirty, mark all outputs dirty | Evan Martin | 2011-12-04 | 2 | -12/+27 |
| | | | | | | | | | | | | | | | The logic before was like: for each output: if edge_dirty or output.dirty: output.dirty = true edge_dirty = true This was wrong in the case where the second output would case the edge to be dirty; we needed to go back and mark the first output dirty as well. Fixed by taking two passes: one to compute the dirty state, then a latter sweep to mark all outputs. Fixes issue 148. | ||||
* | refactor RecomputeOutputDirty to return true/false for dirty | Evan Martin | 2011-12-04 | 3 | -39/+38 |
| | | | | | | | | | Rather than taking whether the edge is dirty as an input, instead only consider the arguments and return true/false, allowing the caller to decide what to do with that information. (In the restat case we were faking out the environment to get particular behavior for this.) Should have no side effects, but this is part of fixing issue 148. | ||||
* | add a test case from issue 148 | Evan Martin | 2011-12-04 | 1 | -0/+20 |
| | |||||
* | windows: don't buffer stdout | Evan Martin | 2011-12-02 | 1 | -0/+5 |
| | |||||
* | HACKING: windows notes | Evan Martin | 2011-12-02 | 1 | -1/+10 |
| | |||||
* | bootstrap.sh: windows (mingw) fixes | Evan Martin | 2011-12-02 | 1 | -3/+3 |
| | | | | | | | - Don't build browse.cc in bootstrap.sh. - Explicitly call out to Python to run configure.py. - Run ninja-bootstrap with no arguments to build the default target, which is "ninja.exe" on Windows. | ||||
* | configure.py: windows updates | Evan Martin | 2011-12-02 | 1 | -25/+28 |
| | | | | | - Drop unused 'windows' platform - Make 'mingw' platform actually work on mingw-on-Windows. | ||||
* | windows: clear process launch structs before using them | Evan Martin | 2011-12-02 | 1 | -0/+2 |
| | |||||
* | Revert "Merge pull request #143 from cipriancraciun/patches/python2" | Evan Martin | 2011-11-25 | 5 | -5/+5 |
| | | | | | | | This reverts commit 38ab41f45ff818b437942b753328a0168914fc86, reversing changes made to 819d6347b424f583d651b86dd1280605ddb23b88. Platforms that don't have /usr/bin/python pointing to python2 are broken. | ||||
* | Merge pull request #143 from cipriancraciun/patches/python2 | Evan Martin | 2011-11-22 | 5 | -5/+5 |
|\ | | | | | Rename all occurences of `python` to `python2` | ||||
| * | Renamed `python` to `python2`. | Ciprian Dorin, Craciun | 2011-11-22 | 1 | -1/+1 |
| | | |||||
| * | Fixed Python 2 scripts prologue (i.e. `#!/usr/bin/env python2`). | Ciprian Dorin, Craciun | 2011-11-22 | 4 | -4/+4 |
| | | |||||
* | | Merge pull request #142 from cipriancraciun/patches/custom-cflags-ldflags | Evan Martin | 2011-11-22 | 1 | -4/+4 |
|\ \ | |/ |/| | Added support for custom CFLAGS and LDFLAGS. | ||||
| * | Added support for custom CFLAGS and LDFLAGS. | Ciprian Dorin, Craciun | 2011-11-22 | 1 | -4/+4 |
|/ | |||||
* | Merge pull request #138 from polrop/fix-warning-in-test | Evan Martin | 2011-11-21 | 3 | -5/+5 |
|\ | | | | | Fix compilation warning in Ninja's test suite. | ||||
| * | Fix compilation warning in Ninja's test suite. | Nicolas Despres | 2011-11-21 | 3 | -5/+5 |
|/ | |||||
* | Merge pull request #136 from polrop/add-extra-warning | Evan Martin | 2011-11-20 | 9 | -9/+18 |
|\ | | | | | Add -Wextra compilation flags | ||||
| * | Remove warning triggered by -Wextra on MinGW. | Nicolas Despres | 2011-11-19 | 5 | -2/+11 |
| | | |||||
| * | Add -Wextra warning flag. | Nicolas Despres | 2011-11-19 | 6 | -7/+7 |
|/ | | | | | | | Fix triggered warnings: - unused parameter - type qualifiers ignored on function return type - missing initializer for member | ||||
* | Merge pull request #135 from polrop/fix-mingw | Evan Martin | 2011-11-19 | 3 | -5/+4 |
|\ | | | | | Fix mingw. | ||||
| * | Fix compilation warning with getopt() on MinGW. | Nicolas Despres | 2011-11-19 | 1 | -1/+1 |
| | | |||||
| * | Fix compilation warning with getopt_long() on MinGW. | Nicolas Despres | 2011-11-19 | 1 | -2/+2 |
| | | |||||
| * | Use more portable way to get the current process ID. | Nicolas Despres | 2011-11-19 | 1 | -2/+1 |
| | | | | | | | | | | | | | | GetProcessId() is available only since Windows XP. Since MinGW define WINVER to 0x0400 which is Windows 2000 I think, we have a compilation error. Using GetCurrentProcessId() instead of GetProcessId(GetCurrentProcess()) fix this issue. | ||||
* | | Merge pull request #133 from polrop/fix-duplication | Evan Martin | 2011-11-19 | 1 | -1/+1 |
|\ \ | |/ |/| | Fix duplicated list of profilers. | ||||
| * | Fix duplicated list of profilers. | Nicolas Despres | 2011-11-19 | 1 | -1/+1 |
|/ |