summaryrefslogtreecommitdiffstats
path: root/src/msvc_helper_test.cc
Commit message (Collapse)AuthorAgeFilesLines
* Make deps=msvc experimentally available on non-Windows.Nico Weber2016-04-061-93/+1
| | | | | | | | | This makes it possible to run most of the clparser tests on non-Windows, and is potentially useful for cross-compiling on non-Windows hosts. Also, the manual didn't document this as Windows-only previously. If you use this on non-Windows, please let me know, else I might undo this change again in the future.
* propagate include normalization failure to caller insteadScott Graham2015-06-181-10/+15
|
* Use a small, standalone testing framework instead of googletest.Nico Weber2014-09-181-2/+0
| | | | | | | | | | | | | | | | | | | | | 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.
* CLParser shouldn't read stderrScott Graham2014-04-141-0/+7
|
* add deps_prefix for localized /showIncludes' output parsingPeter Kümmel2013-10-181-10/+19
|
* Stop `-t msvc -o` from lowercasing paths from /showIncludes output.Nico Weber2013-05-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | /showIncludes prints absolute paths. If a source file says `#include <WiNdOwS.h>`, /showIncludes will use that spelling in its output for the basename and use the on-disk cache for the directory names in the rest for its output. This makes the .d files created by `-t msvc -o` consistent with the .d files written by gcc and clang. Before this change, `-t msvc -o` would convert this output to lower case. This is a problem if a build step produces a header file with mixed case, such as "RuntimeFeatures.h". Due to the lowercasing, the .d file would contain "runtimefeatures.h", while the build step will create "RuntimeFeatures.h". Due to the case difference, ninja would not realize that regeneration of the .h file would require a rebuild of all source files having the header in the .d file. (On the next build, ninja would rebuild them since stat()ing is not case-sensitive on Windows.) One possible fix for this is to make sure that generators always write generated header files in lower case too, but on Mac gcc doesn't do lower-casing and .d files end up with the case-as-written, so generators would have to be different on Mac and Windows, which is undesirable. If case-insensitve path comparisons are useful, they should be done somewhere else (e.g. in CanonicalizePath()) where they affect both paths read from .d files and paths read from .ninja files. This should then be controlled by a top-level variable. This patch changes behavior, but it only has an effect on generated header files, which aren't common, and it only affects -t msvc, which is still marked as experimental. (cmake doesn't use it yet.) (If a file has both `#include <windows.h>` and `<Windows.h>`, this will now take 2 stat() calls instead of just one, but that should have a negligible cost.)
* factor MSVC parsing out of CLWrapper into CLParserEvan Martin2013-04-081-65/+58
|
* fix spaces in headers for -t msvcScott Graham2012-09-201-0/+11
|
* don't emit duplicate headers for msvc helperScott Graham2012-09-171-2/+26
|
* pass env block to cl helperEvan Martin2012-08-151-0/+9
|
* msvc helper: drop system includesEvan Martin2012-08-121-0/+14
| | | | | Drop any #includes that look like they're referencing system headers. This reduces the dependency information considerably.
* msvc helper: attempt to filter out when it prints the input filenameEvan Martin2012-08-121-0/+18
| | | | This is a heuristic but it appears to work for the Chrome build.
* add subprocess-spawning to msvc_helperEvan Martin2012-08-121-0/+10
| | | | | | | | | | Rather than using subprocess.h, reimplement the subprocess code. This allows: 1) using anonymous (instead of named) pipes 2) not using all the completion port craziness 3) printing the output as it happens 4) further variation, like adjusting the environment (in a forthcoming change) without affecting the main subprocess code
* add a module for working with MSVC (cl.exe) behaviorEvan Martin2012-08-121-0/+32
This will be needed for performant builds on Windows.