summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Treat paths of the form "existing-file/something" as non-existentPeter Collingbourne2012-03-072-13/+15
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some people like to construct phony target names by appending a "/something" suffix to an existing target "foo". But if "foo" is an existing file, stat will report ENOTDIR for this path, causing ninja to spew errors. Fix this by treating ENOTDIR in the same way as we do ENOENT -- as a non-existent path.
* | | | drop rtti via -fno-rtti; we don't use itEvan Martin2012-03-061-0/+1
| | | |
* | | | make urtle fit on 24-line terminalEvan Martin2012-03-061-1/+1
| | | |
* | | | adjust rspfile docs to properly renderEvan Martin2012-03-061-16/+16
| | | |
* | | | Merge pull request #238 from pcc/outputs-ready-no-inputsEvan Martin2012-03-052-4/+30
|\ \ \ \ | | | | | | | | | | Mark a phony target with no inputs as outputs-ready
| * | | | Mark a phony target with no inputs as outputs-readyPeter Collingbourne2012-03-052-4/+30
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Even if such a target is dirty (i.e. the file does not exist), it has nothing to do, which makes it safe to mark as outputs-ready. Without this change, ninja will print no output when rebuilding the target (or an order-only dependency thereof), instead of reporting it has "no work to do".
* | | | Merge pull request #237 from pcc/ninja-clean-hintEvan Martin2012-03-051-3/+7
|\ \ \ \ | |/ / / |/| | | Give a useful hint if the user runs "ninja clean" and there is no "clean...
| * | | Give a useful hint if the user runs "ninja clean" and there is no "clean" targetPeter Collingbourne2012-03-051-3/+7
|/ / /
* | | Merge pull request #231 from pcc/exit-cleanup-modifiedEvan Martin2012-03-023-5/+78
|\ \ \ | | | | | | | | Be more selective about deleting output files when interrupted
| * | | Be more selective about deleting output files when interruptedPeter Collingbourne2012-03-013-5/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Specifically, only delete if the file was modified or if the rule uses a depfile. Fixes issue #226.
* | | | add an "urtle" toolEvan Martin2012-03-011-0/+28
|/ / /
* | | Merge pull request #229 from sgraham/overprintEvan Martin2012-02-281-5/+38
|\ \ \ | |_|/ |/| | [win32] Overprint the previous line if we're attached to a console
| * | [win32] Overprint the previous line if we're attached to a consoleScott Graham2012-02-271-5/+38
|/ /
* | fix some public/private errors in rspfile patchEvan Martin2012-02-233-5/+7
| |
* | Merge pull request #217 from PetrWolf/masterEvan Martin2012-02-2315-8/+330
|\ \ | | | | | | Response files
| * \ Merge branch 'master' of github.com:PetrWolf/ninjaPetrWolf2012-02-142-1/+11
| |\ \
| | * | Add comments, remove unwanted whitespace changePetr Wolf2012-02-102-1/+11
| | | |
| * | | Fix tabs, typosPetrWolf2012-02-142-5/+5
| |/ /
| * | Response filesunknown2012-02-0915-9/+321
| | |
* | | Merge pull request #218 from tfarina/fix-graph-todoEvan Martin2012-02-223-7/+6
|\ \ \ | | | | | | | | Fix the TODO in graph.h.
| * | | fix depfile_ accessThiago Farina2012-02-161-1/+1
| | | |
| * | | Fix the TODO in graph.h.Thiago Farina2012-02-162-6/+5
| | | | | | | | | | | | | | | | Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | | | Merge pull request #223 from pcc/restat-clean-manifestEvan Martin2012-02-221-1/+6
|\ \ \ \ | | | | | | | | | | Do not reload the manifest if a restat cleans it while being rebuilt
| * | | | Do not reload the manifest if a restat cleans it while being rebuiltPeter Collingbourne2012-02-211-1/+6
| |/ / /
* | | | Merge pull request #222 from polrop/test-bug-fix-62e9139740Evan Martin2012-02-201-0/+3
|\ \ \ \ | |/ / / |/| | | Add a test for empty path in CanonicalizePath().
| * | | Add a test for empty path in CanonicalizePath().Nicolas Despres2012-02-201-0/+3
|/ / / | | | | | | | | | | | | | | | | | | This test covers bug fix introduced by 62e9139740. However, reverting this patch does not trigger a test failure. Maybe, I am not testing on the right platform (Linux). Anyway, in all cases I think this test deserves to be added.
* | | Merge pull request #176 from pcc/exit-cleanupEvan Martin2012-02-149-111/+360
|\ \ \ | | | | | | | | Implement cleanup-on-interrupt
| * | | If a command fails, wait for all running commands to terminate before we doPeter Collingbourne2012-02-044-21/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, if a command fails, the fate of the other child processes running in parallel was inadequately controlled. On POSIX platforms, the processes were orphaned. Normally they would run to completion, but were liable to being killed by a SIGPIPE. On Windows, the child processes would terminate with the parent. The cleanup-on-interrupt patch caused the SubprocessSet and Builder destructors to clean up after themselves by killing any running child processes and deleting their output files, making the behaviour more predictable and consistent across platforms. If the build is interrupted by the user, this is correct behaviour. But in the case where the build is stopped by a failed command, this would be inconsistent with user expectations. In the latter case, we now let any remaining child processes run to completion before leaving the main loop in Builder::Build.
| * | | Implement cleanup-on-interruptPeter Collingbourne2012-02-048-84/+318
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This causes us to clean up by deleting any output files belonging to currently-running commands before we quit if we are interrupted (either by Ctrl-C or by a command failing). Fixes issue #110.
| * | | Clean up how the build line is managed at the end of the buildPeter Collingbourne2012-02-041-6/+20
| | | |
* | | | graphviz: don't draw edges multiple timesEvan Martin2012-02-142-3/+9
| | | | | | | | | | | | | | | | From a patch from Ian Godin <iangodin@gmail.com>.
* | | | Made graph horizontal (better for filenames)Ian Godin2012-02-141-0/+1
| |/ / |/| |
* | | Fixes the build on OpenIndianaClint Moore2012-02-071-0/+2
| | |
* | | Merge pull request #214 from qhuo/onpipeready-fixEvan Martin2012-02-042-2/+4
|\ \ \ | | | | | | | | Avoid using undefined value in Subprocess::OnPipeReady()
| * | | Avoid using undefined value in Subprocess::OnPipeReady()Qingning Huo2012-02-042-2/+4
| |/ /
* | | Merge pull request #213 from syntheticpp/evaluate-only-onceEvan Martin2012-02-041-3/+2
|\ \ \ | | | | | | | | Evaluate command only once
| * | | compute command twice only in case of an errorPeter Kuemmel2012-02-041-3/+2
| |/ /
* | | don't crash when CanonicalizePath removes all path componentsEvan Martin2012-02-042-0/+22
|/ / | | | | | | From a patch from Peter Kuemmel <syntheticpp@gmx.net>.
* | Merge pull request #211 from okuoku/msvc-build-c4819Evan Martin2012-02-041-1/+1
|\ \ | | | | | | windows: Disable warning C4819
| * | windows: Disable warning C4819okuoku2012-02-031-1/+1
|/ / | | | | | | | | warning C4819: The file contains a character that cannot be represented in the current code page.
* | Merge pull request #199 from qhuo/create-processEvan Martin2012-01-292-2/+38
|\ \ | | | | | | Mark CreateProcess "program not found" failure as non-fatal
| * | Add a test, NoSuchCommand.Qingning Huo2012-01-251-0/+18
| | |
| * | Mark CreateProcess "program not found" failure as non-fatalQingning Huo2012-01-201-2/+20
| | |
* | | Merge pull request #210 from syntheticpp/colon-escapeEvan Martin2012-01-296-27/+115
|\ \ \ | | | | | | | | add colon escaping
| * | | parse $:Peter Kuemmel2012-01-295-4/+78
| | | | | | | | | | | | | | | | | | | | '$:' is a valid string now, it expands to ':' update error messages and show a hint when something went wrong.
| * | | add colon escapingPeter Kuemmel2012-01-293-23/+37
|/ / / | | | | | | | | | | | | | | | | | | | | | Needed for Windows drive names. For instance configure with gtest: python configure.py --with-gtest=c$:\gtest-1.6.0
* | | Merge pull request #209 from scottfranklin/python3_compatEvan Martin2012-01-291-2/+2
|\ \ \ | | | | | | | | Maintain python3 compatibility for ninja_syntax.py
| * | | Explicitly turn input/output lists into list objects.Scott Franklin2012-01-281-2/+2
|/ / / | | | | | | | | | Python 3 does not support map.append/extend.
* | | Merge pull request #208 from okuoku/cygwin-buildEvan Martin2012-01-262-1/+1
|\ \ \ | |_|/ |/| | Fix cygwin build by including stdio.h here
| * | Fix cygwin build by including stdio.h hereokuoku2012-01-252-1/+1
|/ /
generated by cgit v0.12 at 2025-04-24 10:43:05 (GMT)