summaryrefslogtreecommitdiffstats
path: root/src/ninja.cc
Commit message (Collapse)AuthorAgeFilesLines
* make `-w dupbuild` default to `err`Nico Weber2018-04-051-0/+1
| | | | | | | | | You can still opt out of this by passing `-w dupbuild=warn`. But if you're getting this diagnostic, your build files are incorrect and you should ideally just fix them. This is step 3 on https://github.com/ninja-build/ninja/issues/931 I sent an RfC to ninja-build a few months ago; nobody objected.
* [compdb] Move declarations closer to their use.Asanka Herath2018-04-051-3/+3
|
* [compdb] Expand response files inline based on a switch.Asanka Herath2018-04-051-2/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | References to response files in a clang compile_commands.json file can be tricky to deal with when tooling expects all the command flags to be present in the 'command' field. This change introduces a '-x' option to '-t compdb' that will expand @rspfile style response file invocations inline. E.g. ```sh $ ninja -t compdb cc [ { "directory": "/src/foo", "command": "cc -foo -bar @foo.obj.rsp", "file": "foo.cc" } ] $ ninja -t compdb -x cc [ { "directory": "/src/foo", "command": "cc -foo -bar foo.cc", "file": "foo.cc" } ] ```
* Merge pull request #1386 from jmgao/exitNico Weber2018-04-051-14/+16
|\ | | | | Don't clean up after ourselves when exiting.
| * Don't clean up after ourselves when exiting.Josh Gao2018-02-191-14/+16
| | | | | | | | | | | | Destruction of NinjaMain can be an expensive operation when dealing with stupidly large ninjafiles. exit directly instead of returning out of real_main to avoid doing so.
* | Merge pull request #1373 from BillyDonahue/help_k_zeroNico Weber2018-04-051-1/+1
|\ \ | | | | | | Update Usage to show "-k 0" behavior
| * | Update Usage to show "-k 0" behaviorBilly Donahue2018-01-121-1/+1
| |/ | | | | For "-k N", N==0 is interpreted as infinite. It's useful but not documented in the help, unfortunately.
* | Add 'output' field to compdb outputScott Graham2018-02-071-0/+2
|/
* Merge pull request #1219 from QuLogic/resolutionNico Weber2017-10-201-1/+1
|\ | | | | Use high-resolution timestamps
| * Make TimeStamp 64-bit.Elliott Sales de Andrade2017-09-161-1/+1
| | | | | | | | This prepares it for higher-resolution timestamps.
* | escape usage examplesRefael Ackermann2017-10-191-3/+3
|/
* Restore tolerance of self-referencing phony build statementsBrad King2017-09-081-2/+16
| | | | | | | | | | | | | | | | | | | | Since commit v1.8.0^2~3^2~1 (Teach RecomputeDirty to detect cycles in the build graph, 2015-11-13) we correctly reject self-referencing phony build statements like build a: phony a as cycles. Unfortunately this breaks support for CMake 2.8.12.x and 3.0.x because those versions incorrectly produce edges of this form (that we used to tolerate). In order to preserve compatibility with those CMake versions we need to restore tolerance of these edges. Add a special case to the manifest parser to filter out self-referencing inputs of phony edges of the form produced by those CMake versions. Warn by default, but add a `-w phonycycle={err,warn}` option to make it an error. Fixes: #1322
* Factor ManifestParser options into a structureBrad King2017-09-071-4/+5
| | | | | | This will allow more options to be added without updating everywhere that constructs a ManifestParser. Also extend the AssertParse function to take the options so tests can control them.
* Support restat when rebuilding manifestDan Willemsen2017-02-081-4/+13
| | | | | | | | As a fix for #874, we started reloading the entire manifest even if the manifest was never rebuilt due to a restat rule. But this can be slow, so call State::Reset instead, which also fixes the original crash. Fixes #987
* Merge pull request #1181 from DanielWeber/issue-1161Nico Weber2017-01-241-2/+2
|\ | | | | Allow more path components
| * Allow more path componentsDaniel Weber2016-08-221-2/+2
| | | | | | | | | | | | | | - 60 instead of 30 path components - 64 instead of 32 backslashes in a path (windows only) Issue: 1161
* | teach -t commands to optionally print only the final commandNico Weber2016-11-061-5/+35
|/
* Pass location of build.ninja file to browse scriptColin Cross2016-04-191-27/+27
| | | | | Pass the value of -f to the browse python script so it can be passed back to ninja -t query.
* browse: support --port and --no-browser optionsPeter Wu2016-03-111-5/+1
| | | | | | | Add --port option to override the default port (8000). Add --no-browser option to avoid opening a web browser (useful over SSH). Make the target name optional, using "all" if omitted.
* Fix ambiguous call to set_terminate on Windows platformg4m42016-02-171-1/+1
| | | | | | | On Windows set_terminate() could either be the standard C++ one or (actually the same one but in the global namespace) the CRT one declared in corecrt_terminate.h Hence this ambiguity - this patch solves it. Signed-off-by: g4m4 <misept.dieseneuf@gmail.com>
* Replace ManifestParser::FileReader with general FileReaderBrad King2016-02-031-10/+1
| | | | | | Avoid having two separate filesystem interfaces. Simplify test infrastructure by avoiding custom `ManifestParser::FileReader` implementations.
* Merge pull request #1096 from sgraham/dupe-edge-actionNico Weber2016-01-291-1/+3
|\ | | | | dupe_edge_should_err from bool to enum
| * dupe_edge_should_err from bool to enumScott Graham2016-01-271-1/+3
| |
* | Add -d keepdepfile to preserve depfilesShinichiro Hamaji2016-01-281-5/+10
|/ | | | | This is useful when you are developing a tool which generates GCC-style depfiles.
* Remove unnecessary `std::`Tetsuo Kiso2016-01-101-2/+2
|
* Do not attempt to rebuild infinitely in dry-run mode.Nicolas Despres2015-12-091-0/+4
|
* Merge pull request #1007 from mikesep/aixEvan Martin2015-11-121-0/+3
|\ | | | | Support for AIX
| * Fix getopt for AIXMike Seplowitz2015-08-191-0/+3
| | | | | | | | | | | | | | AIX supplies getopt but not getopt_long. We can't use the embedded getopt implementation, since the constness of its arguments doesn't match the AIX system routine.
* | Make links point to org pageNico Weber2015-11-111-1/+1
|/
* Some mucking with std:: for set_terminate and terminate_handlerScott Graham2015-08-101-1/+1
|
* win: print right slashes in 'unknown target' messageNico Weber2015-07-101-3/+3
|
* Let Stat() have an err outparam instead of writing to stderr.Nico Weber2015-03-311-2/+11
| | | | | | | | | Also check for Stat() failure in a few more places. This way, ninja doesn't print two "ninja: error: " lines if stat() fails during a build. It also makes it easier to keep the stat tests quiet. Every caller of Stat() needs to explicitly log the error string if that's desired.
* Add an opt-in flag to make duplicate edges an error (`-w dupbuild=err`).Nico Weber2015-03-241-3/+38
| | | | | This is step 1 on #931. Duplicated edges will become an error by default in the future.
* Swap order of -k and -l in help output, to keep the list alphabetized.Nico Weber2015-03-241-1/+1
|
* Allow manifest rebuild to loop up to 100 timesColin Cross2015-02-031-14/+13
| | | | | | | | | Ninja generators that bootstrap themselves with Ninja may need to rebuild build.ninja multiple times. Replace the 2 cycle loop with a 100 cycle loop, and print the pass number each time it restarts. Original-author: Jamie Gennis <jgennis@gmail.com>
* Revert #223, fixes #874.Nico Weber2014-12-041-5/+4
| | | | | | | | No test since there's still no good way to test code in ninja.cc. Going forward, either move NinjaMain to its own file with tests, or create a system that makes it possible to run integration tests on the ninja binary. Instead, add a comment that explains why the restat optimization isn't done.
* drop NINJA_BOOTSTRAP defineEvan Martin2014-11-181-1/+1
|
* Make sure configure.py and ninja.cc always agree on if -t browse is included.Nico Weber2014-11-141-2/+2
| | | | | | | | | No behavior change on most platforms. On solaris, -t browse was compiled in in ninja.cc but browse.cc wasn't compiled in, which probably means that building on Solaris didn't work. It might be better now. This also makes browse.cc automatically not included in bootstrap builds; previously this was done manually through the NINJA_BOOTSTRAP check.
* fix order of args to CanonicalizePathScott Graham2014-11-091-2/+2
|
* remove CanonicalizePath overloads, test for toplevel behaviourScott Graham2014-11-091-2/+4
|
* Remove extra info from Usage()kwesolowski2014-07-261-3/+0
| | | | As now behavior is similar on on platforms, this can probably be removed
* Small usage info fix for windowskwesolowski2014-07-251-1/+1
|
* Fixed/improved -l N documentation For windowskwesolowski2014-07-251-1/+1
| | | | | On windows "system load" is not so obvious term, added line explaining N argument in -l N
* Merge pull request #755 from zmodem/working_dir_vs_msvc_toolNico Weber2014-06-291-8/+9
|\ | | | | Process the -C option before running RUN_AFTER_FLAGS tools.
| * Process the -C option before running RUN_AFTER_FLAGS tools.Hans Wennborg2014-05-031-8/+9
| | | | | | | | | | | | | | | | | | | | This brings the flag more inline with its description: "change to DIR before doing anything else". The use case is to make it possible to use -C together with -t msvc. When debugging Windows builds, it's handy to be able to copy-paste the commands from "ninja -v" and make them run in the correct directory by adding this flag.
* | Rename -d nowinstatcache to -d nostatcache; might become useful elsewhere (#787)Nico Weber2014-06-271-5/+5
| |
* | do not delete files from the logs that still exist on diskNico Weber2014-06-201-1/+3
| | | | | | | | | | | | | | | | | | | | | | This is to keep the possibility of maybe having a tool that deletes old files in the future, or for having a tool which exposes this information to generators so they can do that. See https://github.com/martine/ninja/pull/697#issuecomment-37140762 and the discussion on #762. Idea from @maximuska!
* | add some statcache testsNico Weber2014-06-151-2/+2
| |
* | Free cache memory once it's no longer used.Nico Weber2014-06-151-7/+2
| | | | | | | | | | Doesn't slow down empty build times measurably, and saves some memory on non-empty builds.
* | on by defaultNico Weber2014-06-151-4/+4
| |