[patch] 98_fheminfo.pm - Erweiterung um Info zu fhem.pl

[betateilchen]

Um diese Informationen über fhem.pl zu erhalten

(http://up.picr.de/24426358so.png)

schlage ich folgenden patch vor:

Code Auswählen Erweitern


Index: 98_fheminfo.pm
===================================================================
--- 98_fheminfo.pm (revision 10791)
+++ 98_fheminfo.pm (working copy)
@@ -75,6 +75,8 @@

   my $branch   = "DEVELOPMENT"; # UNUSED
   my $release  = "5.7";
+  my $fhempl   = fhem('version fhem.pl noheader');
+     $fhempl   =~ s/fhem.pl //;
   my $feature  = $featurelevel ? $featurelevel : $release;
   my $os       = $^O;
   my $arch     = $Config{"archname"};
@@ -115,6 +117,7 @@
   $str  = "Fhem info:\n";
   $str .= sprintf("  Release%*s: %s FeatureLevel: %s\n",
                         2, " ", $release, $feature);
+  $str .= sprintf("  fhem.pl%*s: %s\n",2," ",$fhempl);
   $str .= sprintf("  OS%*s: %s\n",7," ",$os);
   $str .= sprintf("  Arch%*s: %s\n",5," ",$arch);
   $str .= sprintf("  Perl%*s: %s\n",5," ",$perl);


Auf die Statistikfunktionen der fheminfo hat das keine Auswirkungen, es ist ein reines Anzeigefeature.

[rudolfkoenig]

Warum findest du diese Zeile in fheminfo wichtig?

Ich finde die fhem.pl Version alleine nicht so relevant, nur zusammen mit den anderen Modulen hilft sie, und dafuer haben wir version.

[Markus Bloch]

Was helfen würde, wäre die letzte Revision, auf die ausgecheckt bzw. geupdatet wurde. Um das zweifelsfrei zu bestimmen, müsste das durch den update-Befehl ermittelt werden, auf welche Revision exakt geupdatet wurde.

Gruß
Markus

[betateilchen]

Nanu, seit wann werden vorgeschlagene Änderungen nach ihrer Wichtigkeit bewertet? Aber Du hast ja grundsätzlich recht: wichtig ist die Information nicht, das habe ich auch nicht behauptet. Allerdings arbeite ich lieber (und öfter) mit fheminfo als mit version.

Die Ausgabe der uniqueID in der Liste ist auch nicht wichtig - geschweige denn hilfreich - und trotzdem vorhanden.

Wenn wir Deinen begrüßenswerten Ansatz, Änderungen nach ihrer Wichtigkeit zu bewerten, konsequent anwenden, können wir fhem extrem verschlanken.

[betateilchen]

Was helfen würde, wäre die letzte Revision, auf die ausgecheckt bzw. geupdatet wurde.

Das kannst Du nicht zweifelsfrei feststellen, da ich als Anwender immer die Möglichkeit habe, per exclude_from_update bestimmte Module vom Update auszuschließen.

Die aktuelle SVN Version eines updates in die Datei ./FHEM/controls_fhem.txt zu schreiben, sollte nicht das größte Problem sein.

[Markus Bloch]

Die aktuelle SVN Version eines updates in die Datei ./FHEM/controls_fhem.txt zu schreiben, sollte nicht das größte Problem sein.

So hätte ich auch gedacht. Evtl. als Zeile am Anfang

Code Auswählen Erweitern

REV 10798
Dann könnte man das in fheminfo als Overall-Revision anzeigen.

Gruß
Markus

[betateilchen]

Code Auswählen Erweitern

svn info | grep Revision | awk '{ print $2; }'

[rudolfkoenig]

Ab sofort enthaelt controls_fhem.txt in der ersten Zeile die SVN Revision als "REV nr".

[Markus Bloch]

Vielen Dank. Nun ist die Frage, wo sollte man die Info am besten anzeigen? in fheminfo? in version? oder beides?

Was meint ihr?

Gruß
Markus

[betateilchen]

Bau es einfach mal in version ein. Möglichst so, dass man mit "version revision noheader" die Information auch gezielt abfragen kann

[Markus Bloch]

eingecheckt.

Code Auswählen Erweitern

Latest Revision: 10814

File                        Rev   Last Change

fhem.pl                     10769 2016-02-08 12:11:51Z rudolfkoenig
90_at.pm                    10594 2016-01-22 13:27:34Z rudolfkoenig
98_autocreate.pm            10651 2016-01-28 16:18:49Z rudolfkoenig
00_CUL.pm                   10472 2016-01-12 08:34:51Z rudolfkoenig
10_CUL_HM.pm                10757 2016-02-07 19:18:11Z martinp876
No Id found for 99_CustomUtils.pm
98_dewpoint.pm               6757 2014-10-12 18:58:57Z joachim09876
98_dummy.pm                 10530 2016-01-16 19:27:21Z rudolfkoenig
00_FBAHA.pm                 10398 2016-01-07 09:05:47Z rudolfkoenig
10_FBDECT.pm                 9304 2015-09-25 18:33:18Z rudolfkoenig
72_FB_CALLLIST.pm           10716 2016-02-03 19:01:38Z markusbloch
72_FB_CALLMONITOR.pm        10740 2016-02-06 14:45:51Z markusbloch
01_FHEMWEB.pm               10747 2016-02-07 07:39:16Z rudolfkoenig
92_FileLog.pm               10530 2016-01-16 19:27:21Z rudolfkoenig
98_GEOFANCY.pm              10565 2016-01-18 09:23:45Z loredo
98_help.pm                  10809 2016-02-13 09:51:16Z betateilchen
98_HMinfo.pm                10615 2016-01-24 21:09:45Z martinp876
00_HMLAN.pm                 10409 2016-01-08 15:13:59Z martinp876
95_holiday.pm               10530 2016-01-16 19:27:21Z rudolfkoenig
02_HTTPSRV.pm               10733 2016-02-06 09:02:02Z borisneubert
71_LGTV_IP12.pm             10540 2016-01-16 22:09:03Z markusbloch
75_LGTV_RS232.pm             6414 2014-08-17 21:35:34Z markusbloch
91_notify.pm                10694 2016-02-01 06:43:00Z rudolfkoenig
34_panStamp.pm               8803 2015-06-22 19:21:07Z justme1968
73_PRESENCE.pm              10686 2016-01-31 15:46:23Z markusbloch
33_readingsGroup.pm         10532 2016-01-16 19:56:21Z justme1968
No Id found for 74_S0COUNTER.pm
98_structure.pm             10530 2016-01-16 19:27:21Z rudolfkoenig
99_SUNRISE_EL.pm            10569 2016-01-19 06:30:28Z rudolfkoenig
98_SVG.pm                   10792 2016-02-10 20:20:49Z rudolfkoenig
34_SWAP.pm                  10297 2015-12-29 19:04:09Z justme1968
35_SWAP_0000002200000003.pm 10308 2015-12-30 11:42:17Z justme1968
98_telnet.pm                10551 2016-01-17 11:03:44Z rudolfkoenig
59_Twilight.pm               8743 2015-06-14 12:14:57Z dietmar63
99_Utils.pm                 10570 2016-01-19 06:39:23Z rudolfkoenig
98_version.pm               10658 2016-01-29 19:43:10Z markusbloch
91_watchdog.pm              10789 2016-02-10 18:24:20Z rudolfkoenig
55_weco.pm                  10558 2016-01-17 14:17:58Z betateilchen
71_YAMAHA_AVR.pm            10812 2016-02-13 11:18:05Z markusbloch
71_YAMAHA_BD.pm             10614 2016-01-24 19:50:21Z markusbloch

Blocking.pm                 10343 2016-01-03 10:55:45Z rudolfkoenig
Color.pm                    10755 2016-02-07 18:17:53Z justme1968
DevIo.pm                     8954 2015-07-13 16:01:48Z rudolfkoenig
FritzBoxUtils.pm             6574 2014-09-19 17:32:48Z rudolfkoenig
HMConfig.pm                 10758 2016-02-07 19:36:23Z martinp876
HttpUtils.pm                10810 2016-02-13 10:30:16Z rudolfkoenig
RTypes.pm                   10476 2016-01-12 21:03:33Z borisneubert
SetExtensions.pm             9413 2015-10-09 13:13:11Z rudolfkoenig
TcpServerUtils.pm           10346 2016-01-03 12:34:27Z rudolfkoenig
TimeSeries.pm                9013 2015-08-02 10:09:55Z borisneubert


@betateilchen: Entweder via "version revision noheader" oder version_getRevFromControls();

Gruß
Markus

[betateilchen]

@betateilchen: Entweder via "version revision noheader" oder version_getRevFromControls();

Vorsicht Die zweite Variante funktioniert vermutlich nur, wenn das Modul bereits geladen ist, ansonsten gibts nen fhem-Absturz.

Die Variante

Code Auswählen Erweitern

fhem("version revision noheader");

funktioniert immer. Gegebenenfalls wird das Modul automatisch geladen.

[betateilchen]

Ich hätte da ja noch einen Wunsch für 98_version.pm, aber den müssen wir besser per email diskutieren

[Markus Bloch]

Vorsicht Die zweite Variante funktioniert vermutlich nur, wenn das Modul bereits geladen ist, ansonsten gibts nen fhem-Absturz.

Ja das stimmt.

Ich hätte da ja noch einen Wunsch für 98_version.pm, aber den müssen wir besser per email diskutieren

gerne.