Remove timeout on WixNative*.exe to avoid judging on build time.
Bob Arnson committed
May 7, 2020 at 21:43 UTC
45a46d9175aa29c526e22bf6193837543610bb37
1 file changed
+6
-15
src/WixToolset.Core.Native/WixNativeExe.cs
+6
-15
@@ -11,7 +11,6 @@ namespace WixToolset.Core.Native
11
12
internal class WixNativeExe
13
{
14
- private const int TenMinutesInMilliseconds = 600000;
14
private static string PathToWixNativeExe;
15
16
private readonly string commandLine;
@@ -63,22 +62,14 @@ namespace WixToolset.Core.Native
62
process.StandardInput.WriteLine();
63
}
64
66
- if (process.WaitForExit(TenMinutesInMilliseconds))
67
- {
68
- // If the process successfully exits documentation says we need to wait again
69
- // without a timeout to ensure that all of the redirected output is captured.
70
- //
71
- process.WaitForExit();
65
+ // If the process successfully exits documentation says we need to wait again
66
+ // without a timeout to ensure that all of the redirected output is captured.
67
+ //
68
+ process.WaitForExit();
69
73
- if (process.ExitCode != 0)
74
- {
75
- throw new Win32Exception(process.ExitCode);
76
- }
77
- }
78
- else
70
+ if (process.ExitCode != 0)
71
{
80
- process.Kill();
81
- throw new Win32Exception(1460/*ERROR_TIMEOUT*/);
72
+ throw new Win32Exception(process.ExitCode);
73
}
74
}
75