24_TPLinkHS110.pm: Support für TPLink HS100/HS110 v1/2/3 Steckdosen

[DS_Starter]

Hallo Volker,

ich habe noch eine kleine Unschönheit im Modul festgestellt. Wenn das Modul disabled ist, erscheinen unter "set" ein paar unsinnige Auswahlmöglichkeiten. Der angehängte diff behebt das.

viele Grüße
Heiko

[Volker Kettenbach]

Hi Heiko,

ich kann den Patch nicht anwenden. Da stimmt was nicht.

vsauer@volkers-mbp: ~/GitHub/FHEM-TPLink-HS110[develop*]>patch --dry-run < /tmp/24_TPLinkHS110.diff
(Stripping trailing CRs from patch.)
patching file 24_TPLinkHS110.pm
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n] <
Skipping patch.
4 out of 4 hunks ignored -- saving rejects to file 24_TPLinkHS110.pm.rej

Der Inhalt des Patches sieht auch recht komisch aus.

Bitte den Patch gegen die Master Branch von https://github.com/kettenbach-it/FHEM-TPLink-HS110 erzeugen!

Gruß
Volker

[DS_Starter]

Hallo Volker,

sorry, das ist ein SVN-diff.
Kommst du damit klar ? Wenn nicht versuche ich mich nochmal mit dem GIT.

Grüße
Heiko

[Volker Kettenbach]

Ob Git oder Svn ist doch letztlich egal.
Es muss ein mit diff erzeugter Patch sein, der sich auf den Master Branch bezieht

[DS_Starter]

Tja, tut mir leid, in dem Git-Molloch habe ich jetzt wieder endlos Zeit verbracht um irgendeine Stelle zu finden wo man "einfach" einen diff erzeugen kann. Eine simple Beschreibung für solche Tätigkeiten kenne ich ja nach wie vor nicht. Ist mir jetzt echt zu viel Aufwand und schade um die vergeudete (Frei)Zeit nur um eine Zeile zu ändern:

Code Auswählen Erweitern

252 -> return "Device disabled in config" if ($attr{$name}{"disable"} eq "1");

ändere in:

Code Auswählen Erweitern

return if (IsDisabled($name));

Dann passt das und funktioniert wie gewünscht.

Aber habe mit meinem Tortoise-Diff nochmal ein File gegen das SVN erstellt. Sieht jetzt besser aus denke ich.

[Volker Kettenbach]

Tja, tut mir leid, in dem Git-Molloch habe ich jetzt wieder endlos Zeit verbracht um irgendeine Stelle zu finden wo man "einfach" einen diff erzeugen kann. Eine simple Beschreibung für solche Tätigkeiten kenne ich ja nach wie vor nicht. Ist mir jetzt echt zu viel Aufwand und schade um die vergeudete (Frei)Zeit nur um eine Zeile zu ändern:

Hier steht wie das geht:

https://feldspaten.org/2014/12/05/patch-erzeugen-diff-patch/

Mit GIT oder SVN hat das im Grunde ja gar nichts zu tun.
Die Tool heissen diff und patch und können auch aus irgendwelchen GUI wie Eclipse, IntelliJ usw. aufgerufen werden.
Ggf. auch von irgendwelchen GIT oder SVN-Tools.

Vlt. solltest Du Dir wegen GIT das hier mal anschauen:
https://www.sourcetreeapp.com/
Das kann eine Alternative sein, wenn Du keine IDE verwendest.
Ich nutze es gelegentlich, wenn ich nicht meine IDE starten will.

Stichwort IDE: evtl. wäre das hier sinnvoll für Dich: https://code.visualstudio.com/
Mit den entsprechenden Plugins kann das auch Perl. GIT oder SVN sowieso.
Bei diesen Tools heisst das einfach nur VCS (Version Control System).

Ansonsten kann ich Dich nur ermuntern, Dich mit GIT zu beschäftigen.
SVN ist mausetot und im Vergleich zu GIT auch wie Käfer zu Porsche 911.

