Fix #195: run the command line application on the same thread used to start generic host

natemcmaster · natemcmaster · commit ba2d69d8bb5a · 2018-12-18T22:55:06.000-08:00
diff --git a/src/Hosting.CommandLine/Internal/CommandLineService.cs b/src/Hosting.CommandLine/Internal/CommandLineService.cs
@@ -47,7 +47,9 @@ public CommandLineService(ILogger<CommandLineService<T>> logger, CommandLineStat
         public Task<int> RunAsync(CancellationToken cancellationToken)
         {
             _logger.LogDebug("Running");
-            return Task.Run(() => _state.ExitCode = _application.Execute(_state.Arguments), cancellationToken);
+            // TODO support cancellation tokens. See #111
+            _state.ExitCode = _application.Execute(_state.Arguments);
+            return Task.FromResult(_state.ExitCode);
         }
 
         public void Dispose()

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,9 @@ public CommandLineService(ILogger<CommandLineService<T>> logger, CommandLineStat`
`47`	`47`	`public Task<int> RunAsync(CancellationToken cancellationToken)`
`48`	`48`	`{`
`49`	`49`	`_logger.LogDebug("Running");`
`50`		`- return Task.Run(() => _state.ExitCode = _application.Execute(_state.Arguments), cancellationToken);`
	`50`	`+ // TODO support cancellation tokens. See #111`
	`51`	`+ _state.ExitCode = _application.Execute(_state.Arguments);`
	`52`	`+ return Task.FromResult(_state.ExitCode);`
`51`	`53`	`}`
`52`	`54`
`53`	`55`	`public void Dispose()`