Autostart: Shell Script wird nicht richtig ausgeführt

ni28 · October 11, 2020 at 7:04 PM

Hallo zusammen,

ich habe ein Problem mit dem Starten eines Shell Scriptes im Autostart auf meine Pi 3. Ich benutze eine Relais-Karte und jetzt geht es darum, dass der ursprüngliche Zustand der Relais nach einem Stromausfall (und damit Neustart des Pis) wieder hergestellt wird. Dazu habe ich einfach eine Textdatei, in der der Zustand bei jeder Änderung der Relais als Zahl gespeichert wird.

Das Script soll jetzt beim Neustart einfach die Zahl aus der Datei auslesen und per If-Abfrage eben die Relais-Stellung wieder herstellen. Wenn ich das Script per Hand aufrufe, tut es genau was es soll. Allerdings geht es nicht, wenn es per Autostart aufgerufen wird (ich habe alle gängigen Methoden wie rc.local, crontab, startup job, lxde versucht). Wenn ich mit htop nach den laufenden Prozessen schaue, so ist der Prozess dort orange markiert zu sehen, was mich stutzig macht, da das Script ja nur einmal durchläuft und längst fertig sein sollte.

Hier mein Code:

Bash

#!/bin/bash

read status < status_datei

if [ $status == 0 ]
then
    ./relais1.sh
fi


if [ $status == 1 ]
then
    ./relais2.sh
fi


if [ $status == 2 ]
then
    ./relais3.sh
fi

Display More

Die drei Scripte in den If-Abfragen stellen nur die Relais ein.

Vielen Dank schonmal für Eure Hilfe!

Grüße

ni28

Bot · Vor 5 Minuten

Autostart: Shell Script wird nicht richtig ausgeführt? Schau mal ob du hier fündig wirst!

__blackjack__ · October 11, 2020 at 7:24 PM

ni28 Das funktioniert ja schon mal nur dann, wenn das aktuelle Arbeitsverzeichnis stimmt, weil alle Dateien dort relativ angegeben sind. Also die Frage: Stimmt das aktuelle Arbeitsverzeichnis denn wenn Du das wie auch immer nicht manuell startest?

Fliegenhals · October 11, 2020 at 7:27 PM

Hallo und willkommen im Forum,

als erste Maßnahme würde ich es mal mit absoluten statt der relativen Pfade im Script probieren.

Bot · Vor einem Moment

ni28 · October 11, 2020 at 8:51 PM

Ich bin vielleicht ein Held... Mit absoluten Pfaden geht's natürlich..

Vielen Dank für die schnelle Hilfe!

framp · October 11, 2020 at 9:07 PM

... noch auf erledigt setzen

ni28 · October 11, 2020 at 10:01 PM

Hm eine Kleinigkeit scheint doch noch irgendwo in einem anderen Script zu sein. In diesem werden drei Eingänge gepolled und anhand dieser Eingänge die Relais gestellt. Auch dieses Script soll per Autostart gestartet werden (ja ich weiß, Interrupt ist schon schöner, aber es geht nur ums Ausprobieren und sonst läuft kein Script dauerhaft).

Auch in diesem Script waren die Pfade nicht absolut. Das hab ich geändert. Aber auch jetzt lässt sich das Script zwar per Hand starten, nicht jedoch im Autostart. Auch wenn ich die Endlosschleife wieder raus nehme, geht's nicht. Hab ich noch was übersehen? Beim vorherigen Script hat's ja geklappt mit den absoluten Pfaden

Bash

#!/bin/bash

while [ 1 ]
do

read status < /home/pi/Desktop/status_datei
read state_relais1 < /sys/class/gpio/gpio17/value
read state_relais2 < /sys/class/gpio/gpio22/value
read state_relais3 < /sys/class/gpio/gpio27/value


if [ $state_relais1 == 1 ]
then
    if [ $status != 0 ]
    then
        /home/pi/Desktop/relais1.sh
        echo "0"> /sys/class/gpio/gpio23/value
        echo "0"> /sys/class/gpio/gpio24/value
        echo "1"> /sys/class/gpio/gpio18/value
    fi
fi