Die Codezeile habe ich geändert und ins GIT und SVN eingecheckt.

[DS_Starter]

Hi Volker,

danke für die Links. Schaue ich mir an.
Da war ich gestern wohl etwas sehr genervt 

Grüße
Heiko

[Fritz Muster]

Hallo in die Runde,

ich habe ein Problem auf einem Whezzy Pi mit cpan. Hier der Thread dazu. Wäre klasse wenn mich da jemand unterstützen könnte.

Danke und Grüße

Fritz

[Fritz Muster]

Hallo,

also ich kriege das nicht hin. Habe auf meinem Raspberry (wheezy) nun via cpan IO::Socket::Timeout die benötigten Pakete installiert. Bekomme aber wenn ich

Code Auswählen Erweitern

define MbAktTplWlan01 TPLinkHS110 192.168.178.53 eingeben ein

Code Auswählen Erweitern

Cannot load module TPLinkHS110 als Ausgabe.

Wäre dankbar wenn mich hier jemand unterstützen könnte.

Grüße Fritz

[Brockmann]

Code Auswählen Erweitern

Cannot load module TPLinkHS110 als Ausgabe.
Wäre dankbar wenn mich hier jemand unterstützen könnte.

Wenn Du in die Log-Datei schaust, sollte da für diesen Zeitpunkt eine ausführlichere Fehlermeldung stehen. Daran kann man sehen, woran es genau hapert...

[Fritz Muster]

Zunächst einmal Danke für den Hinweis.

Im Log steht folgendes

Code Auswählen Erweitern

2018.07.09 18:02:10 1: reload: Error:Modul 24_TPLinkHS110 deactivated:
Excessively long <> operator at ./FHEM/24_TPLinkHS110.pm line 21.

2018.07.09 18:02:10 0: Excessively long <> operator at ./FHEM/24_TPLinkHS110.pm line 21.

Güße
Fritz

[Brockmann]

Zunächst einmal Danke für den Hinweis.

Hast Du die aktuelleste Version der TPLinkHS110.pm? Es gab da kürzlich einige Änderungen, damit auch Geräte mit neuerer Firmware unterstützt werden.

[Fritz Muster]

Hast Du die aktuelleste Version der TPLinkHS110.pm? Es gab da kürzlich einige Änderungen, damit auch Geräte mit neuerer Firmware unterstützt werden.

Ja, habe von github die aktuelle Version von DS_Starter runtergeladen und ins Verzeichnis fhem/FHEM kopiert. Rechte und Gruppe angepasst. Werde das aber nochmal kontrollieren.

Danke

[Brockmann]

Ja, habe von github die aktuelle Version von DS_Starter runtergeladen und ins Verzeichnis fhem/FHEM kopiert. Rechte und Gruppe angepasst. Werde das aber nochmal kontrollieren.

Also das ist dann zumindest nicht die aktuelle "offizielle" Version der Datei. Warum nimmst Du nicht einfach die Version, die zum aktuellen Lieferumfang von FHEM gehört?
Da sollten die Änderungen von DS_Starter auch schon integriert sein, wenn ich die letzten Posts hier richtig interpretiere.

Ein Fehler in Zeile 21 kommt mir jedenfalls komisch vor. Bei der "offiziellen" Datei sind die ersten 30 Zeilen Kommentare, da kann gar kein Fehler auftreten.
Hast Du mal in die Datei (bei Dir) reingeschaut? Eventuell ist da beim Runterladen was schiefgelaufen?

[Fritz Muster]

Hast Du mal in die Datei (bei Dir) reingeschaut? Eventuell ist da beim Runterladen was schiefgelaufen?

Danke für den Tipp. In der Datei stand tatsächlich Kokolores. Habe die Datei nochmal gezogen und nun läuft es.

Vielen Dank und Grüße

Fritz