| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | |
|
| |\ \ \
| | | | |
| | | | | |
Fixed cygwin compatibility (issue #806)
|
| | | | |
| | | | |
| | | | |
| | | | | |
Fixed platform specific issues causing cygwin build to fail.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
All other counting variables are called foo_count, not num_foos.
No behavior change.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
No test since there's still no good way to test code in ninja.cc. Going
forward, either move NinjaMain to its own file with tests, or create a
system that makes it possible to run integration tests on the ninja
binary. Instead, add a comment that explains why the restat
optimization isn't done.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The original overprinting code, added in 7b3d8c8e, used printf for printing
the status. printf needs one column for the cursor, so the status message
could only take up `width - 1` columns. fc554c22 changed Windows from printf
to WriteConsoleOutput which doesn't move the cursor, so keeping one column
empty is no longer needed. So stop doing that.
Also remove a duplicate call to GetConsoleScreenBufferInfo.
|
| | | | |
| | | | |
| | | | |
| | | | | |
No behavior change. Based on a patch from gmisocpp@gmail.com!
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
In cmd.exe, hitting the "Pause" key or Ctrl-S will pause
programs until a key is pressed. This is apparently
implemented when stdout is writing to, so use printf
instead of Console functions to reset the cursor to
the start of the line. Also happens to simplify the code.
(This already worked in -v mode since that already
prints using printf.)
Based on a patch from gmisocpp@gmail.com!
|
|\ \ \ \ \
| |/ / / / |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
No behavior change on most platforms. On solaris, -t browse was compiled in
in ninja.cc but browse.cc wasn't compiled in, which probably means that building
on Solaris didn't work. It might be better now.
This also makes browse.cc automatically not included in bootstrap builds;
previously this was done manually through the NINJA_BOOTSTRAP check.
|
| |\ \ \ \
| | | | | |
| | | | | | |
whitespace/comment/wrap fixes, no intended functionality change
|
| | | | | | |
|
| |/ / / / |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | | |
| | | | | | |
Throttle the number of pending commands by the parallelism configuration
|
| | |/ / / |
|
| |\ \ \ \
| | | | | |
| | | | | | |
Suppress warnings from building with MinGW
|
| | |/ / / |
|
| | | | | |
|
| | | | | |
|
| |/ / /
| | | |
| | | | |
Include POSIX termios.h
|
| | | |
| | | |
| | | |
| | | | |
Now tests don't print anything. Non-test behavior is unchanged.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Turns out gtest was pulling in sys/stat.h, and we were
using stat() through that in tests. This doesn't work
with old MSVCs, so we should probably replace that with
RealDiskInterface in a follow-up.
|
| | | | |
|
| |\ \ \
| | | | |
| | | | | |
Use a small, standalone testing framework for ninja instead of googletest.
|
| | | | |
| | | | |
| | | | |
| | | | | |
Let me know if you use these!
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Ninja currently uses googletest for testing. That makes building
ninja_test somewhat annoying since it requires that one passes
--with-gtest PATH to configure. It turns out just implementing the bits
of googletest that ninja uses needs about the same amount of code than
making the --with-gtest flag in configure.py work and making googletest
print test results in a way we want (!)
In addition to making configuration simpler, this also makes compiling
tests much faster: On my system, touching src/build_test.cc (the slowest
file to build in ninja) and rebuilding ninja_tests is twice as fast than
without this patch. Building all is noticeably faster too: 5.6s with
this patch, 9.1s without this patch (38% faster).
The most noticeable things missing: EXPECT_* and ASSERT_* don't support
streaming notes to them with operator<<, and for failing tests the lhs
and rhs are not printed. That's so that this header does not have to
include sstream, which slows down building ninja_test almost 20%.
If this turns out to be annoying, we can maybe add it.
|
| |/ / / |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
As now behavior is similar on on platforms, this can probably be removed
|
| | | |
| | | |
| | | |
| | | | |
This makes this implementation more consisten with POSIX load avarage.
|