aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Ridgers <[email protected]>2021-11-03 09:25:54 +0100
committerMartin Ridgers <[email protected]>2021-11-03 09:25:54 +0100
commit30d1beba4d8ab3db48fd7bf8d6cc63d565c20e9c (patch)
treeea480c8fe49dfffba3e4a4b3883a3b10ede5d99f
parentWrapped /apply/ and /exec/ services in a ZEN_WITH_COMPUTE_SERVICES define (diff)
downloadzen-30d1beba4d8ab3db48fd7bf8d6cc63d565c20e9c.tar.xz
zen-30d1beba4d8ab3db48fd7bf8d6cc63d565c20e9c.zip
Removed if-def ZEN_PLATFORM_WINDOWS as its covered by WITH_COMPUTE now
-rw-r--r--zenserver/compute/apply.cpp22
-rw-r--r--zenserver/testing/launch.cpp30
-rw-r--r--zenserver/zenserver.cpp1
3 files changed, 15 insertions, 38 deletions
diff --git a/zenserver/compute/apply.cpp b/zenserver/compute/apply.cpp
index e1ab7c984..4a92b9968 100644
--- a/zenserver/compute/apply.cpp
+++ b/zenserver/compute/apply.cpp
@@ -17,17 +17,15 @@
#include <zenstore/cas.h>
#include <zenstore/cidstore.h>
-#if ZEN_PLATFORM_WINDOWS
-# include <zencore/windows.h>
+#include <zencore/windows.h>
ZEN_THIRD_PARTY_INCLUDES_START
-# include <AccCtrl.h>
-# include <AclAPI.h>
-# include <sddl.h>
-# include <UserEnv.h>
-# pragma comment(lib, "UserEnv.lib")
-# include <atlbase.h>
+#include <AccCtrl.h>
+#include <AclAPI.h>
+#include <sddl.h>
+#include <UserEnv.h>
+#pragma comment(lib, "UserEnv.lib")
+#include <atlbase.h>
ZEN_THIRD_PARTY_INCLUDES_END
-#endif
#include <filesystem>
#include <span>
@@ -36,7 +34,6 @@ using namespace std::literals;
namespace zen {
-#if ZEN_PLATFORM_WINDOWS
struct BasicFunctionJob
{
public:
@@ -136,11 +133,9 @@ BasicFunctionJob::ExitCode()
return gsl::narrow_cast<int>(Ec);
}
-#endif
////////////////////////////////////////////////////////////////////////////////
-#if ZEN_PLATFORM_WINDOWS
struct SandboxedFunctionJob
{
SandboxedFunctionJob() = default;
@@ -330,7 +325,6 @@ SandboxedFunctionJob::SpawnJob(std::filesystem::path ExePath)
return true;
}
-#endif
////////////////////////////////////////////////////////////////////////////////
@@ -772,7 +766,6 @@ HttpFunctionService::ExecAction(const WorkerDesc& Worker, CbObject Action)
std::string_view ExecPath = Desc["path"].AsString();
std::filesystem::path ExePath = SandboxPath / ExecPath;
-#if ZEN_PLATFORM_WINDOWS
WideStringBuilder<512> CommandLine;
CommandLine.Append(L'"');
CommandLine.Append(ExePath.c_str());
@@ -819,7 +812,6 @@ HttpFunctionService::ExecAction(const WorkerDesc& Worker, CbObject Action)
GetExitCodeProcess(ProcessInformation.hProcess, &ExitCode);
// Gather outputs
-#endif // ZEN_PLATFORM_WINDOWS
FileContents OutputData = zen::ReadFile(SandboxPath / "build.output");
diff --git a/zenserver/testing/launch.cpp b/zenserver/testing/launch.cpp
index 8e58b481e..013aa28b8 100644
--- a/zenserver/testing/launch.cpp
+++ b/zenserver/testing/launch.cpp
@@ -11,20 +11,17 @@
#include <zencore/iobuffer.h>
#include <zencore/iohash.h>
#include <zencore/logging.h>
+#include <zencore/windows.h>
#include <zenstore/cas.h>
-#if ZEN_PLATFORM_WINDOWS
-# include <zencore/windows.h>
-
ZEN_THIRD_PARTY_INCLUDES_START
-# include <AccCtrl.h>
-# include <AclAPI.h>
-# include <atlbase.h>
-# include <sddl.h>
-# include <UserEnv.h>
+#include <AccCtrl.h>
+#include <AclAPI.h>
+#include <atlbase.h>
+#include <sddl.h>
+#include <UserEnv.h>
ZEN_THIRD_PARTY_INCLUDES_END
-# pragma comment(lib, "UserEnv.lib")
-#endif
+#pragma comment(lib, "UserEnv.lib")
#include <filesystem>
#include <span>
@@ -33,7 +30,6 @@ using namespace std::literals;
namespace zen {
-#if ZEN_PLATFORM_WINDOWS
struct BasicJob
{
public:
@@ -133,19 +129,9 @@ BasicJob::ExitCode()
return gsl::narrow_cast<int>(Ec);
}
-#else
- struct BasicJob
- {
- void SetWorkingDirectory(...);
- void SpawnJob(...);
- void Wait();
- int32_t ExitCode();
- };
-#endif // ZEN_PLATFORM_WINDOWS
////////////////////////////////////////////////////////////////////////////////
-#if ZEN_PLATFORM_WINDOWS
struct SandboxedJob
{
SandboxedJob() = default;
@@ -335,8 +321,6 @@ SandboxedJob::SpawnJob(std::filesystem::path ExePath)
return true;
}
-#else
-#endif // ZEN_PLATFORM_WINDOWS
////////////////////////////////////////////////////////////////////////////////
diff --git a/zenserver/zenserver.cpp b/zenserver/zenserver.cpp
index 06f3911ec..d8f213e56 100644
--- a/zenserver/zenserver.cpp
+++ b/zenserver/zenserver.cpp
@@ -743,6 +743,7 @@ ZenServer::InitializeStructuredCache(ZenServiceConfig& ServiceConfig)
} // namespace zen
#if ZEN_PLATFORM_WINDOWS
+
class ZenWindowsService : public WindowsService
{
public: