summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
| * | 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-044-19/+39
|\ \ \ | |/ / |/| | browse tool: Allow customization of port number, Py3 fix
| * | browse: support --port and --no-browser optionsPeter Wu2016-03-114-17/+34
| | | | | | | | | | | | | | | | | | | | | 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-251-1/+1
| | |
* | | 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.
* | | 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.
* | 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 #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-036-2/+122
|\ \ | | | | | | Add support for build statement implicit outputs
| * | Add support for build statement implicit outputsBrad King2016-02-036-2/+122
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| | | |
* | | | 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.
* | | Make dupbuild=err work in subninjaScott Graham2016-01-274-47/+63
| | |
* | | Remove unnecessary `std::`Tetsuo Kiso2016-01-105-11/+11
| | |
* | | Do not attempt to rebuild infinitely in dry-run mode.Nicolas Despres2015-12-091-0/+4
| | |
* | | Merge pull request #1062 from tfarina/bindings-typedefEvan Martin2015-12-062-3/+3
|\ \ \ | | | | | | | | make use of Bindings typedef
| * | | make use of Bindings typedefThiago Farina2015-12-042-3/+3
| |/ / | | | | | | | | | | | | | | | | | | Looks like we declared this typedef but never used it. It seems we just forgot to use it so this patch makes uses of it now. Otherwise we could just delete it.
* | | add "msvc_deps_prefix" to Rule::IsReservedBinding()Niklas Rosenstein2015-12-061-1/+2
|/ / | | | | | | | | | | | | | | | | fixes the error about an unexpected variable for a rule that declares the msvc_deps_prefix. The manual suggests that this should work since Ninja 1.5 (https://ninja-build.org/manual.html#ref_rule). Closes #1043
* | Merge pull request #1007 from mikesep/aixEvan Martin2015-11-127-4/+31
|\ \ | | | | | | Support for AIX
| * | Fix getopt for AIXMike Seplowitz2015-08-194-0/+10
| | | | | | | | | | | | | | | | | | | | | 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.
| * | Implement GetLoadAverage on AIX using libperfstatMike Seplowitz2015-08-191-0/+12
| | |
| * | Fix AIX compilation error related to printf macrosMike Seplowitz2015-08-191-3/+8
| | | | | | | | | | | | | | | On AIX, inttypes.h gets indirectly included by build_log.h. It's easiest just to ask for the printf format macros right away.
| * | Strip tabs from od's outputMike Seplowitz2015-08-191-1/+1
| | | | | | | | | | | | /bin/od on Solaris and AIX both generate tabs.
* | | Merge pull request #999 from colincross/dumbstatusNico Weber2015-11-121-2/+3
|\ \ \ | | | | | | | | Print status when edge finishes on dumb terminals
| * | | Print status when edge finishes on dumb terminalsColin Cross2015-07-311-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On smart terminals ninja prints the status line both before and after running a command, reusing the same line if possible. On a dumb terminal that doesn't support reusing the line, it only prints the status before starting the command, but prints the output of the command when the command finishes, by which point other commands may have started and printed their status line. This makes it impossible to determine what command produced a line of output. Modify BuildEdgeStarted to only print the status line if the command is going to lock the console, or if ninja is running on a smart terminal. Modify BuildEdgeFinished to always print the status line unless the command locked the console, in which case the status was already printed and no other command can have printed any lines. The end result will be dumb terminal output that much more closely matches smart terminal output. One disadvantage is that dumb terminals won't show anything when starting a command, making it harder to tell what commands are currently running, but I expect most interactive uses of ninja will use a smart terminal.
* | | | Merge pull request #964 from nicolasdespres/sighupNico Weber2015-11-123-1/+36
|\ \ \ \ | | | | | | | | | | Cleanup build on SIGHUP.
| * | | | Cleanup build on SIGHUP.Nicolas Despres2015-09-203-1/+36
| | |_|/ | |/| | | | | | | | | | | | | | SIGHUP is sent when the connection hang up (i.e. when the terminal window is closed or the ssh connection is closed).
* | | | Merge pull request #763 from drbo/masterNico Weber2015-11-123-31/+38
|\ \ \ \ | | | | | | | | | | Allow paths with international characters in depfiles
| * \ \ \ Merge remote-tracking branch 'upstream/master'Demetri Obenour2014-05-3112-60/+124
| |\ \ \ \
| * | | | | Added test of international character supportDemetri Obenour2014-05-082-29/+36
| | | | | |
| * | | | | Added support for international characters in makefile dependency namesDemetri Obenour2014-05-081-2/+2
| | | | | |
* | | | | | Make links point to org pageNico Weber2015-11-114-8/+8
| | | | | |
* | | | | | Fix typo.Nicolas Despres2015-09-241-1/+1
| |_|/ / / |/| | | |
* | | | | Merge pull request #1014 from moroten/optimize-stringpieceNico Weber2015-09-071-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Optimize StringPiece hash map
| * | | | | Optimize StringPiece hash mapFredrik Medley2015-09-031-1/+1
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | Replace strncmp with memcmp to improve performance. Signed-off-by: Fredrik Medley <fredrik.medley@gmail.com>
* | | | | Merge LLVM r242069, make spell checking allocate less memory.Nico Weber2015-09-071-18/+20
|/ / / /
* | | | Remove accidental changesScott Graham2015-08-102-1105/+1105
| | | |
* | | | Some mucking with std:: for set_terminate and terminate_handlerScott Graham2015-08-104-1109/+1108
| | | |
* | | | Make sure not to re-define __STDC_FORMAT_MACROS.Lindley French2015-07-311-0/+2
| |_|/ |/| |