summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Maybe I mustn't update?github-actions-apt-upgradeJan Niklas Hasse2020-10-021-2/+1
|
* I hate apt and UbuntuJan Niklas Hasse2020-10-021-1/+1
|
* Maybe I need to specify?Jan Niklas Hasse2020-10-021-2/+1
|
* GitHub Actions: Upgrade packages before installing new onesJan Niklas Hasse2020-10-021-0/+1
|
* Merge pull request #1768 from jhasse/github-actions-lintJan Niklas Hasse2020-10-022-0/+63
|\ | | | | Run more CI tests on Ubuntu 20.04
| * clang-tidy: Drop readability-container-size-empty due to testsJan Niklas Hasse2020-09-191-2/+0
| |
| * GitHub Actions: Update to Ubuntu 20.04Jan Niklas Hasse2020-09-191-4/+4
| |
| * Merge branch 'master' into github-actions-lintJan Niklas Hasse2020-09-1922-140/+356
| |\ | |/ |/|
* | cmake: add BUILD_TESTING option (ON by default) (#1839)David Callu2020-09-141-39/+41
| | | | | | | | option provided by cmake's Module CTest enable_testing() is call by this Module
* | Merge pull request #1789 from neheb/ifJan Niklas Hasse2020-09-101-2/+1
|\ \ | | | | | | [clang-tidy] fix small false positive
| * | [clang-tidy] fix small false positiveRosen Penev2020-06-171-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | else if has to be on the same line it seems. Found with readability-misleading-indentation Signed-off-by: Rosen Penev <rosenp@gmail.com>
* | | Merge pull request #1786 from neheb/membJan Niklas Hasse2020-09-102-8/+3
|\ \ \ | | | | | | | | [clang-tidy] remove redundant member init
| * | | [clang-tidy] remove redundant member initRosen Penev2020-06-172-8/+3
| |/ / | | | | | | | | | | | | | | | Found with readability-redundant-member-init Signed-off-by: Rosen Penev <rosenp@gmail.com>
* | | Merge pull request #1836 from aharrison24/cmake-browse-mode-supportJan Niklas Hasse2020-08-282-3/+63
|\ \ \ | | | | | | | | CMake: Add support for "browse" mode
| * | | CMake: Add platform checks for browse mode supportAlastair Harrison2020-08-281-10/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Browse mode requires a number of POSIX features to be available. This commit adds configure-time checks that the 'unistd.h' header is available and that the `inline.sh` script executes successfully. If the checks pass then browse mode is enabled.
| * | | Improve error handling in inline.shAlastair Harrison2020-08-281-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the script would generate some output and return a zero error code, even if the calls to `od` or `sed` failed. This change ensures that: - If `od` or `sed` fail then the script will fail. - Output will only be written on success.
| * | | CMake: Add support for "browse" modeAlastair Harrison2020-08-271-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes ninja-build/ninja#1822, fixes ninja-build/ninja#1853 Adds support for `ninja -t browse` to CMake builds. The platform support logic is copied from configure.py, so Windows, Solaris and AIX are treated as 'unsupported' platforms. All other platforms are assumed to be supported. As discussed in #1853, when built via CMake the `ninja` executable looks for a binary called `python` in the current path, in order to launch the "browse" mode. The behaviour differs from that of the configure.py script, which looks for a python executable that has the *same name* as the python executable that invoked the configure script.
* | | | Merge pull request #1823 from serpent7776/patch-1Jan Niklas Hasse2020-08-271-1/+1
|\ \ \ \ | |/ / / |/| | | Use c++ instead of g++
| * | | Use c++ instead of g++Krzysztof Leśniak2020-08-011-1/+1
| | | | | | | | | | | | | | | | `configure.py` assumes `g++` is present on the system, which is not the case, e.g. for FreeBSD. `c++` should be used insted, which should be a link to system c++ compiler. This will be `g++` for linux, but `clang++` for FreeBSD.
* | | | Mark this 1.10.1.gitJan Niklas Hasse2020-08-181-1/+1
| | | |
* | | | Merge pull request #1780 from jhasse/delay-openJan Niklas Hasse2020-08-034-47/+89
|\ \ \ \ | |/ / / |/| | | Delay actually opening log files until the first write, fix #1724
| * | | Delay actually opening log files until the first write, fix #1724Jan Niklas Hasse2020-05-234-47/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling DepsLog/BuildLog::OpenForWrite will now only save the file path. The actual opening of the file (moved to OpenForWriteIfNeeded) happens right before the first write attempt. This is needed so that the files aren't held open when the generator runs (i.e. RebuildManifest) as it may call tools like recompact which won't be able to open the file on Windows. The disadvantage is that now the error reporting happens at a later time and will be reported as a failed write, not a failed open.
* | | | cmake: use PROJECT_{SOURCE,BINARY}_DIR instead of CMAKE_{SOURCE,BINARY}_DIRDavid Callu2020-07-171-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CMAKE_SOURCE_DIR refer to the full path to the top level of the current CMake source tree PROJECT_SOURCE_DIR refer to the source directory of the last call to the project() command made in the current directory scope or one of its parents when ninja is use as a subproject, the build fail because of this.
* | | | Fix typosDimitris Apostolou2020-07-073-5/+5
| | | |
* | | | Merge pull request #1804 from ninja-build/nico-patch-2Nico Weber2020-06-211-1/+6
|\ \ \ \ | | | | | | | | | | Include sys/select.h in subprocess-posix.cc
| * | | | Include sys/select.h in subprocess-posix.ccNico Weber2020-06-211-1/+6
| | |/ / | |/| | | | | | | | | | | | | | pselect() is in sys/select.h in "newer" (2001) versions of posix, so add an include for it. While here, only include poll.h if USE_PPOLL is defined.
* | | | Merge pull request #1803 from ninja-build/nico-patch-1Nico Weber2020-06-211-0/+2
|\ \ \ \ | |/ / / |/| | | Include unistd.h in disk_interface.cc
| * | | Include unistd.h in disk_interface.ccNico Weber2020-06-211-0/+2
|/ / / | | | | | | | | | stat() needs unistd.h in addition to sys/stat.h and sys/types.h per POSIX. At least one (hobby) OS does need unistd.h, so add an include for it.
* | | GitHub Actions: Switch to MSVC's subdirectoryJan Niklas Hasse2020-06-151-1/+1
| | |
* | | GitHub Actions: PowerShell requires \ instead of /Jan Niklas Hasse2020-06-151-1/+1
| | |
* | | GitHub Actions: Run ninja_test directly (Windows)Jan Niklas Hasse2020-06-151-1/+1
| | |
* | | [clang-tidy] check empty instead of size (#1784)Rosen Penev2020-06-044-4/+4
| | | | | | | | | | | | | | | Found with readability-container-size-empty Signed-off-by: Rosen Penev <rosenp@gmail.com>
* | | [clang-tidy] remove pointless c_str() (#1785)Rosen Penev2020-06-041-1/+1
| | | | | | | | | | | | | | | Found with readability-redundant-string-cstr Signed-off-by: Rosen Penev <rosenp@gmail.com>
* | | Add missing Apache license header, fix #1781Jan Niklas Hasse2020-05-251-0/+14
| | |
* | | bugfix: Process escaped colon in GCC depfiles. (#1774)zero91782020-05-203-24/+121
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Added ability to parse escaped colons in GCC Dep files enabling ninja to parse dep files of GCC 10 on Windows * Added generated depfile_parser.cc * Addressed formatting * Added extra tests with real world examples of paths produced by both GCC 10 and Clang and GCC pre 10. Adjusted one test so it doesn't fail * Adjusted regular expression to not match \: if the character following the : is either EOF or whitespace * Fixed typo in regex (should be 0x20 for space not 0xa) * Changed regular expression form using lookahead to instead matching a separate expression. This was needed as re2c pre version 1.17 is broken when using lookaheads. Also added tests for \: followed by whitespace * Addressed formatting * Forgot a missing std:: * Fixed formatting for spaces after , as well as respecting column width
| * Add more clang-tidy checksJan Niklas Hasse2020-06-041-0/+4
| |
| * GitHub Actions: Build and test both Debug and Release configsJan Niklas Hasse2020-05-181-18/+37
| |
| * GitHub Actions: Also run on Ubuntu 18.04Jan Niklas Hasse2020-05-182-0/+42
|/ | | | Builds with both GCC and Clang and runs clang-tidy, too.
* MinGW now has _mktemp_s, use overload with size parameterJan Niklas Hasse2020-05-181-11/+2
| | | | | | | | | MSVC also used the custom implementation, since the function isn't a macro and therefore #ifndef _mktemp_s didn't work as intended.
* GitHub Actions: Run ninja_test directlyJan Niklas Hasse2020-05-141-1/+1
|
* Merge pull request #1769 from scivision/pytest_winJan Niklas Hasse2020-04-301-0/+1
|\ | | | | test: skip non-Windows tests if on Windows
| * test: skip non-Windows tests if on WindowsMichael Hirsch, Ph.D2020-04-271-0/+1
|/
* Add output test for 'no work to do.'Jan Niklas Hasse2020-04-241-0/+3
|
* Merge pull request #1765 from past-due/macos_deployment_target_1Jan Niklas Hasse2020-04-181-0/+2
|\ | | | | macOS workflow: specify the minimum version of macOS supported
| * Add MACOSX_DEPLOYMENT_TARGET to macos workflowpast-due2020-04-171-0/+2
|/
* Merge pull request #1763 from nomadbyte/masterJan Niklas Hasse2020-04-151-1/+1
|\ | | | | Null terminate the out2 string passed as filter2 to log.Restat
| * Null terminate the out2 string passed as filter2 to log.Restatyydevelop2020-04-141-1/+1
|/
* Turn CONTRIBUTING.md into a linkJan Niklas Hasse2020-04-061-1/+2
|
* Merge pull request #1748 from scivision/optsJan Niklas Hasse2020-04-061-1/+12
|\ | | | | enable Intel, IBM XL, and PGI compilers without header/source changes
| * only apply optional flags when compiler worksMichael Hirsch, Ph.D2020-04-061-1/+12
|/