C#: Batch Datei starten

[AFU]

Hallo,

Ich habe die Challenge, dass es ich es nicht zusammenbringe, eine Batch-Datei über die CMD.exe zu starten.


Hier mal die Definition, um "Commands" auszuführen. Dies klappt bei grunsätzlich, außer zum Installieren von Batch-Files. Für alle unten angeführten Varianten erhalte ich meistens von der CMD die Rückmeldung: "The System cannot find the specified file", obwohl der "/c " Paraemter verwendet wird. toInstall beeinhaltet den Zielpfad zb: C:\installation. Die Batch-Datei findet sich dann unter C:\installation\test\install.bat.
Gehe ich manuell ins Verzeichnis und führe die Batch-Datei aus, klappt es einwandrei.

public void ExecuteCommandSync(object command)
        {
            try
            {
                // create the ProcessStartInfo using "cmd" as the program to be run,
                // and "/c " as the parameters.
                // Incidentally, /c tells cmd that we want it to execute the command that follows,
                // and then exit.
                System.Diagnostics.ProcessStartInfo procStartInfo =
                    new System.Diagnostics.ProcessStartInfo("cmd", "/c " + command);

                // The following commands are needed to redirect the standard output.
                // This means that it will be redirected to the Process.StandardOutput StreamReader.
                procStartInfo.RedirectStandardOutput = true;
                procStartInfo.UseShellExecute = false;
                // Do not create the black window.
                procStartInfo.CreateNoWindow = true;
                // Now we create a process, assign its ProcessStartInfo and start it
                System.Diagnostics.Process proc = new System.Diagnostics.Process();
                proc.StartInfo = procStartInfo;
                proc.Start();
                //                proc.WaitForExit();
                // Get the output into a string
                string result = proc.StandardOutput.ReadToEnd();
                // Display the command output.
                Console.WriteLine(result);
                textBox1.Text += result;
            }
            catch (Exception objException)
            {
                // Log the exception
            }
        }

Hier dann die Versuche, es über diese Funktion aufzurufen

string fulltest = toInstall + "\\server\\test\\install.bat
string command = "start cmd.exe /c cd /d " + fulltest + " &&  install.bat";
ExecuteCommandSync(command);

Weitere Versuche unabhängig von der public void Methode oben.

Process P = new Process();
P.StartInfo.FileName = "cmd.exe"; at";
P.StartInfo.Arguments = @"/c " + fulltest;
P.Start();

string strCmdText;
strCmdText = "/C " + toInstall + "\\server\\test\\install.bat";
System.Diagnostics.Process.Start("CMD.exe", strCmdText);

System.Diagnostics.Process.Start("cmd.exe", @"/c toInstall + '\\server\\test\\install.bat'");
ProcessInfo = new ProcessStartInfo("cmd.exe", "/c " + toInstall + "\\server\\test\\install.bat");
System.Diagnostics.Process.Start(toInstall + "\\server\\test\\install.bat");

Ich brauche keine komplexe Methode/Funktion, ich möchte nur genau einmal die install.bat ausführen. Alternativ auch gerne über powershell. Auch das habe ich mit dem gleichen Ergebnis getestet.

[Nuebel]

string fulltest = toInstall + "\\server\\test\\install.bat
[...]
P.StartInfo.Arguments = @"/c " + fulltest;

Bezieht sich das @-Token in C# hier nur auf das "/c" oder auf den konkatenierten String "/c " + fulltest? Wenn Letzteres der Fall ist, werden die doppelten Backlsashes (\\) in fulltest wörtlich interpretiert. Lass das @ mal weg. Bzw. lass dir mal den String command ausgeben. Windows-Pfadangaben sind scheiße.

[AFU]

habe ich geprüft, passt.

[DMW007]

Im oberen Post steht etwas von lokalen C:\ Pfaden, die meisten Beispiele enthalten aber UNC-Pfade. Was davon hast du erfolglos getestet?

using System;

using System.Diagnostics;



namespace BatchStartTest {

    class Program {

