| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
This allows paths with "[", "]" and "%" to appear in depfiles.
Previously, only "[" would be handled properly.
Fixes #1227.
|
|\
| |
| | |
Allow paths with international characters in depfiles
|
| | |
|
| |
| |
| |
| | |
The correct order is ASSERT_EQ(expected, actual).
|
| |
| |
| |
| | |
This is a prerequisite for fixing #417.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
| |
|
|
|
|
| |
Fixes #752.
|
|
|
|
|
|
|
| |
See http://llvm.org/PR15642. I checked that gcc does produce depfiles
containing "$$" for files with "$" signs in their name (and as of
r178540, so does clang). I also checked that .d files that escape dollar
signs with "\$" are not read correctly by make.
|
|
|
|
|
| |
See funny paths in
https://github.com/google/libcxx/tree/master/test/iterators/stream.iterators/istreambuf.iterator/
|
|
|
|
|
| |
The SpecialChars test covers a bunch of different special characters,
including tilde.
|
| |
|
|
|
|
| |
Fixes issue #327.
|
| |
|
|\
| |
| | |
fix warning: "comparison between signed and unsigned integer expressions"
|
| | |
|
|/ |
|
|
|
|
|
|
|
|
|
|
| |
Older versions of GCC would produce broken depfiles when -MT or -MQ is used
gcc43 -MT foo.o -MMD -MF foo.o.d -o foo.o -c foo.c
will result in the following depfile
foo.o foo.o: <dependencies>
Parse multiple outputs unifying duplicates and correctly report errors if
they are different.
|
| |
|
| |
|
|
|
|
| |
Add some comments as well.
|
|
|
|
|
|
|
|
| |
The logic was wrong if the input looked like
foo : bar baz
with a space before the colon.
Test from Frances <frances.buontempo@gmail.com>.
|
|
|