From 133be3dc1f1eb3f3c6252b950e89895469f1193c Mon Sep 17 00:00:00 2001 From: Brian Silverman Date: Tue, 8 Sep 2015 18:56:58 -0400 Subject: Use TEST_TMPDIR for writing temporary files if it's set. Bazel expects all tests to do this. --- src/google/protobuf/testing/googletest.cc | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'src') diff --git a/src/google/protobuf/testing/googletest.cc b/src/google/protobuf/testing/googletest.cc index b8bd790b..2b9cddef 100644 --- a/src/google/protobuf/testing/googletest.cc +++ b/src/google/protobuf/testing/googletest.cc @@ -94,6 +94,13 @@ string TestSourceDir() { namespace { string GetTemporaryDirectoryName() { + // Tests run under Bazel "should not" use /tmp. Bazel sets this environment + // variable for tests to use instead. + char *from_environment = getenv("TEST_TMPDIR"); + if (from_environment != NULL && from_environment[0] != '\0') { + return string(from_environment) + "/protobuf_tmpdir"; + } + // tmpnam() is generally not considered safe but we're only using it for // testing. We cannot use tmpfile() or mkstemp() since we're creating a // directory. -- cgit v1.2.3