        static void Main(string[] args) {

            string fileName = @"/c C:\test.bat";

            var procStartInfo = new ProcessStartInfo("cmd", fileName) {

                RedirectStandardOutput = true,

                UseShellExecute = false

            };

            procStartInfo.RedirectStandardOutput = true;

            procStartInfo.UseShellExecute = false;



            var proc = new Process {

                StartInfo = procStartInfo

            };

            proc.Start();



            string result = proc.StandardOutput.ReadToEnd();

            Console.WriteLine(result);

            Console.Read();

        }

    }

}

Wobei du nicht mal eine explizite cmd.exe Instanz brauchst, wenn es nur ein Batch-Skript ist. Dann reicht auch einfach

string fileName = @"C:\test.bat";

var procStartInfo = new ProcessStartInfo(fileName) {

    RedirectStandardOutput = true,

    UseShellExecute = false

};

Gibt in beiden Fällen das aus, was C:\test.bat nach stdout schreibt. Wie die Backslashes escaped werden spielt keine Rolle, man könnte genau so gut händisch "C:\\test.bat" escapen.

Wenn du eine gesonderte cmd Shell startest und darin Befehle ausführst wie bei dem Beispiel

// [...]
string command = "start cmd.exe /c cd /d " + fulltest + " &&  install.bat";

musst du deine Befehle quoten, sobald Leerzeichen verwendet werden. Die Shell kann nicht wissen, welche Argumente zu welcher Instanz gehören und interpretiert alles nach einem Leerzeichen als neues Argument bzw. eigenständigen Befehl:

C:\Users\Daniel>cmd /c cd C:\ && test.bat
Der Befehl "test.bat" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.

C:\Users\Daniel>cmd /c "cd C:\ && test.bat"

C:\>echo "test.bat called"
"test.bat called"

Der Rote Teil wird an die neu erstellte cmd Instanz übergeben. Sie wechselt das Arbeitsverzeichnis nach C:\ und wird anschließend beendet, weil der grüne Teil in der aufrufenden Shell startet. Diese wiederum befindet sich im Profilordner und findet test.bat daher nicht. Im zweiten Beispiel werden beide Befehl dem /c Argument übergeben und dadurch miteinander verkettet ausgeführt, wie erwartet.

Je nachdem was du da startest ist das vorherige Wechseln des Arbeitsverzeichnisses btw eine gute Idee. Wenn das was komplexeres ist, dass ggf. andere Dateien/Skripte ausführt und relative Pfade verwendet, werden die dann nicht zerschossen.

[AFU]

Hallo Daniel,

danke fürs Feedback. Es sind nur lokale Pfade. Der Pfad wird aber aus Variable + String zusammengesetzt.
Zb

string toinstall = "C:\installation":
string path = toInstall + "\\test\\bla";

Mir ist nicht ganz klar, wie ich den unentstehenden Part mit den "" für Variable + String anpassen soll:

string fileName = @"C:\test.bat";
var procStartInfo = new ProcessStartInfo(fileName) {
    RedirectStandardOutput = true,
    UseShellExecute = false
};

Kannst du hier aushelfen?

[DMW007]

ProcessStartInfo erhält im Konstruktor den Pfad der auszuführenden Datei. Ob der Pfad nun hartkodiert in einer Variable steht oder dynamisch zusammengebaut wird, spielt am Ende keine Rolle.

string installBasePath = "C:\\install-test";

string fullScriptPath = $"{installBasePath}\\test.bat";

var procStartInfo = new ProcessStartInfo(fullScriptPath) {

    RedirectStandardOutput = true,

    UseShellExecute = false

};

Wichtig ist, dass der Backslash entweder korrekt maskiert wird wie oben (\\ statt \) oder du den String wörtlich interpretieren lässt (vorangestelltes @):

string installBasePath = @"C:\install-test";

string fullScriptPath = $@"{installBasePath}\test.bat";

OT: Bei solchen Themen würde ich mir auch mal Gedanken machen, ob es da keine besseren Wege für gibt. Je nachdem was deine Skripte machen, kann das ggf. auch eine native Bibliothek. Im besten Falle spart das nicht nur Arbeit, sondern auch Nerven und ist flexibler.

[AFU]

Leider klappt es noch nicht, irgendwas mache ich falsch:

Meine Log-Methode gibt folgendes für die Variablen vor Ausführung der Install.bat retour:

