I\'m trying to execute a command line program with parameters from C#. I would have imagined that standing this up and making this happen would be trivial in C# but its prov
I realized I may have left out some detail that some people may need to solve this in the future.
Here are the values of the method parameters at run time. I had some confusion as to what the object ProcessStartInfo and Process needed to be stood up correctly I think others may as well.
exeDir = "C:\folder1\folder2\bin\keytool.exe"
args = "-delete -noprompt -alias server.us.goodstuff.world -storepass changeit -keystore keystore.jks"
public bool ExecuteCommand(string exeDir, string args)
{
try
{
ProcessStartInfo procStartInfo = new ProcessStartInfo();
procStartInfo.FileName = exeDir;
procStartInfo.Arguments = args;
procStartInfo.RedirectStandardOutput = true;
procStartInfo.UseShellExecute = false;
procStartInfo.CreateNoWindow = true;
using (Process process = new Process())
{
process.StartInfo = procStartInfo;
process.Start();
process.WaitForExit();
string result = process.StandardOutput.ReadToEnd();
Console.WriteLine(result);
}
return true;
}
catch (Exception ex)
{
Console.WriteLine("*** Error occured executing the following commands.");
Console.WriteLine(exeDir);
Console.WriteLine(args);
Console.WriteLine(ex.Message);
return false;
}
Between Dmitry's assistance and the following resource,
http://www.codeproject.com/Articles/25983/How-to-Execute-a-Command-in-C
I was able to cobble this together. Thank you!
Wait for the process to end (let it do its work):
ProcessStartInfo procStartInfo = new ProcessStartInfo("cmd", "/c " + command);
procStartInfo.RedirectStandardOutput = true;
procStartInfo.UseShellExecute = false;
procStartInfo.CreateNoWindow = true;
// wrap IDisposable into using (in order to release hProcess)
using(Process process = new Process()) {
process.StartInfo = procStartInfo;
process.Start();
// Add this: wait until process does its work
process.WaitForExit();
// and only then read the result
string result = process.StandardOutput.ReadToEnd();
Console.WriteLine(result);
}
When it comes to executing CLI processes from C#, it may seem like a simple task, but there are quite a few pitfalls that you might not even notice until much later. For example, both of the currently given answers will not work if the child process writes enough data to stdout, as explained here.
I wrote a library that simplifies working with CLIs by abstracting the Process
interaction entirely, solving the whole task by executing one method - CliWrap.
Your code would then look like this:
var cmd = Cli.Wrap("cmd")
.WithArguments(a => a.Add("/c").Add(command));
var result = await cmd.ExecuteBufferedAsync();
var stdOut = result.StandardOutput;