if [ $state_relais2 == 1 ]
then
    if [ $status != 1 ]
    then
        /home/pi/Desktop/relais2.sh
        echo "0"> /sys/class/gpio/gpio18/value
        echo "0"> /sys/class/gpio/gpio24/value
        echo "1"> /sys/class/gpio/gpio23/value
    fi
fi


if [ $state_relais3 == 1 ]
then
    if [ $status != 2 ]
    then
        /home/pi/Desktop/relais3.sh
        echo "0"> /sys/class/gpio/gpio18/value
        echo "0"> /sys/class/gpio/gpio23/value	
        echo "1"> /sys/class/gpio/gpio24/value	
    fi
fi
done

Display More

Bot · Vor einem Moment

framp · October 11, 2020 at 10:46 PM

Redirecte doch mal die Ausgaben beim Aufruf des Scripts im Autostart in eine Log Datei. In der solltest Du dann Fehlermeldungen sehen die Dir eine Indikation geben wo der Hase im Pfeffer liegt

**hyle** · October 11, 2020 at 10:50 PM Official Post

Autostart womit (auf welche Weise) und mit welchem Befehl genau?

//Edit: Sorry framp! In Villarriba wurde noch gelesen, während...

framp · October 11, 2020 at 10:53 PM

hyle Wenn die Scriptausgaben in eine Log Datei geschrieben werden sollte das eigentlich irrelevant sein. Aber sehen wir mal

Bot · Vor einem Moment

**hyle** · October 11, 2020 at 10:55 PM Official Post

Ich habe da so eine Vermutung, aber ja. Warten wir es ab!

__blackjack__ · October 12, 2020 at 3:09 AM

ni28 Die Endlosschleife würde ich mit ``true`` schreiben. Und sei es nur um zu vermeiden das Leute das lesen und denken die 1 hätte irgendwie eine besonders ”wahre” Bedeutung.

Bei der Bash nehme ich immer ``[[ … ]]`` für Tests. Dann muss man sich beispielsweise weniger Gedanken ums ”quoten” machen.

Das die Ausgaben auf die Pins nach dem Ausgabewert und nicht nach der Pinnummer sortiert sind, finde ich verwirrend.

Da steht ja im Grunde dreimal kopiert und leicht angepasst Code mit der gleichen Struktur mit unterschiedlichen Werten. Solche Wiederholungen vermeidet man als Programmierer eigentlich immer durch Schleifen und/oder Funktionen.

Die beiden verschachtelten ``if`` in jedem der Blöcke kann man mit einem einzigen ``if`` abfrühstücken.

Ich würde da ja noch eine kleine Wartezeit einbauen, damit das Polling nicht 100% CPU-Zeit verbraucht. Ich weiss es wird Winter, aber man kann auch anders heizen. ?

Zwischenstand, ungetestet, und natürlich immer noch mit dem Problem, falls ich das nicht aus Versehen gefixt habe:

Bash

#!/bin/bash

readonly InputPins=( 17 22 27 )
readonly OutputPins=( 18 23 24 )

