summaryrefslogtreecommitdiffstats
path: root/src/ninja.cc
Commit message (Collapse)AuthorAgeFilesLines
* compdb: Dump every rule without any arguments, fix #1377Jan Niklas Hasse2019-08-021-16/+28
|
* Improve const-correctness in compdb related methodsJan Niklas Hasse2019-08-021-1/+2
|
* Resurrect the 'rules' tool.Nicolas Despres2019-05-091-0/+52
| | | | | | | | | This tool is useful for writing shell completion script for tools expecting a rule name as argument. The tool was dropped by 34b46f28c. Fix #1024.
* query: load dyndep files for queried edgesBrad King2019-04-181-0/+7
|
* graph: load dyndep filesBrad King2019-04-181-1/+1
| | | | | | Teach the `-t graph` tool to load dyndep files because they are part of the build graph. Issue a warning when the dyndep file cannot be loaded cleanly. This will help users visualize the complete build graph.
* clean: remove unnecessary Cleaner constructor variantBrad King2019-04-181-1/+1
| | | | | | | | `Cleaner` provides two constructors that are the same except that one constructs a "real" disk interface internally and the other takes a caller-provided disk interface. A real disk interface is already available at the only call site for the former constructor. Use it directly and drop the unnecessary constructor variant.
* Restore depfile toleration of multiple output paths on distinct linesBrad King2018-11-191-1/+18
| | | | | | | | | | | | | | | | Prior to introduction of depfile parser handling of multiple rules, ninja silently accepted a depfile of the form: out: in1 in2 in3 other: otherIn1 otherIn2 otherIn3 and incorrectly treated `other` and `otherIn*` as additional inputs to `out`. Now we prefer to reject this just as we already do for a depfile specifying multiple outputs on one line. However, this can break existing cases where such a depfile was silently tolerated. Add a `-w depfilemulti={err,warn}` option to control this behavior, and make it just a warning by default.
* Fit --help output into 80 columns and move verbose up, fix #1500Jan Niklas Hasse2018-11-161-3/+3
|
* Explain why there's no browse tool, fix #1478Jan Niklas Hasse2018-11-141-3/+6
|
* Make -j 0 run unlimited parallel builds, fix #1309Martell Malone2018-11-141-3/+6
|
* Add --verbose as an alternative spelling for -v, fix #1310Jan Niklas Hasse2018-11-131-1/+2
|
* Misc typo fixes by https://github.com/codespell-project/codespell/Mo Zhou2018-09-061-1/+1
|
* 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.
generated by cgit v0.12 at 2024-08-30 00:39:16 (GMT)