Terminalbefehl als App / Script umsetzen

[Atalantia]

Gib mal als Zeile 3 ein:

display dialog result as string

Dann siehst Du die Antwort die zurückkommt.

 

[Maddeen]

Also so?

try
    do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"
    display dialog result as string
    if result is "Domain Windows_10q35_5_own_vBIOS started" then
        display notification "VM started successfully" with title "Gaming VM"
    else
        display notification "VM couldn't started" with title "Gaming VM"
    end if
on error
    display notification "Script Error" with title "Gaming VM"
end try

 

[Maddeen]

Hat geklappt - VM läuft auch!
Allerdings verstehe ich nicht, warum er die Fehlermeldung bringt. Für mich sieht das Result genau so aus, wie ich es eingegeben habe

Der Editor gibt im Bereich ANTWORTEN folgendes aus

tell current application
    do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"
        --> "Domain Windows_10q35_5_own_vBIOS started
"
end tell
tell application "Script Editor"
    display dialog "Domain Windows_10q35_5_own_vBIOS started
"
        --> {button returned:"OK"}
    display notification "VM couldn't started" with title "Gaming VM"
end tell

Im Bereich EVENTS steht folgendes:

tell current application
    do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"
end tell
tell application "Script Editor"
    display dialog "Domain Windows_10q35_5_own_vBIOS started
"
    display notification "VM couldn't started" with title "Gaming VM"
end tell

 

[oneOeight]

Da ist wieder das Problem mit dem Zeilenumbruch.
Mach mal result contains statt =

if result contains "Domain Windows_10q35_5_own_vBIOS started" then

 

[Maddeen]

Leider nein @oneOeight
Script wie folgt angepasst:

try
    do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"
    display dialog result as string
    if result contains "Domain Windows_10q35_5_own_vBIOS started" then
        display notification "VM started successfully" with title "Gaming VM"
    else
        display notification "VM couldn't started" with title "Gaming VM"
    end if
on error
    display notification "Script Error" with title "Gaming VM"
end try

EVENTS

tell current application
    do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"
end tell
tell application "Script Editor"
    display dialog "Domain Windows_10q35_5_own_vBIOS started
"
    display notification "VM couldn't started" with title "Gaming VM"
end tell

ANTWORTEN

tell current application
    do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"
        --> "Domain Windows_10q35_5_own_vBIOS started
"
end tell
tell application "Script Editor"
    display dialog "Domain Windows_10q35_5_own_vBIOS started
"
        --> {button returned:"OK"}
    display notification "VM couldn't started" with title "Gaming VM"
end tell

 

[oneOeight]

Dann probier mal Varianten.

if result is "Domain Windows_10q35_5_own_vBIOS started
" then

if result is "Domain Windows_10q35_5_own_vBIOS started\n" then

 

[Maddeen]

Leider auch ohne Erfolg.
Die letzte Variante scheint er auch nicht zu akzeptieren, weil er es sofort nach dem Ausführen so anpasst, wie deine erste Variante

 

[oneOeight]

\n steht auch für newline, also neue Zeile.
Vielleicht ist es auch \r also Carriage return.
Also probier mal \r.

Sonst vereinfachen mit
If result contains "started"

 

[Atalantia]

Du musst

display dialog result as string

wieder rausnehmen sonst ist result danach etwas anderes. Danach oneOeights Vorschlag nochmal versuchen.

 

[Maddeen]

Ahhh .. jetzt gehts. Vielen Dank an euch zwei!!!
Die Lösung war also in der Tat den display dialog wieder zu entfernen und dann mit "result contains" zu arbeiten!!
Screenshot und finales Script habe ich für die Nachwelt jetzt noch mal angehangen.

Generell noch eine Frage bzgl. einer "Gold-Rand-Lösung".
Weiß jemand von euch ggf. auch, wie ich es hinbekomme, dass die Notifications nicht den Scripteditor als "Absender" haben sondern den Namen der "App".
Oder geht das dann wirklich nur, wenn ich eine native App daraus mache?

try
    do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"
    if result contains "Domain Windows_10q35_5_own_vBIOS started" then
        display notification "VM started successfully" with title "Gaming VM"
    else
        display notification "VM couldn't started" with title "Gaming VM"
    end if
on error
    display notification "Script Error" with title "Gaming VM"
end try

 

[oneOeight]

Probier doch aus und speicher als Programm.

 

[Maddeen]

Das hab ich ja schon. Daher die "Gänsefüßchen" bei "App"
Vom generellen Aufbau (Paketinhalt) sieht es auch nach einer vollwertigen App aus.

Ich guck mal, ob ich sonst noch Infos finden kann

 

[oneOeight]

Info.plist bearbeiten

 

[Maddeen]

Ok... ich hatte eigentlich das standard icns mit meiner File überschrieben. Warum auch immer, mag er das wohl nicht.
Jetzt habe ich die icns umbenannt und die plist entsprechend angepasst - läuft

Danke dir noch mal und schönes Wochenende.

P.S
Sofern du kannst / magst, würde ich mich über einen Link freuen, wo ich erkennen kann, wie man unterschiedliche Error-Messages auf Basis der Error-Results ausgeben kann
Also ähnlich der "if result contains"
sowas wie

• if error contains "domain already active"
• display notification "Your VM is already started"
• if error conatins "domain not found"
• display notification "Your VM can not be found"

Aktuell habe ich ja nur ein "on error" - das kann aber natürlich mehrere Ursachen haben.

 

[oneOeight]

Da musst du zwischen dem return code von do shell script und dem result unterscheiden.
On error triggert wohl nur wenn do shell script nicht richtig beendet.
Du musst also result auswerten.
D.h. if then else für alle Fälle.

 

[Atalantia]

on error erstrg

speichert die Fehlermeldung in der Variablen erstrg

 

[Atalantia]

Du kannst auch mit

set r to do shell script "/usr/bin/ssh root@10.20.0.1 virsh start Windows_10q35_5_own_vBIOS"

das result vom shell script in der variablen r speichern und später mit ifs auswerten.

 

[Maddeen]

Danke... ich versuch mich mal reinzufuxxen Vielleicht bekomme ich es auch mit swift in eine saubere app, wo der User den Namen seiner VM direkt eingeben und speichern kann.
Bzw. von mehreren VMs ... hab mir gerade bei udemy nen angeblich guten Kurs geschossen... we'll see.

Schönes WE

 

[Atalantia]

Swift ist nicht schwierig aber das Cocoa Framework ist kompliziert.

 

[mausfang]

Mach mal nach dem do Shell ein „log result“ ...