summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| | * Avoid double-scheduling build edges in another caseBrad King2016-02-032-7/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Merge pull request #1125 from nico/wintoolsNico Weber2016-04-069-211/+290
| |\ \ | | | | | | | | Make deps=msvc experimentally available on non-Windows.
| | * | Make deps=msvc experimentally available on non-Windows.Nico Weber2016-04-069-211/+290
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Merge pull request #1118 from nico/posixspawn2Nico Weber2016-04-052-54/+56
| |\ \ | | | | | | | | Use posix_spawn() instead of fork()/exec().
| | * | Use posix_spawn() instead of fork()/exec().Nico Weber2016-03-212-54/+56
| | | | | | | | | | | | | | | | | | | | 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.
| * | | Merge pull request #1112 from Lekensteyn/browse-tool-fixesNico Weber2016-04-045-21/+46
| |\ \ \ | | |/ / | |/| | browse tool: Allow customization of port number, Py3 fix
| | * | browse: support --port and --no-browser optionsPeter Wu2016-03-115-20/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| | * | browse.py: Python 3 compatibilityPeter Wu2016-03-111-2/+5
| | | |
| | * | browse.py: allow port override via environment variablePeter Wu2016-02-252-2/+3
| | | |
| * | | Merge pull request #1113 from nico/nosetsidNico Weber2016-03-052-31/+3
| |\ \ \ | | | | | | | | | | Revert #910.
| | * | | Revert #910.Nico Weber2016-03-052-31/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | Merge pull request #1102 from bradking/remove-unnecessary-StatIfNecessaryNico Weber2016-03-051-2/+1
| |\ \ \ | | |/ / | |/| | Remove StatIfNecessary call that is never necessary
| | * | Remove StatIfNecessary call that is never necessaryBrad King2016-02-251-2/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Merge pull request #1110 from G4m4/explicit_std_prefixNico Weber2016-02-171-1/+1
| |\ \ | | | | | | | | Fix ambiguous call to set_terminate on Windows platform
| | * | 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>
| * | Merge pull request #1107 from nicolasdespres/ninja_syntax_implicit_outputsNico Weber2016-02-072-1/+13
| |\ \ | | | | | | | | Add support for implicit outputs in ninja_syntax.py.
| | * | Add support for implicit outputs in ninja_syntax.py.Nicolas Despres2016-02-062-1/+13
| | | |
| * | | Merge pull request #1108 from edbaunton/patch-1Nico Weber2016-02-071-2/+2
| |\ \ \ | | |/ / | |/| | Add link to mailing list
| | * | Add link to mailing listEd Baunton2016-02-061-2/+2
| |/ / | | | | | | As a convenience to new users, provide a link to the mailing list.
| * | Merge pull request #1106 from nico/winfixNico Weber2016-02-051-5/+3
| |\ \ | | | | | | | | win: Only drop /showIncludes in bootstrap builds.
| | * | win: Only drop /showIncludes in bootstrap builds.Nico Weber2016-02-051-5/+3
| | | | | | | | | | | | | | | | Fixes #1103.
| * | | Merge pull request #1101 from fbenkstein/vs2010Nico Weber2016-02-051-0/+2
| |\ \ \ | | | | | | | | | | disable warning about conditional expressions
| | * | | MSVC: disable warning about constant conditional expressionsFrank Benkstein2016-02-051-0/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | | Merge pull request #1100 from nico/docNico Weber2016-02-031-9/+5
| |\ \ \ | | |_|/ | |/| | Minor updates to the manual.
| | * | Minor updates to the manual.Nico Weber2016-02-031-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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.
| * | | Merge pull request #1033 from colincross/failedNico Weber2016-02-031-2/+9
| |\ \ \ | | | | | | | | | | Print output file on failure
| | * | | Print output file on failureColin Cross2015-10-051-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | | Merge pull request #989 from bradking/implicit-outputsNico Weber2016-02-037-3/+148
| |\ \ \ \ | | |_|/ / | |/| | | Add support for build statement implicit outputs
| | * | | Add support for build statement implicit outputsBrad King2016-02-037-3/+148
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | | Merge pull request #1060 from bradking/deduplicate-disk-interfaceNico Weber2016-02-0312-91/+99
| |\ \ \ \ | | | | | | | | | | | | Deduplicate disk abstraction infrastructure
| | * | | | Replace ManifestParser::FileReader with general FileReaderBrad King2016-02-035-63/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid having two separate filesystem interfaces. Simplify test infrastructure by avoiding custom `ManifestParser::FileReader` implementations.
| | * | | | Expose more details in FileReader::ReadFile signatureBrad King2016-02-037-24/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| | * | | | Factor a `FileReader` base class out of `DiskInterface`Brad King2016-02-031-6/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some clients will need only the ability to read files, so provide this as a more narrow interface than the full disk interface.
| * | | | | Merge pull request #1099 from bradking/fix-perftest-compilationNico Weber2016-02-032-2/+2
| |\ \ \ \ \ | | |/ / / / | | | / / / | | |/ / / | |/| | | perftest: fix compilation after dupe_edge_should_err change
| | * | | perftest: fix compilation after dupe_edge_should_err changeBrad King2016-02-022-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | Fix some ManifestParser constructor calls missed by commit 56bab441b7 (dupe_edge_should_err from bool to enum, 2016-01-27).
| * | | | Merge pull request #1076 from colincross/canonicalizeNico Weber2016-02-022-18/+18
| |\ \ \ \ | | |/ / / | |/| | | Canonicalize "." to "."
| | * | | Canonicalize "." to "."Colin Cross2015-12-162-18/+18
| | | | | | | | | | | | | | | | | | | | | | | | | "." is a legal path, if the string is empty after canonicalization return ".".
| * | | | Merge pull request #1096 from sgraham/dupe-edge-actionNico Weber2016-01-295-54/+61
| |\ \ \ \ | | | | | | | | | | | | dupe_edge_should_err from bool to enum
| | * | | | dupe_edge_should_err from bool to enumScott Graham2016-01-275-54/+61
| | | | | |
| * | | | | Merge pull request #1098 from shinh/keepdepNico Weber2016-01-294-8/+19
| |\ \ \ \ \ | | |/ / / / | |/| | | | Add -d keepdepfile to preserve depfiles
| | * | | | Add -d keepdepfile to preserve depfilesShinichiro Hamaji2016-01-284-8/+19
| |/ / / / | | | | | | | | | | | | | | | | | | | | This is useful when you are developing a tool which generates GCC-style depfiles.
| * | | | Merge pull request #1095 from sgraham/dupbuild-errNico Weber2016-01-274-47/+63
| |\ \ \ \ | | | | | | | | | | | | Make dupbuild=err work in subninja
| | * | | | Make dupbuild=err work in subninjaScott Graham2016-01-274-47/+63
| |/ / / /
| * | | | Merge pull request #1094 from sphawk/masterNico Weber2016-01-271-1/+1
| |\ \ \ \ | | | | | | | | | | | | fix for localized version of cl.exe
| | * | | | fix for localized version of cl.exesphawk2016-01-251-1/+1
| | | | | |
| * | | | | Merge pull request #1085 from tetsuok/remove-unnecessary-stdNico Weber2016-01-275-11/+11
| |\ \ \ \ \ | | |/ / / / | |/| | | | Remove unnecessary `std::`
| | * | | | Remove unnecessary `std::`Tetsuo Kiso2016-01-105-11/+11
| | | | | |
| * | | | | Merge pull request #1086 from tetsuok/use-container-based-infra-for-ciEvan Martin2016-01-101-0/+1
| |\ \ \ \ \ | | |/ / / / | |/| | | | Use container-based infrastructure for travis-ci.
| | * | | | Use container-based infrastructure for travis-ci.Tetsuo Kiso2016-01-101-0/+1
| |/ / / / | | | | | | | | | | | | | | | To start up faster.
| * | | | Merge pull request #1073 from ninja-build/manual-fontNico Weber2015-12-291-6/+0
| |\ \ \ \ | | | | | | | | | | | | use the default font size for manual headings
generated by cgit v0.12 at 2024-08-25 06:00:58 (GMT)