diff options
| author |  Laszlo Csomor <laszlocsomor@google.com> | 2017-01-11 13:45:21 +0000 |
|---|---|---|
| committer |  Marcel Hlopko <hlopko@google.com> | 2017-01-11 16:02:58 +0000 |
| commit | 8e0f29c7e0440935c780cbb752b673cc0a191833 (patch) | |
| tree | 885a8797dc37861cd3a80c9efd60c819680b4540 /src/test/cpp/util/file_test.cc | |
| parent | 45bddab11593ca1536e4d3402fbb631e335e9f3a (diff) | |
Bazel client, Windows: support /dev/null
Support passing /dev/null and NUL as flag values
for flags like --bazelrc.
In particular, special-case these paths in
methods like blaze_util::ReadFile,
blaze_util::IsDirectory, etc.
Fixes https://github.com/bazelbuild/bazel/issues/2354
RELNOTES[NEW]: Windows: "/dev/null" is now a supported path, e.g. --bazelrc=/dev/null now works
--
PiperOrigin-RevId: 144195994
MOS_MIGRATED_REVID=144195994
Diffstat (limited to 'src/test/cpp/util/file_test.cc')
| -rw-r--r-- | src/test/cpp/util/file_test.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/test/cpp/util/file_test.cc b/src/test/cpp/util/file_test.cc index aa654cc07f..52d7f8d5f8 100644 --- a/src/test/cpp/util/file_test.cc +++ b/src/test/cpp/util/file_test.cc @@ -70,6 +70,9 @@ TEST(FileTest, TestReadFile) { ASSERT_TRUE(ReadFile(filename, &actual, 5)); ASSERT_EQ(std::string("hello"), actual); + + ASSERT_TRUE(ReadFile("/dev/null", &actual, 42)); + ASSERT_EQ(std::string(""), actual); } } // namespace blaze_util |