Bugproblematik bei 10_CUL_HM.pm

Begonnen von CoolTux, 24 November 2017, 09:44:56

Vorheriges Thema - Nächstes Thema

CoolTux

https://forum.fhem.de/index.php/topic,79858.0.html

Wie kann man hier am besten verfahren? Martin wurde wohl bereits informiert, dennoch tut sich nichts trotz einer Kleinigkeit.
Welche möglichkeiten, ausser Patch bei Martin einreichen, haben die anderen Developer um die User zu unterstützen?



Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

rudolfkoenig

Mir faellt ein:
- Immer wieder auf das Problem hinweisen und warten.
- Alternativmodul schreiben.

Selbst sein Modul zu Patchen ist keine Option.


CoolTux

Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Hauswart

Zitat von: rudolfkoenig am 24 November 2017, 10:11:07
Selbst sein Modul zu Patchen ist keine Option.
Du sprichst hier von Patchen und via SVN verteilen? Gegen Patchen und via Thread verteilen ist in Ordnung? Siehe: https://forum.fhem.de/index.php/topic,79858.msg720176.html#msg720176
1. Installation:
KNX, Tasmota (KNX), Sonos, Unifi

2. Installation:
HM-CFG-USB, Unifi (, SIGNALduino 868, MySensors, SIGNALduino 433)

CoolTux

Zitat von: Hauswart am 24 November 2017, 10:25:05
Du sprichst hier von Patchen und via SVN verteilen? Gegen Patchen und via Thread verteilen ist in Ordnung? Siehe: https://forum.fhem.de/index.php/topic,79858.msg720176.html#msg720176

Meine Meinung. Ich finde es ok. Das Problem ist das ich finde das das (man eine Menge das) File nach Bereinigung des Problemes wieder entfernt werden sollte.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Thorsten Pferdekaemper

Hi,
erst einmal vorweg: Ich bin ein Verfechter des "strengen Code-Ownership". D.h. für jedes Stückchen Coding gibt es einen "Oberboss", der im Endeffekt das Sagen hat.
Aaaaber:
Es kann gut sein, dass derjenige mal krank ist, vom Auto überfahren wird oder ohne Handy im australischen Outback wandert. In dem Fall finde ich es auch nicht ganz richtig, wenn man das ganze Modul neu schreiben müsste. Da sollten wir schon, zumindest bei Sachen, die ganz klar ein Fehler sind (wie hier), auch in "fremdem" Coding einen Fix einbauen können. Es ist mir auch klar, dass jemand anders nicht immer wissen kann, was eigentlich die Intention des betreffenden Codings war, aber ein erfahrener Entwickler wird trotzdem in den meisten Fallen etwas machen können.
Natürlich muss darüber der eigentliche Maintainer immer so gut es geht informiert werden und das sollte auch nicht leichtfertig gemacht werden. Andererseits ist aber auch blöd, wenn man bei einem so wichtigen Modul wie 10_CUL_HM.pm irgendwelche Umgehungslösungen braucht, damit es überhaupt läuft.

Längerfristig könnte ich mir vorstellen, dass jeder Maintainer einen "Vize-Maintainer" ernennt.

Gruß,
    Thorsten 
FUIP

herrmannj

Vorschlag:

Jemand (namentlich) übernimmt die Prozess und macht:

- Info am Martin, PM und Mail
- letzte funktionierende version im SVN wieder herstellen.

- Martin macht das dann heil sobald er die Chance dazu hat.

Wir haben alle ein rl !




betateilchen

Da ich im Homematic-Thread die Lösung ja schon identifiziert habe, kann ich das gerne übernehmen:


  • martin anschreiben
  • funktionierendes Modul mit bugfix des aktuellen Problems als interims-Lösung einchecken, bis martin das selbst korrigiert hat

-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

herrmannj


Thorsten Pferdekaemper

Zitat von: betateilchen am 24 November 2017, 11:36:29
Da ich im Thread die Lösung ja schon identifiziert habe, kann ich das gerne übernehmen:
Ich bin auch dafür.
Gruß,
   Thorsten
FUIP

betateilchen

kann bitte jemand den dortigen Thread https://forum.fhem.de/index.php/topic,79858.0.html schließen?
Da wird inzwischen über Dinge diskutiert, die mit dem Bug nix mehr zu tun haben...
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

CoolTux

#11
Zitat von: herrmannj am 24 November 2017, 11:31:14
Vorschlag:

Jemand (namentlich) übernimmt die Prozess und macht:

- Info am Martin, PM und Mail
- letzte funktionierende version im SVN wieder herstellen.

- Martin macht das dann heil sobald er die Chance dazu hat.

Wir haben alle ein rl !

In diesem Fall wäre ein Roolback etwas suboptimal. Laut dem Comment waren einige wichtige Bugfixes dabei. Daher würde hier wohl ein fix, zu mal es nur um ein Wort geht, besser angebracht.

Ich kann mich an Martin per PN wenden (Mail ist aktuell mir nicht bekannt, kann aber mal schauen) und einen fix im SVN bereit stellen.

Meinungen dazu?


Sorry hat Udo seinen Post übersehen.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

CoolTux

Zitat von: betateilchen am 24 November 2017, 11:39:44
kann bitte jemand den dortigen Thread https://forum.fhem.de/index.php/topic,79858.0.html schließen?
Da wird inzwischen über Dinge diskutiert, die mit dem Bug nix mehr zu tun haben...

Vielleicht erstmal ne kleine Mahnung in Worte einbringen das man sich doch bitte am Thema halten soll.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

herrmannj

Zitat von: betateilchen am 24 November 2017, 11:39:44
kann bitte jemand den dortigen Thread https://forum.fhem.de/index.php/topic,79858.0.html schließen?
Da wird inzwischen über Dinge diskutiert, die mit dem Bug nix mehr zu tun haben...
done

betateilchen

Danke.

Der bugfix ist eingecheckt. Falls Rudi Lust und Zeit findet, könnte er vielleicht heute nochmal ein update generieren, damit den Anwendern schnell geholfen werden kann.
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!