From 9f49d34c17ee7d55273bfe13cf7c6c3e014b36e9 Mon Sep 17 00:00:00 2001 From: Kiran Challa Date: Wed, 14 Jun 2017 19:52:54 -0700 Subject: [PATCH] Revert "Workaround: On xplat machines, provided executable permission to published standalone application executable" This reverts commit 51f77ea7f3b4a704691d0def9dd746fe0e32d5dc. --- .../Deployers/SelfHostDeployer.cs | 60 ------------------- 1 file changed, 60 deletions(-) diff --git a/src/Microsoft.AspNetCore.Server.IntegrationTesting/Deployers/SelfHostDeployer.cs b/src/Microsoft.AspNetCore.Server.IntegrationTesting/Deployers/SelfHostDeployer.cs index 83800ba225..2ab172b830 100644 --- a/src/Microsoft.AspNetCore.Server.IntegrationTesting/Deployers/SelfHostDeployer.cs +++ b/src/Microsoft.AspNetCore.Server.IntegrationTesting/Deployers/SelfHostDeployer.cs @@ -39,9 +39,6 @@ namespace Microsoft.AspNetCore.Server.IntegrationTesting if (DeploymentParameters.PublishApplicationBeforeDeployment) { DotnetPublish(); - - // Temporary workaround for https://github.com/dotnet/cli/issues/6286 - ProvideExecutePermission(); } var hintUrl = TestUriHelper.BuildTestUri(DeploymentParameters.ApplicationBaseUriHint); @@ -199,62 +196,5 @@ namespace Microsoft.AspNetCore.Server.IntegrationTesting StopTimer(); } } - - // Temporary workaround for https://github.com/dotnet/cli/issues/6286 - private void ProvideExecutePermission() - { - if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows) - && DeploymentParameters.RuntimeFlavor == RuntimeFlavor.CoreClr - && DeploymentParameters.ApplicationType == ApplicationType.Standalone) - { - var executablePath = Path.Combine( - DeploymentParameters.PublishedApplicationRootPath, - DeploymentParameters.ApplicationName); - - var chmodProcessStartInfo = new ProcessStartInfo - { - FileName = "chmod", - Arguments = $"755 {executablePath}", - UseShellExecute = false, - CreateNoWindow = true, - RedirectStandardOutput = true, - RedirectStandardError = true - }; - - var chmodProcess = new Process() { StartInfo = chmodProcessStartInfo }; - try - { - chmodProcess.StartAndCaptureOutAndErrToLogger("chmod", Logger); - } - catch (Exception ex) - { - Logger.LogError( - "Error occurred while starting the process '{processName}'. Exception: {exception}", - chmodProcessStartInfo.FileName, - ex.ToString()); - } - - var hasExited = chmodProcess.WaitForExit(3 * 1000); - - if (hasExited) - { - if (chmodProcess.ExitCode != 0) - { - Logger.LogError( - "Process {processName} with id {pid} exited with code {exitCode} or failed to start.", - chmodProcessStartInfo.FileName, - HostProcess.Id, - HostProcess.ExitCode); - } - } - else - { - Logger.LogError( - "Process {processName} with id {pid} did not exit in the given amount of time.", - chmodProcessStartInfo.FileName, - HostProcess.Id); - } - } - } } }