Autor Thema: Patch 00_CUL.pm/culfw für WMBUS Typ C  (Gelesen 245 mal)

Online kaihs

  • Developer
  • Sr. Member
  • ****
  • Beiträge: 909
Patch 00_CUL.pm/culfw für WMBUS Typ C
« am: 24 Juni 2018, 11:07:11 »
Hallo Rudi,

anbei ein Patch für 00_CUL.pm und die culfw für die Unterstützung des Empfangs vom WMBUS Typ C.
Die Anpassungen an der culfw basieren im Wesentlichen auf der Arbeit von Ingram (https://github.com/Ingramz/culfw) mit einer kleinen Anpassung, um die Auswertung durch das WMBUS Modul zu erleichtern. Die Diskussion dazu findet sich hier.

Die Anpassungen wurden von einem User erfolgreich getestet.

Wäre schön wenn du die Patches einfließen lassen könntest.

Gruß,

Kai
Banana Pi, Add-On Board mit 1.8" TFT LCD und IR-Sender, FHEM V5.8, CULFW V1.61, div. Homematic Komponenten, Pollin Funksteckdosen, Selbstbau CUL433 MHz, Jeelink Clone, EMT7110, EC3000
Selbstbau CUL868MHz für Wireless M-Bus, FHEMduino mit Logilink Temp.-sensoren und Auriol Wetterstation

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 18630
Antw:Patch 00_CUL.pm/culfw für WMBUS Typ C
« Antwort #1 am: 24 Juni 2018, 19:54:08 »
Eigentlich sollte das Patchen der culfw Quellen der mbus Autor (tostmann) uebernehmen, aber ich vermute, er ist nicht so aktiv.

Ich habe aber auch Probleme:% patch -p0 < ../culfw_mbus_c.patch
(Stripping trailing CRs from patch.)
patching file clib/mbus/mbus_packet.c
Hunk #1 FAILED at 578.
1 out of 1 hunk FAILED -- saving rejects to file clib/mbus/mbus_packet.c.rej
(Stripping trailing CRs from patch.)
patching file clib/mbus/mbus_packet.h
Hunk #1 FAILED at 22.
Hunk #2 FAILED at 42.
2 out of 2 hunks FAILED -- saving rejects to file clib/mbus/mbus_packet.h.rej
(Stripping trailing CRs from patch.)
patching file clib/rf_mbus.c
(Stripping trailing CRs from patch.)
patching file clib/rf_mbus.h
(Stripping trailing CRs from patch.)
patching file docs/commandref.html
Da ich MBUS nicht testen kann, brauche ich funktionierende Patches.
Ich frage mich, wie Ihr das erfolgreich getestet habt.

Online kaihs

  • Developer
  • Sr. Member
  • ****
  • Beiträge: 909
Antw:Patch 00_CUL.pm/culfw für WMBUS Typ C
« Antwort #2 am: 24 Juni 2018, 20:33:56 »
Sorry, das liegt irgendwie an den Zeilenenden.
Die Dateien im mbus Verzeichnis sind im Windows Format (CR+LF), wahrscheinlich weil sie ursprünglich von Texas Instruments stammen.

svn diff gibt die dann anscheinend nicht mehr aus und patch scheitert dann.
Jetzt muss ich erst mal rausbekommen wie ich unter Linux einen patch einer Windowsdatei erzeugen kann der mit patch wieder angewendet werden kann.

Würdest du alternativ die beiden betroffenen Dateien (siehe Anhang) komplett übernehmen?
Ich könnte das auch selbst einchecken, ich habe Schreibzugriff auf das culfw Repository.
« Letzte Änderung: 24 Juni 2018, 20:36:14 von kaihs »
Banana Pi, Add-On Board mit 1.8" TFT LCD und IR-Sender, FHEM V5.8, CULFW V1.61, div. Homematic Komponenten, Pollin Funksteckdosen, Selbstbau CUL433 MHz, Jeelink Clone, EMT7110, EC3000
Selbstbau CUL868MHz für Wireless M-Bus, FHEMduino mit Logilink Temp.-sensoren und Auriol Wetterstation

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 18630
Antw:Patch 00_CUL.pm/culfw für WMBUS Typ C
« Antwort #3 am: 24 Juni 2018, 20:54:59 »
Zitat
Ich könnte das auch selbst einchecken, ich habe Schreibzugriff auf das culfw Repository.
Das waere mir lieber, dann stehe ich nicht in der SVN-Historie als der Suendenbock da :)

