Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | move BuildConfig out of globals | Evan Martin | 2012-09-04 | 1 | -14/+15 |
| | |||||
* | remove DiskInterface from globals | Evan Martin | 2012-09-04 | 1 | -5/+4 |
| | |||||
* | remove unfortunate header dependency | Evan Martin | 2012-09-04 | 1 | -1/+0 |
| | | | | This was temporarily added, and now it can be removed. | ||||
* | clarify setter | Evan Martin | 2012-09-04 | 2 | -5/+10 |
| | |||||
* | move BuildLog to DependencyScan | Evan Martin | 2012-09-04 | 9 | -22/+32 |
| | | | | | | | The build log is needed in computing whether an edge is dirty, so I think it belongs here. (It's a bit weird that Builder needs to reach into it to record completed commands, maybe it will become cleaner with more thought.) | ||||
* | fix --debug for glibc pedantic mode | Evan Martin | 2012-09-04 | 1 | -0/+1 |
| | |||||
* | pass Builder as arg to build-running functions | Evan Martin | 2012-09-04 | 1 | -15/+15 |
| | | | | Reducing use of globals. | ||||
* | remove config from BuildLog, rename members | Evan Martin | 2012-09-02 | 4 | -30/+24 |
| | |||||
* | remove a redundant arg to RecomputeOutputDirty | Evan Martin | 2012-09-02 | 3 | -24/+19 |
| | |||||
* | split out dirty recomputation logic from Edge class | Evan Martin | 2012-09-02 | 6 | -88/+117 |
| | | | | | | | | | | Rather than passing States and DiskInterfaces through all the calls, put the necessary ambient information in a new DependencyScan object and move the code accordingly. Note: I didn't move the source location of the functions to preserve history, though this does result in a sort of weird order for the functions in graph.cc. | ||||
* | Merge pull request #407 from maximuska/proposed/dirty-on-missing-depfile | Evan Martin | 2012-08-30 | 2 | -3/+34 |
|\ | | | | | safer build: consider target dirty if depfile is missing | ||||
| * | safer build: consider target dirty if depfile is missing | Maxim Kalaev | 2012-08-30 | 2 | -3/+34 |
|/ | |||||
* | Merge pull request #403 from nico/everylineweighsyoudown | Evan Martin | 2012-08-28 | 1 | -2/+0 |
|\ | | | | | Remove unused macro NINJA_UNUSED_ARG. | ||||
| * | Remove unused macro NINJA_UNUSED_ARG. | Nico Weber | 2012-08-28 | 1 | -2/+0 |
|/ | |||||
* | point to HACKING.md in README | Evan Martin | 2012-08-24 | 1 | -0/+1 |
| | |||||
* | drop DepfileParserTest.Tilde, as it's covered by .SpecialChars | Evan Martin | 2012-08-24 | 1 | -13/+0 |
| | | | | | The SpecialChars test covers a bunch of different special characters, including tilde. | ||||
* | link directly to gtest zip | Evan Martin | 2012-08-24 | 1 | -1/+3 |
| | |||||
* | update old test to cover newer rule attributes | Evan Martin | 2012-08-24 | 1 | -1/+4 |
| | |||||
* | add test that checks attributes on rules | Evan Martin | 2012-08-24 | 1 | -0/+11 |
| | |||||
* | windows: pass /Zi to gtest compile as well | Evan Martin | 2012-08-24 | 1 | -1/+1 |
| | |||||
* | remove crlfs from .gitignore | Evan Martin | 2012-08-24 | 1 | -17/+17 |
| | | | | These were introduced by a change of mine on Windows, whoops. | ||||
* | fix test broken in 697350d | Evan Martin | 2012-08-17 | 1 | -2/+10 |
| | | | | That's what I get for making last-second adjustments before checking in! | ||||
* | make it more explicit that a bad build log causes us to rebuild | Evan Martin | 2012-08-17 | 3 | -4/+12 |
| | | | | | (Committing this on top of b56fe80 since they're related, but I may end up reverting both.) | ||||
* | Merge pull request #401 from syntheticpp/win-network-path | Evan Martin | 2012-08-17 | 2 | -0/+27 |
|\ | | | | | on windows a network path starts with two backslashes | ||||
| * | on windows a network path starts with two backslashes | Peter Kuemmel | 2012-08-17 | 2 | -0/+27 |
| | | |||||
* | | drop HACKING from doxygen | Evan Martin | 2012-08-16 | 1 | -1/+1 |
| | | | | | | | | | | I tried just fixing the code to pull in HACKING.md but it didn't show up in the doxygen output; it's maybe too long to include anyway. | ||||
* | | use 4 space tabs in configure.py, warn on re2c missing | Evan Martin | 2012-08-16 | 1 | -13/+14 |
| | | |||||
* | | many updates for HACKING.md | Evan Martin | 2012-08-16 | 1 | -26/+78 |
| | | | | | | | | In particular, describe a policy for good patches. | ||||
* | | Merge pull request #400 from nico/dynre2c | Evan Martin | 2012-08-16 | 1 | -6/+16 |
|\ \ | | | | | | | Only write re2c rules if a re2c binary is found in the PATH. | ||||
| * | | Only write re2c rules if a re2c binary is found in the PATH. | Nico Weber | 2012-08-16 | 1 | -6/+16 |
| | | | |||||
* | | | Merge pull request #399 from nico/hackless | Evan Martin | 2012-08-16 | 1 | -2/+2 |
|\ \ \ | |/ / |/| | | Remove -fcolor-diagnostics reference from HACKING | ||||
| * | | Remove -fcolor-diagnostics reference from HACKING | Nico Weber | 2012-08-16 | 1 | -2/+2 |
|/ / | | | | | | | configure.py adds that flag automatically if CXX is set to clang. | ||||
* | | convert HACKING to markdown, add MSVC section | Evan Martin | 2012-08-15 | 2 | -76/+113 |
| | | |||||
* | | if a file is missing in the log, count it as dirty | Scott Graham | 2012-08-15 | 2 | -6/+63 |
| | | | | | | | | | | | | | | This could cause overbuilding (if the log is missing an entry and the right file is already in place) but is otherwise necessary for correctness (if a file is already in place but we don't have a log entry for it). | ||||
* | | add a helper binary for wrapping cl.exe | Evan Martin | 2012-08-15 | 3 | -8/+167 |
| | | | | | | | | Modify bootstrap etc. to make use of this binary. | ||||
* | | pass env block to cl helper | Evan Martin | 2012-08-15 | 3 | -1/+17 |
| | | |||||
* | | Merge branch 'master' of github.com:martine/ninja | Evan Martin | 2012-08-13 | 11 | -18/+560 |
|\ \ | |||||
| * | | create phony rules for all binaries | Evan Martin | 2012-08-13 | 1 | -5/+3 |
| | | | |||||
| * | | msvc helper: drop system includes | Evan Martin | 2012-08-12 | 3 | -3/+30 |
| | | | | | | | | | | | | | | | Drop any #includes that look like they're referencing system headers. This reduces the dependency information considerably. | ||||
| * | | includes_normalize: also lowercase cross-drive includes | Evan Martin | 2012-08-12 | 3 | -10/+9 |
| | | | | | | | | | | | | It seems to me inconsistent to normalize one but not the other. | ||||
| * | | add functions for normalizing win32 include paths | Scott Graham | 2012-08-12 | 4 | -1/+252 |
| | | | | | | | | | | | | | | | | | | | | | (Note from Evan: this is landing Scott's code more or less verbatim without a lot of analysis; it could maybe be simplified and reduced, but it's only intended to be used in the MSVC helper so it's fine to be experimental.) | ||||
| * | | add .ilk (incremental linker) to .gitignore | Evan Martin | 2012-08-12 | 1 | -0/+1 |
| | | | |||||
| * | | msvc helper: attempt to filter out when it prints the input filename | Evan Martin | 2012-08-12 | 3 | -0/+47 |
| | | | | | | | | | | | | This is a heuristic but it appears to work for the Chrome build. | ||||
| * | | add subprocess-spawning to msvc_helper | Evan Martin | 2012-08-12 | 3 | -1/+116 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than using subprocess.h, reimplement the subprocess code. This allows: 1) using anonymous (instead of named) pipes 2) not using all the completion port craziness 3) printing the output as it happens 4) further variation, like adjusting the environment (in a forthcoming change) without affecting the main subprocess code | ||||
| * | | doc some cl.exe flags in the configure script | Evan Martin | 2012-08-12 | 1 | -2/+7 |
| | | | |||||
| * | | move Win32Fatal into util | Evan Martin | 2012-08-12 | 3 | -8/+7 |
| | | | |||||
| * | | add a module for working with MSVC (cl.exe) behavior | Evan Martin | 2012-08-12 | 4 | -0/+99 |
| | | | | | | | | | | | | This will be needed for performant builds on Windows. | ||||
| * | | use correct path separator for --with-gtest source | Evan Martin | 2012-08-12 | 1 | -2/+2 |
| | | | |||||
| * | | doc that ReadFile reads in text mode on Windows | Evan Martin | 2012-08-12 | 1 | -1/+2 |
| | | | |||||
* | | | rename -V to --version and clean up --help output | Evan Martin | 2012-08-13 | 1 | -9/+8 |
|/ / |