Variable toInstall: C:\bla
Varialbe FullscriptPath: C:\bla\server\blabla\install.bat

Meiner Ansicht nach sieht meine toInstall-Variable so aus: C:\\bla

 log("Variable toInstall: " + toInstall + "\r\n");
                string fullScriptPath = $"{toInstall}\\server\\blabla\\install.bat";
                log("Varialbe FullscriptPath: " + fullScriptPath + "\r\n");
                var procStartInfo = new ProcessStartInfo(fullScriptPath)
                {
                    RedirectStandardOutput = true,
                    UseShellExecute = false
                };
                var proc = new Process
                {
                    StartInfo = procStartInfo
                };
                proc.Start();

Ich vermute mal, dass noch irgendwas mit den Backslashes nicht passt.

[DMW007]

Da das in meinem PoC oben funktioniert hat und dein Pfad syntaktisch korrekt aussieht, würde ich noch mal den Pfad prüfen. Kannst den ja einfach mal in die Suche kopieren und schauen ob er die Datei findet, oder ob sich ein Tippfehler eingeschlichen hat. Ansonsten schau mal in stdout was das Skript dorthin schreibt:

string stdout = proc.StandardOutput.ReadToEnd();

Console.WriteLine(stdout);

Wenn das Skript etwas installiert und somit eine Weile laufen muss, wäre es sicher auch sinnvoll, proc.WaitForExit(); nach dem Start aufzurufen. Falls notwendig mit entsprechend hohem Timeout als Parameter.

[AFU]

Stdout ist leer.
Ich habe das Problem gefunden. Ich habe dazu die CMD in einem anderen Verzeichnis gestartet und probiert, den Pfad direkt aufzurufen, einmal mit mit einem bzw. zwei Slashes. Siehe Screenshot was dabei rauskommt.
Auch das Ausführen beider Pfade mit Hochkomma klappt nicht.
Ich muss quasi zuerst irgendwie ins Verzeichnis navigieren und dann ausführen, anders klappt es nicht.
Hast du hier noch eine Idee?

[DMW007]

Dass selbst der maskierte Pfad funktioniert, wundert mich zugegeben auch etwas. Aber wie man sieht ist es kein generelles Problem der Shell oder des C# Codes sondern hängt damit zusammen, was dieses Skript macht. Daher auch die Nachfrage und Anregung, das versuchen nativ zu lösen.

Wenn das Batchskript andere Dateien mit relativen Pfaden referenziert, findet er die ohne entsprechend gesetzten cwd nicht. Die Fehlermeldung wird vermutlich daher stammen. Das kannst du nur auf zwei Arten lösen:
1. Das Batchskript ist entsprechend flexibel geschrieben und nutzt %~dp0 als Präfix für alle Befehle, um absolute Pfadangaben zum Skript selbst hin zu erzeugen
2. Wie hier gezeigt das Arbeitsverzeichnis vorher durch cd entsprechend zu wechseln:

cmd /c "cd C:\install-test && test.bat"

Beispiel:

:: test.bat
@echo "Check ffmpeg version"
ffmpeg -version

Wird das aus einem anderen Verzeichnis aufgerufen, wirft der ffmpeg Aufruf einen Fehler, weil er relativ sucht (hier in C:\Users\Daniel):

C:\Users\Daniel>C:\install-test\test.bat
"Check ffmpeg version"

C:\Users\Daniel>ffmpeg -version
Der Befehl "ffmpeg" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.

C:\Users\Daniel>cmd /c "cd C:\install-test && test.bat"
"Check ffmpeg version"

C:\install-test>ffmpeg -version
ffmpeg version N-93447-g5252d594a1 Copyright (c) 2000-2019 the FFmpeg developers
built with gcc 8.2.1 (GCC) 20190212
...

Wenn man den ffmpeg Aufruf nun mit absolutem Pfad durchführt

@set cwd=%~dp0
%cwd%ffmpeg -version

Dann kann das Skript auch aus einem anderen Arbeitsverzeichnis heraus aufgerufen werden:

C:\Users\Daniel>C:\install-test\ffmpeg -version
ffmpeg version N-93447-g5252d594a1 Copyright (c) 2000-2019 the FFmpeg developers
built with gcc 8.2.1 (GCC) 20190212
...