Auch dein 00_CUL.pm Patch ist problematisch, da es gegen eine mindestens ein Jahr alte Variante erstellt wurde: STACKABLE_CC wird seitdem ueber $sccMods referenziert, was auch TSSTACKED und STACKABLE enthaelt. Ich koennte es zurechbasteln, aber ich wuerde es gerne getestet haben vor dem Einchecken, und das kann ich nicht.Bei FHEM gelten andere Regeln, fuer 00_CUL.pm brauche ich ein Patch.

Online kaihs

  • Developer
  • Sr. Member
  • ****
  • Beiträge: 909
Antw:Patch 00_CUL.pm/culfw für WMBUS Typ C
« Antwort #4 am: 24 Juni 2018, 21:04:07 »
Ich frage mich, wie Ihr das erfolgreich getestet habt.

User zwehn hat einen Kamstrup Multical 21 der WMBUS C sendet.
Er hat eine von mir auf Basis dieser Quellen compilierte culfw auf einen nanoCUL geflasht und dann den Empfang getestet.
Das hat dann einschließlich der Decodierung durch das WMBUS Modul geklappt, siehe https://forum.fhem.de/index.php/topic,24517.msg812889.html#msg812889
Banana Pi, Add-On Board mit 1.8" TFT LCD und IR-Sender, FHEM V5.8, CULFW V1.61, div. Homematic Komponenten, Pollin Funksteckdosen, Selbstbau CUL433 MHz, Jeelink Clone, EMT7110, EC3000
Selbstbau CUL868MHz für Wireless M-Bus, FHEMduino mit Logilink Temp.-sensoren und Auriol Wetterstation

Online kaihs

  • Developer
  • Sr. Member
  • ****
  • Beiträge: 909
Antw:Patch 00_CUL.pm/culfw für WMBUS Typ C
« Antwort #5 am: 24 Juni 2018, 21:18:36 »
Auch dein 00_CUL.pm Patch ist problematisch, da es gegen eine mindestens ein Jahr alte Variante erstellt wurde

Mea culpa, ich habe einen total veralteten Patch in der ersten Nachricht angehängt :-(

Anbei die richtige Version, sorry nochmal.

Banana Pi, Add-On Board mit 1.8" TFT LCD und IR-Sender, FHEM V5.8, CULFW V1.61, div. Homematic Komponenten, Pollin Funksteckdosen, Selbstbau CUL433 MHz, Jeelink Clone, EMT7110, EC3000
Selbstbau CUL868MHz für Wireless M-Bus, FHEMduino mit Logilink Temp.-sensoren und Auriol Wetterstation

Online kaihs

  • Developer
  • Sr. Member
  • ****
  • Beiträge: 909
Antw:Patch 00_CUL.pm/culfw für WMBUS Typ C
« Antwort #6 am: 24 Juni 2018, 21:41:24 »
Das waere mir lieber, dann stehe ich nicht in der SVN-Historie als der Suendenbock da :)

Erledigt. Alle Beschwerden dazu also an mich weiterleiten  ;)
Banana Pi, Add-On Board mit 1.8" TFT LCD und IR-Sender, FHEM V5.8, CULFW V1.61, div. Homematic Komponenten, Pollin Funksteckdosen, Selbstbau CUL433 MHz, Jeelink Clone, EMT7110, EC3000
Selbstbau CUL868MHz für Wireless M-Bus, FHEMduino mit Logilink Temp.-sensoren und Auriol Wetterstation

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 18630
Antw:Patch 00_CUL.pm/culfw für WMBUS Typ C
« Antwort #7 am: 25 Juni 2018, 11:03:47 »
Zitat
Anbei die richtige Version....
Viel besser, habs eingecheckt.

 

decade-submarginal