Jey-Cee / ioBroker.enocean

Connect and control your EnOcean devices with ioBroker.
Other
24 stars 12 forks source link

[Bug]: Fehlendes Verzeichnis /dev/serial/by-id führt zu Fehlermeldung #170

Closed uklatt closed 9 months ago

uklatt commented 10 months ago

Contact Details

No response

What happened?

Bei Version 0.9.0 innerhalb eines buanet Docker Containers und Nutzung über ser2net erscheint die o.g. Fehlermeldung.

Ich habe das Verzeichnis manuell erstellt. Dann erscheint die Meldung nicht mehr.

Tschau Uwe

Adapter Version

0.9.0

JS-Controller Version

4.0.24

Node JS Version

18.17.1

NPM Version

9.6.7

Operating System

Debian

Relevant log output

No response

Anything else?

No response

Jey-Cee commented 10 months ago

Danke fürs melden. Welche Meldung meinst du? Ein Log oder Screenshot wäre hier gut.

uklatt commented 10 months ago

Wenn das Verzeichnis nicht existiert, kommt die Fehlermeldung bei jedem Start des Adapters. In meiner Docker Konfiguration ist das Verzeichnis standardmäßig nicht vorhanden. Ich musste es manuell anlegen, damit der Adapter nicht mehr "meckert". Das tritt anscheinend auf, wenn keine USB devices vorhanden sind. (Ich nutze nur ser2net für die Anbindung.)

Tschau Uwe

Holger-Will commented 10 months ago

kommt die Fehlermeldung bei jedem Start des Adapters. was sagt die Fehlermeldung genau? Wäre super, wenn du das Verzeichnis nochmal löschen könntest, und dann einen Screenshot oder den Text aus den Logs hier posten könntest...

uklatt commented 10 months ago

Ich wollte heute das Verzeichnis testhalber entfernen und habe dabei festgestellt: Es ist nicht mehr da! Keine Ahnung, ob der udev daemon das gelöscht hat. Interessanterweise kommt die Fehlermeldung nun nicht mehr (auch ohne das Verzeichnis /dev/serial/by-id). Sehr merkwürdig.

Tschau Uwe

Holger-Will commented 10 months ago

Danke, dass Du das nochmal getestet hast/bzw. testen wolltest.

@Jey-Cee sollte das nicht mit dem commit erledigt sein?

Jey-Cee commented 10 months ago

@Holger-Will ich glaub du hast den falschen commit erwischt, es ist dieser hier: 9f3e4a40c854cdf14b6a193a4dedb7aa573168ad

Holger-Will commented 10 months ago

uups! ja richtig

Holger-Will commented 10 months ago

181 beinhaltet einen fix: prüfen ob /dev/serial/by-id vorhanden ist bevor es gelesen wird... ich denke wir können dann auch den try catch block drum herum löschen...

Holger-Will commented 9 months ago

mit dem merge von heute, kann das geschlossen werden.