| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The change in commit v1.2.0~3^2~3^2~3 (Fix duplicate edge Pool crash in
the minimally invasive way, 2013-03-18) avoids double-scheduling in a
case involving duplicate out edges. However, double-scheduling may also
occur on a consistent graph when an edge and one of its dependencies
share an order-only input:
$ cat build.ninja
...
build c: touch
build b: touch || c
build a: touch | b || c
$ ninja a
$ rm a c
$ ninja a
In this case 'c' will build first. When NodeFinished('c') loops over
the out edges it will find AllInputsReady is true for 'b' and call
EdgeFinished('b') since it is not wanted (up to date). This will
call NodeFinished('b') which will loop over its out edges, find
AllInputsReady is true for 'a', and call ScheduleEdge('a'). When
we eventually return to the loop in NodeFinished('c') it will move
on to its second output and find that AllInputsReady is true for
'a' and call ScheduleEdge('a') again.
Teach ScheduleEdge to tolerate duplicate calls for an edge that has
already been scheduled. Avoid calling EdgeScheduled more than once
for the same edge.
|
| |\ \
| | | |
| | | | |
Make deps=msvc experimentally available on non-Windows.
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This makes it possible to run most of the clparser tests on non-Windows,
and is potentially useful for cross-compiling on non-Windows hosts.
Also, the manual didn't document this as Windows-only previously.
If you use this on non-Windows, please let me know, else I might undo
this change again in the future.
|
| |\ \
| | | |
| | | | |
Use posix_spawn() instead of fork()/exec().
|
| | | |
| | | |
| | | |
| | | |
| | | | |
posix_spawn() is a syscall on OS X and Solaris and a bit faster. It's
also easier emulate for cygwin, and the code is a bit simpler.
|
| |\ \ \
| | |/ /
| |/| | |
browse tool: Allow customization of port number, Py3 fix
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | | |
|
| | | | |
|
| |\ \ \
| | | | |
| | | | | |
Revert #910.
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The change caused some issues (it makes it impossible ot use
posix_spawn() and makes it harder to suspend children on ctrl-z). After
discussing with jln: Since it fixes a corner case that can be fixed by
explicitly running commands that need it in a wrapper that setsid()s
them, let's try reverting it for a while. Please shout if this is a
problem for you.
See also #1097.
|
| |\ \ \
| | |/ /
| |/| | |
Remove StatIfNecessary call that is never necessary
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The call to StatIfNecessary in DependencyScan::RecomputeOutputsDirty was
added by commit v1.4.0^2~7^2~1 (Share more code between CleanNode() and
RecomputeDirty(), 2013-09-02) while consolidating code paths. However,
it was needed only when called from RecomputeDirty because prior to
refactoring the CleanNode code path did not call it.
Later commit v1.6.0^2~46^2 (Let DependencyScan::RecomputeDirty() work
correclty with cyclic graphs, 2014-12-07) added back to RecomputeDirty a
loop over outputs that calls StatIfNecessary. Therefore
RecomputeOutputsDirty no longer needs to call StatIfNecessary for either
of its own callers.
|
| |\ \
| | | |
| | | | |
Fix ambiguous call to set_terminate on Windows platform
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
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>
|
| |\ \
| | | |
| | | | |
Add support for implicit outputs in ninja_syntax.py.
|
| | | | |
|
| |\ \ \
| | |/ /
| |/| | |
Add link to mailing list
|
| |/ /
| | |
| | | |
As a convenience to new users, provide a link to the mailing list.
|
| |\ \
| | | |
| | | | |
win: Only drop /showIncludes in bootstrap builds.
|
| | | |
| | | |
| | | |
| | | | |
Fixes #1103.
|
| |\ \ \
| | | | |
| | | | | |
disable warning about conditional expressions
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The standard headers for Visual Studio 2008 generate a warning about
constant conditional expressions when compiled with exception support
disabled. This is caused by the _CATCH_ALL macro in xstddef which is
defined thusly:
#ifdef _HAS_EXCEPTIONS
#define _CATCH_ALL } catch (...) {
#else
#define _CATCH_ALL } if (0) {
#endif
|
| |\ \ \
| | |_|/
| |/| | |
Minor updates to the manual.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Update link to Chromium's ninja docs (fixes #1038)
* Update cmake URL to what it redirects to, and mention that ninja
is well-supported on all platforms in newer CMake versions.
* Let "others" link to the wiki page listing generators.
|
| |\ \ \
| | | | |
| | | | | |
Print output file on failure
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Modify the FAILED: output to provide the output files that failed to
build, followed by the failed command on the next line. This makes the
failure much easier to read, as you can immediately see much shorter
name of the file that failed instead of trying to parse a very long
command line. It also makes manually re-running the failed command much
easier because you can copy the whole line without ending up with the
FAILED: prefix.
|
| |\ \ \ \
| | |_|/ /
| |/| | | |
Add support for build statement implicit outputs
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Some build rules produce outputs that are not mentioned on the command
line but that should be part of the build graph. Such outputs should
not be named in the `$out` variable. Extend the build statement syntax
to support specification of implicit outputs using the syntax
`| out1 out2` after the explicit outputs and before the `:`.
For example, compilation of a Fortran source file `foo.f90` that defines
`MODULE FOO` may now be specified as:
rule fc
command = f95 -c $in -o $out
build foo.o | foo.mod: fc foo.f90
The `foo.mod` file is an implicit output generated by the compiler based
on the content of the source file and not mentioned on the command line.
|
| |\ \ \ \
| | | | | |
| | | | | | |
Deduplicate disk abstraction infrastructure
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Avoid having two separate filesystem interfaces. Simplify test
infrastructure by avoiding custom `ManifestParser::FileReader`
implementations.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Return a status so callers can distinguish a missing file from an empty
file. This allows our VirtualFileSystem test infrastructure to report
as missing any file for which it has no entry.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Some clients will need only the ability to read files, so provide this
as a more narrow interface than the full disk interface.
|
| |\ \ \ \ \
| | |/ / / /
| | | / / /
| | |/ / /
| |/| | | |
perftest: fix compilation after dupe_edge_should_err change
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fix some ManifestParser constructor calls missed by commit 56bab441b7
(dupe_edge_should_err from bool to enum, 2016-01-27).
|
| |\ \ \ \
| | |/ / /
| |/| | | |
Canonicalize "." to "."
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
"." is a legal path, if the string is empty after canonicalization
return ".".
|
| |\ \ \ \
| | | | | |
| | | | | | |
dupe_edge_should_err from bool to enum
|
| | | | | | |
|
| |\ \ \ \ \
| | |/ / / /
| |/| | | | |
Add -d keepdepfile to preserve depfiles
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | | |
This is useful when you are developing a tool which generates
GCC-style depfiles.
|
| |\ \ \ \
| | | | | |
| | | | | | |
Make dupbuild=err work in subninja
|
| |/ / / / |
|
| |\ \ \ \
| | | | | |
| | | | | | |
fix for localized version of cl.exe
|
| | | | | | |
|
| |\ \ \ \ \
| | |/ / / /
| |/| | | | |
Remove unnecessary `std::`
|
| | | | | | |
|
| |\ \ \ \ \
| | |/ / / /
| |/| | | | |
Use container-based infrastructure for travis-ci.
|
| |/ / / /
| | | | |
| | | | |
| | | | | |
To start up faster.
|
| |\ \ \ \
| | | | | |
| | | | | | |
use the default font size for manual headings
|