From 7671eeeb25eec0348bf66a55f364fee1ea68ac1d Mon Sep 17 00:00:00 2001 From: Jordan Dominion Date: Sat, 14 Oct 2023 22:24:22 -0400 Subject: [PATCH] Simplify OD install paths --- src/Tgstation.Server.Host/Components/Engine/EngineManager.cs | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/Tgstation.Server.Host/Components/Engine/EngineManager.cs b/src/Tgstation.Server.Host/Components/Engine/EngineManager.cs index 7297d8e07e..0b7367076d 100644 --- a/src/Tgstation.Server.Host/Components/Engine/EngineManager.cs +++ b/src/Tgstation.Server.Host/Components/Engine/EngineManager.cs @@ -501,10 +501,7 @@ namespace Tgstation.Server.Host.Components.Engine /// A representing the running operation. async ValueTask InstallVersionFiles(JobProgressReporter progressReporter, ByondVersion version, Stream customVersionStream, CancellationToken cancellationToken) { - var installFullPath = ioManager.ResolvePath( - version.Engine == EngineType.Byond - ? version.ToString() - : Guid.NewGuid().ToString()); // too much BS with OD i swear, we can't use the provided committish because it may expand later + var installFullPath = ioManager.ResolvePath(version.ToString()); async ValueTask DirectoryCleanup() { await ioManager.DeleteDirectory(installFullPath, cancellationToken);