while true
do
    read -r status < /home/pi/Desktop/status_datei
    for (( i = 0; i < ${#InputPins[*]}; i++ ))
    do
        read -r relais_state < "/sys/class/gpio/gpio${InputPins[i]}/value"
        if [[ $relais_state == 1 ]] && [[ $status != "$i" ]]
        then
            "/home/pi/Desktop/relais$((i + 1)).sh"
            for (( j = 0; j < ${#OutputPins[*]}; j++ ))
            do
                echo $((i == j)) > "/sys/class/gpio/gpio${OutputPins[j]}/value"
            done
        fi
    done
    sleep 0.01
done

Display More

Ich persönlich würde da wahrscheinlich zur leichteren Lesbarkeit mindestens zwei bis drei Funktionen einführen.

ni28 · October 12, 2020 at 8:41 PM

Hallo,

danke für Eure Antworten.

Quote from hyle

Autostart womit (auf welche Weise) und mit welchem Befehl genau?

Wie eingangs beschrieben, hab ichs mit allen gängigen Methoden versucht ohne Erfolg. Zur Zeit mache ich es über rc.local

Quote from framp

Redirecte doch mal die Ausgaben beim Aufruf des Scripts im Autostart in eine Log Datei. In der solltest Du dann Fehlermeldungen sehen die Dir eine Indikation geben wo der Hase im Pfeffer liegt

Bin mir nicht ganz sicher, ob ich es richtig gemacht habe. Wollte stdout und stderr speichern und habe dafür in rc.local den Script-Aufruf folgendermaßen angepasst:

Bash

/home/pi/Desktop/steuerung.sh >> error.log 2>&1

Die Datei ist nach reboot leer.

@__blackjack__: Ja du hast recht, dass der Code nicht sehr "effizient" geschrieben ist. Das liegt hauptsächlich daran, dass ich zwischendurch immer wieder was geändert hab (uA wegen dem Autostart Problem). Die Reihenfolge des Setzten der Pins hat tatsächlich so einen Sinn, da die Reihenfolge mit Ausschalten und Einschalten eingehalten werden muss. Da kommen auch noch sleeps dazwischen, damit die Relais auch genug Zeit zum schalten haben. Aber trotzdem danke für deine Anmerkungen, ich werde sie später sicherlich beherzigen.

Bot · Vor einem Moment

ni28 · October 24, 2020 at 12:25 PM

Hallo,

auch nach weiteren Versuchen hat sich am Verhalten des Scripts nichts geändert. Eine Fehlermeldung (wie ich in meinem vorherigen Post geschrieben habe) sehe ich auch keine.

Was mir noch aufgefallen ist: Wenn ich mir mit htop den Prozess, der durch Autostart gestartet wurde, anschaue, so ist als User "root" eingetragen. Wenn ich das Script von Hand starte, ist es der User "pi". Das ist ja auch nachvollziehbar. Könnte dies aber zu meinem Fehlerbild führen? Zumindest kann es doch kein Problem mit den Rechten sein, wenn das Autostart-Script als root ausgeführt wird?

Danke für Eure Antworten und Grüße

ni28

ni28 · October 25, 2020 at 10:56 AM

Hallo,

ich denke, ich habe das Problem lokalisiert.

Ich habe das Script jetzt nochmal in C geschrieben. Auch hier passiert das selbe. Aus diesem Grund habe ich den Aufruf des zweiten sh-Scripts raus genommen und dessen Inhalt direkt in meinem C Programm eingebaut. Und jetzt kann ich auch in meiner error-Datei einen Fehler sehen. Ich rufe per SSH auf einem anderen Pi ein Script auf und hieran scheint es zu hängen, der Rest des C-Programms läuft nämlich wie es soll.

Code

Permission denied, please try again.
Permission denied, please try again.
pi@192.168.0.80: Permission denied (publickey,password).

Der SSH Zugriff erfolgt ohne Passwort über einen public key. Das Programm wird im Autostart als root ausgeführt, der key ist aber für den user pi angelegt. Ich könnte mir vorstellen, dass das das Problem ist.

Jetzt stellt sich natürlich dann die Frage: Wie lösen?

Hat jemand eine Idee?

Grüße und Danke

**hyle** · October 25, 2020 at 12:39 PM Official Post

Quote from ni28

Ich rufe per SSH auf einem anderen Pi ein Script auf

Ist das eine neue Info oder habe ich das überlesen?

Quote from ni28

Jetzt stellt sich natürlich dann die Frage: Wie lösen?

Dann starte das ganze doch mal als pi! Möglichkeiten gibt es ja nicht nur eine. Z.B. kann man das per Cronjob oder mit einer Systemd Unit.

Bot · Vor einem Moment

ni28 · October 25, 2020 at 1:48 PM

Quote from hyle

Ist das eine neue Info oder habe ich das überlesen?

Ja das ist eine neue Info. Anfangs dachte ich noch, dass es nur am Aufruf des Scriptes liegen würde.. Schande über mein Haupt

Mein Versuch eines "su pi -c" im rc.local zielte auch darauf ab, war allerdings nicht von Erfolg gekrönt.

Systemd scheint soweit zu funktionieren!

Hätte ich Schlauberger mal am Anfang das ganze Problem beschrieben, wäre die Sache schnell erledigt gewesen... Besten Dank für die Hilfe!

holgerble · October 25, 2020 at 4:41 PM

Würde auch via crontab gehen..

@reboot /home/pi/scripts/dein_script.sh

Participate now!