summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNat Goodspeed <nat@lindenlab.com>2023-07-08 09:04:33 -0400
committerNat Goodspeed <nat@lindenlab.com>2023-07-08 09:04:33 -0400
commitf37d2c307617302f2ed5dfead7e280da54a7d3e4 (patch)
tree6353b2a5cf188427aaa0e03136a48ccf9b599d01
parent908fb3fed6b858da4dc2b1c840b849e30ade2046 (diff)
SL-18837: Don't use LLDir, use NamedTempFile::temp_path.
Remove llcommon circular dependency on llfilesystem, which doesn't work for this case anyway.
-rw-r--r--indra/llcommon/CMakeLists.txt2
-rw-r--r--indra/llcommon/tests/llprocess_test.cpp5
2 files changed, 2 insertions, 5 deletions
diff --git a/indra/llcommon/CMakeLists.txt b/indra/llcommon/CMakeLists.txt
index 7412514e6b..54020a4231 100644
--- a/indra/llcommon/CMakeLists.txt
+++ b/indra/llcommon/CMakeLists.txt
@@ -308,7 +308,7 @@ if (LL_TESTS)
LL_ADD_INTEGRATION_TEST(llleap "" "${test_libs}")
LL_ADD_INTEGRATION_TEST(llmainthreadtask "" "${test_libs}")
LL_ADD_INTEGRATION_TEST(llpounceable "" "${test_libs}")
- LL_ADD_INTEGRATION_TEST(llprocess "" "${test_libs};llfilesystem")
+ LL_ADD_INTEGRATION_TEST(llprocess "" "${test_libs}")
LL_ADD_INTEGRATION_TEST(llprocessor "" "${test_libs}")
LL_ADD_INTEGRATION_TEST(llprocinfo "" "${test_libs}")
LL_ADD_INTEGRATION_TEST(llrand "" "${test_libs}")
diff --git a/indra/llcommon/tests/llprocess_test.cpp b/indra/llcommon/tests/llprocess_test.cpp
index 827837d62a..6fcc6fd8aa 100644
--- a/indra/llcommon/tests/llprocess_test.cpp
+++ b/indra/llcommon/tests/llprocess_test.cpp
@@ -30,7 +30,6 @@
#include "../test/namedtempfile.h"
#include "../test/catch_and_store_what_in.h"
#include "stringize.h"
-#include "../llfilesystem/lldir.h"
#include "llsdutil.h"
#include "llevents.h"
#include "llstring.h"
@@ -152,11 +151,9 @@ struct PythonProcessLauncher
/// Launch Python script; verify that it launched
void launch()
{
- std::string logpath{ gDirUtilp->getExpandedFilename(LL_PATH_LOGS, "apr.log") };
+ std::string logpath{ NamedTempFile::temp_path("apr", ".log").string() };
#if LL_WINDOWS
_putenv_s("APR_LOG", logpath.c_str());
-#else
- setenv("APR_LOG", logpath.c_str(), 1);
#endif
try
{