PATCH: DbLog: DbLogInclude minIntervall verhindert Logging

[gero]

Hallo,

mir ist aufgefallen, dass bei DbLogInclude keine Readings geloggt werden, bei denen ein MinIntervall angegeben ist.
Anbei ein kleiner Patch, der das Problem behebt.

Gruß,
Gero

[CoolTux]

Frage ist ob das wirklich ein Problem ist, oder nicht doch vom Entwickler so gewollt.

[gero]

Die Command Reference ist meiner Meinung nach ziemlich eindeutig:

Code Auswählen Erweitern


set <device> DbLogInclude regex:MinInterval [regex:MinInterval] ...


Falls es tatsächlich Absicht sein sollte, müßte die Command Reference geändert werden.
Außerdem ist es im Code vorgesehen. Leider mit einem kleinen Bug.


Gruß,
Gero

[CoolTux]

Hast Du den Entwickler schon angeschrieben und Dein Patch eingereicht bei ihm? Wenn nicht bitte machen.


Danke Dir
Grüße

[gero]

Der Maintainer ist normalerweise tobiasfaust. Er hat aber wohl keine Zeit mehr sich darum zu kümmern. Daher kümmert sich rapster um das Modul.
Aber du hast recht, der Beitrag ist evtl. im falschen Unterform gelandet. Falls rapster ihn nicht sieht, werde ich mich nochmal bei ihm melden.

[Benni]

Hallo gero,

den Bug dürfte wohl ich "damals" bei der Einführung von DBLogInclude in meinem Patch mit eingebaut haben.

Das war natürlich nicht so gewollt, sondern so, wie es in der Commandref auch beschrieben ist.

Leider komme ich z.Z. nur zum allernötigsten, deshalb konnte ich es mir noch nicht genauer anschauen.

Danke für die Korrektur! 

Gruß Benni.

[Amenophis86]

Eine Frage, ist das Problem inzwischen behoben, dass man minIntervall wieder einstellen kann?

[mdorenka]

bei mir noch nicht (gestern aus nightly installiert) noch nicht - daher musst du wohl patchen

[Tobias]

bin gerade im Endstress beim Hausbau .... kannst du bitte mal rapster anpingen??

[Tobias]

Nachfrage: hat Benni oder Rapster ihn schon eingebaut?

[Benni]

Hallo Tobi,

Ich habe da bisher nichts gemacht.

Gruß Benni.

[gero]

Ich hatte rapster per PN angeschrieben. Bisher ist aber noch nichts passiert.

Gruß,
Gero

[N23]

Push.... ist echt nervig jedes mal den Fehler patchen zu müssen...

[rapster]

Habs eingecheckt

Gruß
Claudiu

[N23]

Danke