Probleme mit if, then, else im Script

schmetty2 · 24. September 2018, 09:10

Hallo,

ich möchte einem vorhanden Script die powerstate Abfrage voran stellen. Das funktioniert soweit auch.

instandby=$(wget -qO - powerstate |grep '</e2instandby>' | sed -e 's/<\/e2instandby>//g' | sed -e 's/false/on/g' | sed -e 's/true/off/g')

liefert je nach Zustand on oder off. Allerdings kriege ich kein nachfolgendes if, then, else hin.

if [ "$instandby" ] = [ "on" ]
then
wget -q -O - 'http://vuuno4kse/web/message?text=test&type=2&timeout=10'

Damit passiert nichts. Hat jemand einen Tip für mich, wo liegt mein Fehler?

mfg

tsiegel · 24. September 2018, 09:29

Mal so probiert

if [ "$instandby" ] = [ "on" ] ; then
wget -q -O - 'http://vuuno4kse/web/message?text=test&type=2&timeout=10'
fi

tsiegel · 24. September 2018, 09:41

und evt. mal den Befehl abändern in :
wget 'http://vuuno4kse/web/message?text=test&type=2&timeout=10'

rdamas · 24. September 2018, 10:03

Shell-Script

if [ "x$instandby" = "xon" ]; then
wget -q -O - 'http://vuuno4kse/web/message?text=test&type=2&timeout=10'
fi

tsiegel · 24. September 2018, 10:09

@rdamas

schmetty2 schrieb:

liefert je nach Zustand on oder off

Nur mal so für nen wissensdurstigen Anfänger. Für was steht denn jeweils das x in [ "x$instandby" = "xon" ] ?

rdamas · 24. September 2018, 10:33

Damit kein Leerstring verglichen werden muss, wird auf beiden Seiten ein "x" am Argument hinzugefügt. Oder ein anderes Zeichen, wie du magst. Das ist hilfreich, vor allem wenn mal wieder nicht klar ist, welche Shell den Test am Ende durchführt. Da gibt es leider subtile Unterschiede - das mit dem zusätzlichen Zeichen im String funktioniert aber aus meiner Erfahrung so gut wie immer.

schmetty2 · 24. September 2018, 12:39

Danke für Eure Tips.
Aber leider poppt die Messagebox immer noch nicht auf.
Hab's auch mal mit den urprünglichen false und true probiert, kommt auch nix.

tsiegel · 24. September 2018, 12:52

Kommt denn die Message hoch, wenn Du die Überprüfung mal testweise weglässt und nur
wget -q -O - 'http://vuuno4kse/web/message?text=test&type=2&timeout=10'
eingibst?

schmetty2 · 24. September 2018, 13:29

Ja,
stell ich sie an den Anfang poppt sie auf, also das Script startet.

markusw · 24. September 2018, 13:37

Was steht in $instandby ?

tsiegel · 24. September 2018, 13:44

schmetty2 schrieb:

instandby=$(wget -qO - powerstate |grep '</e2instandby>' | sed -e 's/<\/e2instandby>//g' | sed -e 's/false/on/g' | sed -e 's/true/off/g')

ist das ein Tippfehler? Fehlt da nicht ein Bindestrich vor dem O?

instandby=$(wget -q -O - powerstate |grep '</e2instandby>' | sed -e 's/<\/e2instandby>//g' | sed -e 's/false/on/g' | sed -e 's/true/off/g')

rdamas · 24. September 2018, 13:50

Wäre jetzt auch mein Verdacht, dass da nicht "on" drin steht, sondern evtl. noch Leerzeichen und Leerzeile.

Versuch mal, hinter dem letzten sed in der Zeile wo du instandby liest, noch ein "| xargs" dran zu setzen, um die Leerzeichen wegzubekommen. Vielleicht tut's das schon. Ansonsten vielleicht das Shell-Script mit -x starten, um zu schauen was da passiert.

Edit: @tsiegel: das ist kein Tippfehler, -qO ist kurz für -q -O.

tsiegel · 24. September 2018, 13:59

Man lernt halt täglich was dazu.

jones1024 · 24. September 2018, 14:12

Und so:??

Shell-Script

#!/bin/bash -x
## das -x ist für Debugausgaben
instandby=$(wget -O - -q http://IP-der-Box/web/powerstate | grep "$<\|</$e2instandby" | tr -d '\n' | sed "s/.*<e2instandby>$.*$<\/e2instandby.*/\\1\n/g")
if [ $instandby == "true" ]; then
echo "Box ist im Standby"
elif [ $instandby == "false" ]; then
echo "Box ist nicht im Standby"
else
echo "Box ist nicht erreichbar"
fi

Alles anzeigen

jones1024 · 24. September 2018, 15:30

Wenn du noch um ein tr bei der Variablenbildung erweiterst, kannst du die Variablen im if else Bereich auch in Anführungszeichen schreiben.

Shell-Script

#!/bin/bash -x
instandby=$(wget -O - -q http://IP-der-Box/web/powerstate | grep "$<\|</$e2instandby" | tr -d '\n' | sed "s/.*<e2instandby>$.*$<\/e2instandby.*/\\1\n/g" | tr -d '[:space:]')
if [ "$instandby" == "true" ]; then
echo "Box ist im Standby"
elif [ "$instandby" == "false" ]; then
echo "Box ist nicht im Standby"
else
echo "Box ist nicht erreichbar"
fi

Alles anzeigen

schmetty2 · 24. September 2018, 19:45

So,
jetzt gehen gar keine wget Befehle mehr. Ich mach morgen weiter.
Danke für Eure Beiträge.

mfg

schmetty2 · 25. September 2018, 08:21

schmetty2 schrieb:

So,
jetzt gehen gar keine wget Befehle mehr.

Das Phänomen verstehe ich nicht. Erstelle ich eine neue Scriptdatei, läuft das Script nicht. Nehme ich die Kopie eines vorhanden, welches schon auf der Box läuft, gehts.
Das selbe hatte ich schon mal Python Scripten auf'm Androiden, deshalb viel mir das wieder ein.
Jedenfalls geht es jetzt. Allerdings nicht ohne "elif" Auch mit der Messagebox.

Danke

Shell-Script

#!/bin/sh
instandby=$(wget -qO - http://IP/web/powerstate |grep '</e2instandby>' | sed -e 's/<\/e2instandby>//g' | sed -e 's/false/no/g' | sed -e 's/true/yes/g')
sleep 5
if [ $instandby == "yes" ]; then
wget -q -O - http://IP/web/remotecontrol?command=116
elif [ $instandby == "no" ]; then
wget -q -O - http://IP/web/remotecontrol?command=139
else
echo
fi

Alles anzeigen

schomi · 25. September 2018, 09:30

Wird daran liegen das die neue Datei nicht ausführbar ist. Berechtigung korrigieren.

schmetty2 · 25. September 2018, 09:43

Ne,
sind alle auf 755
Auserdem würde beim Erstellen eines CronJobs die Fehlemeldung" kann nicht ausgeführt werden" oder so ähnlich kommen.

jones1024 · 25. September 2018, 09:54

Wie sieht denn die Debugausgabe im Terminal aus?

Was kommt denn bei der Ausführung von wget -O - -q http://IP-der-Box/web/powerstate ???

Also IP-der-Box musst du durch eine IP ersetzen