summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | Merge pull request #1196 from danw/ReadFile_optJan Niklas Hasse2018-11-141-1/+11
|\ \ \ \ \ \ | | | | | | | | | | | | | | Optimize ReadFile allocations
| * | | | | | Optimize ReadFile allocationsDan Willemsen2016-11-071-1/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of continuously reallocating the output string, call reserve() up front to enlarge the string's buffer. Also use feof() instead of an empty read to detect the end of the file. This saves a syscall on <64kb files, which is about 5% (200ns) of the time to read a file on my machine. For our larger build.ninja files (~550MB), this saves about 500ms, which is more than half the time spent in this function. (Measured by adding METRICS_RECORD to this function during manifest_parser_perftest) For the standard manifest_parser_perftest, this only saves ~20ms out of ~600ms for a full run.
* | | | | | | Improve error message when Python wasn't found for the browse toolJan Niklas Hasse2018-11-141-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | See #1306.
* | | | | | | Explain why there's no browse tool, fix #1478Jan Niklas Hasse2018-11-141-3/+6
| |/ / / / / |/| | | | |
* | | | | | Fix rendering of code block, fix #955Jan Niklas Hasse2018-11-141-1/+1
| | | | | |
* | | | | | Make -j 0 run unlimited parallel builds, fix #1309Martell Malone2018-11-141-3/+6
| |_|/ / / |/| | | |
* | | | | Merge pull request #1264 from gahr/DirName-compile-timeJan Niklas Hasse2018-11-131-3/+4
|\ \ \ \ \ | | | | | | | | | | | | DirName's separators and their length are known at compile time
| * | | | | DirName's separators and their length are known at compile timePietro Cerutti2017-04-111-3/+4
| | | | | |
* | | | | | Add --verbose as an alternative spelling for -v, fix #1310Jan Niklas Hasse2018-11-131-1/+2
| | | | | |
* | | | | | Add unit test for #1491Jan Niklas Hasse2018-11-131-0/+2
| | | | | |
* | | | | | Merge pull request #1491 from mathstuf/elision-off-by-nJan Niklas Hasse2018-11-131-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | util: don't add ellipses width when deciding if they're necessary
| * | | | | | util: don't add ellipses width when deciding if they're necessaryBen Boeckel2018-11-091-1/+1
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | If the string fits, just use it. If we need the ellipses, *then* we need to compute the width based on that.
* | | | | | Allow disabling of escape code stripping, fix #1475Jan Niklas Hasse2018-11-133-6/+20
|/ / / / / | | | | | | | | | | | | | | | | | | | | Don't strip colors when CLICOLOR_FORCE is set to a non-zero value. This environment variable is also used by CMake's Make back-end.
* | | | | Merge pull request #1307 from moroten/fix-too-long-path-test-bugJan Niklas Hasse2018-11-083-14/+74
|\ \ \ \ \ | | | | | | | | | | | | Verify GetFullPathName return value
| * | | | | Verify GetFullPathName return valueFredrik Medley2018-11-083-14/+74
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | GetFullPathName previously failed silently on long path names resulting in uninitialized path result. Signed-off-by: Fredrik Medley <fredrik.medley@autoliv.com>
* | | | | Make output_test.py independent of the environmentJan Niklas Hasse2018-11-071-2/+8
| | | | |
* | | | | Merge pull request #1488 from maciej-pawlowski-opera/issue-1227-2Jan Niklas Hasse2018-11-074-510/+431
|\ \ \ \ \ | | | | | | | | | | | | Fix parsing some special chars in depfiles (take 2)
| * | | | | Fix parsing some special chars in depfilesMaciej Pawlowski2018-11-073-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows paths with "[", "]" and "%" to appear in depfiles. Previously, only "[" would be handled properly. Fixes #1227.
| * | | | | Regenerate depfile_parser.cc and lexer.cc with newer re2cMaciej Pawlowski2018-11-072-505/+423
| | | | | |
* | | | | | Fix older VS compatibility issues and PDB files generation issue. (#1435)ikifof2018-11-073-8/+21
|/ / / / / | | | | | | | | | | Fixes #1411.
* | | | | Merge pull request #1473 from zdave/query-term-stdoutJan Niklas Hasse2018-11-051-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Query terminal width from STDOUT_FILENO
| * | | | | Query terminal width from STDOUT_FILENODavid Emett2018-09-291-1/+1
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | Rather than 0 (stdin). So it will work even if ninja's stdin isn't connected to the terminal for whatever reason.
* | | | | Exclude broken DiskInterfaceTests from MSVC, see #1423Jan Niklas Hasse2018-11-051-0/+2
| | | | |
* | | | | Merge pull request #1326 from 0-wiz-0/masterJan Niklas Hasse2018-11-052-1/+2
|\ \ \ \ \ | | | | | | | | | | | | Two fixes for NetBSD
| * | | | | Add NetBSD in another clause.Thomas Klausner2017-09-101-1/+1
| | | | | |
| * | | | | Include missing header for pselect().Thomas Klausner2017-09-101-0/+1
| | | | | |
* | | | | | Merge pull request #1457 from V-for-Vasili/AIX-supportJan Niklas Hasse2018-11-052-0/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | Port to AIX platform
| * | | | | | Port to AIX platform.Vasili Skurydzin2018-08-022-0/+7
| | |/ / / / | |/| | | | | | | | | | | | | | | | Taking care of printf format specifiers and large files using compiler macros in configure.py
* | | | | | Add test for #1418 (edge output should match status)Jan Niklas Hasse2018-11-051-0/+21
| | | | | |
* | | | | | Revert "Fix confusing smart console output from concurrent builds"Jan Niklas Hasse2018-11-052-14/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 52c1d0c8f8545231581c4d51cb0a85f50564c415. Fixes #1418.
* | | | | | Add script to test Ninja's outputJan Niklas Hasse2018-11-052-1/+67
| | | | | | | | | | | | | | | | | | | | | | | | First test checks for #1214.
* | | | | | Do not always strip colored output in verbose mode, fix #1214Jan Niklas Hasse2018-11-053-1/+7
| | | | | |
* | | | | | Ignore Visual Studio Code project filesJan Niklas Hasse2018-11-021-0/+3
| | | | | |
* | | | | | AppVeyor: Also build with MinGWJan Niklas Hasse2018-11-021-7/+33
| | | | | |
* | | | | | Merge pull request #1472 from Hi-Angel/masterJan Niklas Hasse2018-11-021-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix wrong description of script installation
| * | | | | | Fix wrong description of script installationKonstantin Kharlamov2018-09-281-1/+1
| |/ / / / / | | | | | | | | | | | | The described way of installation makes zsh fail with `_arguments:comparguments:325: can only be called from completion function`. Per [zsh documentation](https://github.com/zsh-users/zsh-completions/blob/master/zsh-completions-howto.org#telling-zsh-which-function-to-use-for-completing-a-command) the correct way is to use `$fpath`.
* | | | | | Merge pull request #1299 from pcc/fflushJan Niklas Hasse2018-10-301-0/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Flush stdout after printing "Cleaning..." message
| * | | | | | Flush stdout after printing "Cleaning..." messagePeter Collingbourne2017-07-051-0/+1
| | | | | | |
* | | | | | | Add more instructions for building ninja in Windows (#1169)Andrew Maclean2018-10-301-2/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This should clarify that the instructions are for Visual Studio. It also opens the possibility for others to write similar sections for using gcc/clang in Windows.
* | | | | | | Merge pull request #1481 from rickysarraf/fix-testcase-whoamiJan Niklas Hasse2018-10-291-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix testcase failure in docker environment
| * | | | | | | Replace `whoami` with a more generic commandRitesh Raj Sarraf2018-10-121-1/+1
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In our docker environment, the normal user does not have a name. This results in the `whoami` command to fail which expects a name to print Replace `whoami` with `id -u`, which print print the numeric id I have no name!@7427761b8f4c:/tmp/d$ whoami whoami: cannot find name for user ID 1000 I have no name!@7427761b8f4c:/tmp/d$ id uid=1000 gid=0(root) groups=0(root) Signed-off-by: Ritesh Raj Sarraf <ritesh.sarraf@collabora.com>
* | | | | | | Merge pull request #1485 from nomis/freebsd-11Jan Niklas Hasse2018-10-291-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix compilation on FreeBSD 11.2 (use st_mtim.tv_nsec)
| * | | | | | | Fix compilation on FreeBSD 11.2 (use st_mtim.tv_nsec)Simon Arlott2018-10-251-1/+1
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ./src/disk_interface.cc: In member function 'virtual TimeStamp RealDiskInterface::Stat(const string&, std::__cxx11::string*) const': ./src/disk_interface.cc:216:51: error: 'struct stat' has no member named 'st_mtimensec'; did you mean 'st_mtim'? return (int64_t)st.st_mtime * 1000000000LL + st.st_mtimensec; ^~~~~~~~~~~~ st_mtim
* | | | | | | Merge pull request #1350 from korli/posix_spawn_errnoJan Niklas Hasse2018-10-291-27/+39
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | posix_spawn_* calls don't set errno, use the return value.
| * | | | | | | posix_spawn_* calls don't set errno, use the return value.Jerome Duval2017-11-011-27/+39
| | | | | | | |
* | | | | | | | Merge pull request #1430 from whereswaldon/patch-1Jan Niklas Hasse2018-10-291-1/+1
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Fix simple typo in HACKING.md
| * | | | | | | Fix simple typo in HACKING.mdChristopher Waldon2018-04-251-1/+1
| | |/ / / / / | |/| | | | |
* | | | | | | Merge pull request #1417 from stefanb2/topic-silence-gcc-fallthrough-warningsNico Weber2018-10-244-1/+24
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Silence GCC -Wimplicit-fallthrough warnings
| * | | | | | | Add NINJA_FALLTHROUGH macroStefan Becker2018-04-114-1/+24
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Borrow macro implementation from OpenSSL code. Add the macro after each fallthrough switch case to indicate our intention to the compiler. This silences GCC -Wimplicit-fallthrough warnings, which is implied by GCC 7.x -Wextra.
* | | | | | | Merge pull request #1467 from cdluminate/masterNico Weber2018-10-246-8/+8
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Misc typo fixes by https://github.com/codespell-project/codespell/
generated by cgit v0.12 at 2024-11-08 05:19:16 (GMT)