Autor Thema: commandref_join.pl Warnungen  (Gelesen 387 mal)

Offline RichardCZ

  • Tester
  • Sr. Member
  • ****
  • Beiträge: 558
  • HoBo: zwischen Weltherrschaft und Intensivstation
    • Experimenteller FHEM Fork
commandref_join.pl Warnungen
« am: 18 April 2020, 21:38:06 »
Jedesmal wenn ich bei mir einen "Make update" mache, kommen beim Lauf von commandref_join.pl  diese Warnungen:

*** EN SISPM line 570: div with attributes (apart from class) is not allowed
*** EN FHEM/70_SISPM.pm: Unbalanced div (1, last line ok: 569)
*** EN WifiLight line 4892: table with attributes (apart from class) is not allowed
*** EN FHEM/32_WifiLight.pm: Unbalanced table (1, last line ok: 4891)
*** DE WifiLight line 5537: table with attributes (apart from class) is not allowed
*** DE FHEM/32_WifiLight.pm: Unbalanced table (1, last line ok: 5536)


ist das nur bei mir oder sind die Pappenheimer bekannt?

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 25321
Antw:commandref_join.pl Warnungen
« Antwort #1 am: 18 April 2020, 21:40:26 »
Das ist bei allen. Zu mindest bei allen den die Warnungen nicht ausschalten. Bei einem normalen Update beim User wird das generieren mit nowarn aufgerufen.
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://paypal.me/pools/c/8gULisr9BT
My FHEM Git: https://git.cooltux.net/FHEM/
Mein Dokuwiki:
https://www.cooltux.net

Online Wzut

  • Developer
  • Hero Member
  • ****
  • Beiträge: 3496
Antw:commandref_join.pl Warnungen
« Antwort #2 am: 19 April 2020, 08:34:23 »
gibt es vllt. auch die Möglichkeit diese Prüfung auch svn seitig bei einem commit laufen zu lassen und ggf. wie bei anderen Fehlern dann einfach abzulehnen ?
Maintainer der Module: MAX, MPD, UbiquitiMP, UbiquitiOut, SIP, BEOK, readingsWatcher

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 25321
Antw:commandref_join.pl Warnungen
« Antwort #3 am: 19 April 2020, 08:36:16 »
Ich weiß es nicht, bin aber der Meinung daß dies seit kurzem so gemacht wird. Bin mir aber echt unsicher.
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://paypal.me/pools/c/8gULisr9BT
My FHEM Git: https://git.cooltux.net/FHEM/
Mein Dokuwiki:
https://www.cooltux.net

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 22327
Antw:commandref_join.pl Warnungen
« Antwort #4 am: 19 April 2020, 17:35:55 »
Diese Pruefung habe ich am 2019-12-06 eingebaut, sie ist auch Teil der SVN pre-commit, die beiden Dateien wurden aber seitdem nicht angefasst.
Die Diskussion dazu ist hier dokumentiert.
Das ironische an der Sache ist, dass die drei angemeckerten Probleme eigentlich keine sind, da die "Probleme" auskommentiert sind.
Da ich aber im Hook nicht mit HTML-Kommentarerkennung anfangen will, habe ich die beiden Module geaendert.
Gefällt mir Gefällt mir x 2 Liste anzeigen

Offline herrmannj

  • Global Moderator
  • Hero Member
  • ****
  • Beiträge: 5605
Antw:commandref_join.pl Warnungen
« Antwort #5 am: 19 April 2020, 17:40:12 »
Hat also richtig viel gebracht der thread :) Danke @Rudi
smartVisu mit fronthem, einiges an HM, RFXTRX, Oregon, CUL, Homeeasy, ganz viele LED + Diverse

 

decade-submarginal