[PATCH] - fhem.pl Aufnahme von Hilfsmodulen (HttpUtils, ...) in version output

[Markus Bloch]

Ja natürlich, allerdings funktioniert sie so nicht, da ja CommandVersion so noch aus fhem.pl genommen wird.

Gruß
Markus

[betateilchen]

Ganz so doof bin ich dann doch nicht das mit der fhem.pl war mir schon klar, das hatte ich vorab schon geändert.

Du solltest aber bitte auch Module auflisten, die keine Id besitzen und diese Module nicht einfach unterschlagen.

(http://up.picr.de/24248994mg.png)

Da ist 98_version.pm nicht aufgeführt...

[Markus Bloch]

Ok. Anbei die neue Version. Ausgabe ist bei mir:

Code Auswählen Erweitern

File                        Rev   Last Change

fhem.pl                     10397 2016-01-07 08:36:49Z rudolfkoenig
90_at.pm                    10048 2015-11-29 14:51:40Z rudolfkoenig
98_autocreate.pm            10165 2015-12-13 11:14:15Z rudolfkoenig
99_backup.pm: NO ID FOUND
00_CUL.pm                   10146 2015-12-10 10:17:42Z rudolfkoenig
99_CULflash.pm: NO ID FOUND
10_CUL_HM.pm                10411 2016-01-08 15:18:17Z martinp876
99_CustomUtils.pm: NO ID FOUND
98_dewpoint.pm               6757 2014-10-12 18:58:57Z joachim09876
98_dummy.pm                  8809 2015-06-23 18:02:33Z 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           10317 2015-12-31 12:12:25Z markusbloch
72_FB_CALLMONITOR.pm        10349 2016-01-03 14:20:57Z markusbloch
01_FHEMWEB.pm               10398 2016-01-07 09:05:47Z rudolfkoenig
92_FileLog.pm               10365 2016-01-05 10:57:56Z rudolfkoenig
98_GEOFANCY.pm              10325 2016-01-02 00:28:34Z loredo
00_HMLAN.pm                 10409 2016-01-08 15:13:59Z martinp876
95_holiday.pm               10004 2015-11-25 12:58:12Z rudolfkoenig
99_JsonList.pm: NO ID FOUND
82_LGTV2012.pm                  2 2014-03-17 11:05:19Z juliantatsch
75_LGTV_RS232.pm             6414 2014-08-17 21:35:34Z markusbloch
91_notify.pm                 8953 2015-07-13 15:13:06Z rudolfkoenig
34_panStamp.pm               8803 2015-06-22 19:21:07Z justme1968
73_PRESENCE.pm              10295 2015-12-29 18:47:24Z markusbloch
33_readingsGroup.pm         10157 2015-12-12 11:35:11Z justme1968
74_S0COUNTER.pm: NO ID FOUND
98_structure.pm             10382 2016-01-06 16:21:13Z rudolfkoenig
99_SUNRISE_EL.pm             9831 2015-11-08 19:13:09Z rudolfkoenig
98_SVG.pm                    9577 2015-10-21 17:45:02Z 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                10398 2016-01-07 09:05:47Z rudolfkoenig
59_Twilight.pm               8743 2015-06-14 12:14:57Z dietmar63
98_update.pm                10247 2015-12-23 19:53:31Z rudolfkoenig
99_Utils.pm                  7914 2015-02-08 11:14:10Z rudolfkoenig
98_version.pm               10247 2015-12-23 19:53:31Z markusbloch
91_watchdog.pm              10314 2015-12-31 08:44:32Z rudolfkoenig
55_weco.pm                   5517 2014-04-13 00:37:45Z betateilchen
99_XmlList.pm                1840 2012-09-12 13:52:08Z rudolfkoenig
71_YAMAHA_AVR.pm            10355 2016-01-03 22:00:41Z markusbloch
71_YAMAHA_BD.pm             10036 2015-11-28 16:02:57Z markusbloch

Blocking.pm                 10343 2016-01-03 10:55:45Z rudolfkoenig
Color.pm                     9280 2015-09-20 08:07:41Z justme1968
DevIo.pm                     8954 2015-07-13 16:01:48Z rudolfkoenig
FritzBoxUtils.pm             6574 2014-09-19 17:32:48Z rudolfkoenig
HMConfig.pm                 10410 2016-01-08 15:15:50Z martinp876
HttpUtils.pm                10137 2015-12-09 09:01:41Z rudolfkoenig
RTypes.pm: NO ID FOUND
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

[Markus Bloch]

Da ist 98_version.pm nicht aufgeführt...

Ist ja auch nicht geladen

[betateilchen]

Ist ja auch nicht geladen

Witzbold. Spätestens wenn ich den Befehl zum zweiten Mal ausführe, ist die Datei sehr wohl vollständig geladen!

[betateilchen]

Ok. Anbei die neue Version. Ausgabe ist bei mir:
99_backup.pm: NO ID FOUND

Wieso konntest/wolltest Du dich nicht an das Ausgabeformat halten, das in meinem Beispiel stand (No Id am Anfang stehend), und das schon seit zwei Jahren so existiert?

[Markus Bloch]

Weil dieses Format in fhem.pl so nicht existiert. Ich habe mich an die bestehende CommandVersion Funktion gehalten was Output angeht. Da ist diese Meldung auch bisher so nicht enthalten. Ich nehme an, die kommt aus  cfgDB_Fileversion().

Kann ich aber gerne so übernehmen.

[betateilchen]

Ich habe mich an die bestehende CommandVersion Funktion gehalten was Output angeht. Da ist diese Meldung auch bisher so nicht enthalten.

Die Meldung wurde bisher sehr wohl von CommandVersion in der fhem.pl so ausgegeben, auch wenn sie nicht in dieser Funktion selbst generiert wurde.

Ich nehme an, die kommt aus  cfgDB_Fileversion().

Exakt so ist es, wenn eine Moduldatei aus der configDB geladen wird anstatt aus dem Dateisystem und diese Moduldatei keine Id enthält.


Edit: wobei man auf die Pfadangabe durchaus verzichten kann, in configDB ist die aus technischen Gründen vorhanden. Kann ich bei Gelegenheit irgendwann anpassen.

[betateilchen]

Die Deklaration

Code Auswählen Erweitern

sub cfgDB_svnId;

in der fhem.pl kann entfallen, wenn die CommandVersion() ausgelagert wird.

[Markus Bloch]

So in Ordnung?

Code Auswählen Erweitern

File                        Rev   Last Change

fhem.pl                     10397 2016-01-07 08:36:49Z rudolfkoenig
90_at.pm                    10048 2015-11-29 14:51:40Z rudolfkoenig
98_autocreate.pm            10165 2015-12-13 11:14:15Z rudolfkoenig
No Id found for 99_backup.pm
00_CUL.pm                   10146 2015-12-10 10:17:42Z rudolfkoenig
No Id found for 99_CULflash.pm
10_CUL_HM.pm                10411 2016-01-08 15:18:17Z martinp876
No Id found for 99_CustomUtils.pm
98_dewpoint.pm               6757 2014-10-12 18:58:57Z joachim09876
98_dummy.pm                  8809 2015-06-23 18:02:33Z 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           10317 2015-12-31 12:12:25Z markusbloch
72_FB_CALLMONITOR.pm        10349 2016-01-03 14:20:57Z markusbloch
01_FHEMWEB.pm               10398 2016-01-07 09:05:47Z rudolfkoenig
92_FileLog.pm               10365 2016-01-05 10:57:56Z rudolfkoenig
98_GEOFANCY.pm              10325 2016-01-02 00:28:34Z loredo
00_HMLAN.pm                 10409 2016-01-08 15:13:59Z martinp876
95_holiday.pm               10004 2015-11-25 12:58:12Z rudolfkoenig
No Id found for 99_JsonList.pm
82_LGTV2012.pm                  2 2014-03-17 11:05:19Z juliantatsch
75_LGTV_RS232.pm             6414 2014-08-17 21:35:34Z markusbloch
91_notify.pm                 8953 2015-07-13 15:13:06Z rudolfkoenig
34_panStamp.pm               8803 2015-06-22 19:21:07Z justme1968
73_PRESENCE.pm              10295 2015-12-29 18:47:24Z markusbloch
33_readingsGroup.pm         10157 2015-12-12 11:35:11Z justme1968
No Id found for 74_S0COUNTER.pm
98_structure.pm             10382 2016-01-06 16:21:13Z rudolfkoenig
99_SUNRISE_EL.pm             9831 2015-11-08 19:13:09Z rudolfkoenig
98_SVG.pm                    9577 2015-10-21 17:45:02Z 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                10398 2016-01-07 09:05:47Z rudolfkoenig
59_Twilight.pm               8743 2015-06-14 12:14:57Z dietmar63
98_update.pm                10247 2015-12-23 19:53:31Z rudolfkoenig
99_Utils.pm                  7914 2015-02-08 11:14:10Z rudolfkoenig
98_version.pm               10247 2015-12-23 19:53:31Z markusbloch
91_watchdog.pm              10314 2015-12-31 08:44:32Z rudolfkoenig
55_weco.pm                   5517 2014-04-13 00:37:45Z betateilchen
99_XmlList.pm                1840 2012-09-12 13:52:08Z rudolfkoenig
71_YAMAHA_AVR.pm            10355 2016-01-03 22:00:41Z markusbloch
71_YAMAHA_BD.pm             10036 2015-11-28 16:02:57Z markusbloch

Blocking.pm                 10343 2016-01-03 10:55:45Z rudolfkoenig
Color.pm                     9280 2015-09-20 08:07:41Z justme1968
DevIo.pm                     8954 2015-07-13 16:01:48Z rudolfkoenig
FritzBoxUtils.pm             6574 2014-09-19 17:32:48Z rudolfkoenig
HMConfig.pm                 10410 2016-01-08 15:15:50Z martinp876
HttpUtils.pm                10137 2015-12-09 09:01:41Z rudolfkoenig
No Id found for RTypes.pm
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

Bei mir wird 98_version.pm sauber angezeigt. Ich weis du verwendest configDb. Bitte führ mal {Dumper(\%INC)} in FHEM aus und schau, was dort zu 98_version.pm steht.

[betateilchen]

Bei mir wird 98_version.pm sauber angezeigt. Ich weis du verwendest configDb. Bitte führ mal {Dumper(\%INC)} in FHEM aus und schau, was dort zu 98_version.pm steht.

98_version.pm wird inzwischen auch angezeigt.

Aber soll das Ergebnis wirklich so aussehen?

Code Auswählen Erweitern


File              Rev   Last Change

fhem.pl           10397 2016-01-07 08:36:49Z rudolfkoenig
No Id found for ./FHEM/99_myUtils.pm
90_at.pm          10421 2016-01-09 07:54:37Z rudolfkoenig
57_Calendar.pm        0 2016-01-03  borisneubert
98_dummy.pm        8809 2015-06-23 18:02:33Z rudolfkoenig
01_FHEMWEB.pm     10398 2016-01-07 09:05:47Z rudolfkoenig
92_FileLog.pm     10365 2016-01-05 10:57:56Z rudolfkoenig
55_GDS.pm         10388 2016-01-06 17:17:55Z betateilchen
91_notify.pm       8953 2015-07-13 15:13:06Z rudolfkoenig
99_SUNRISE_EL.pm   9831 2015-11-08 19:13:09Z rudolfkoenig
98_telnet.pm      10423 2016-01-09 08:35:53Z rudolfkoenig
99_Utils.pm        7914 2015-02-08 11:14:10Z rudolfkoenig
98_version.pm     10247 2015-12-23 19:53:31Z markusbloch
59_Weather.pm     10403 2016-01-07 19:44:04Z borisneubert

No Id found for AutoLoader.pm
No Id found for autosplit.ix
No Id found for base.pm
No Id found for Base.pm
No Id found for Base64.pm
No Id found for Basename.pm
Blocking.pm       10343 2016-01-03 10:55:45Z rudolfkoenig
No Id found for bytes.pm
No Id found for Carp.pm
No Id found for Check.pm
No Id found for Cmd.pm
No Id found for Common.pm
No Id found for Complex.pm
No Id found for Conditional.pm
No Id found for Config.pm
configDB.pm       10345 2016-01-03 11:44:41Z betateilchen
No Id found for Config_git.pl
No Id found for Config_heavy.pl
No Id found for constant.pm
No Id found for Constants.pm
No Id found for Cwd.pm
No Id found for Date.pm
No Id found for DBI.pm
No Id found for Deflate.pm
No Id found for deprecate.pm
No Id found for Diff.pm
No Id found for DirHandle.pm
No Id found for Dumper.pm
No Id found for DynaLoader.pm
No Id found for Errno.pm
No Id found for Exporter.pm
No Id found for Extra.pm
No Id found for Extract.pm
No Id found for Fcntl.pm
No Id found for feature.pm
No Id found for File.pm
No Id found for FileHandle.pm
No Id found for Find.pm
No Id found for FTP.pm
GDSweblink.pm      9432 2015-10-11 10:38:18Z betateilchen
No Id found for Glob.pm
No Id found for GlobMapper.pm
No Id found for Gunzip.pm
No Id found for Gzip.pm
No Id found for Hash.pm
No Id found for Heavy.pm
No Id found for HiRes.pm
HttpUtils.pm      10137 2015-12-09 09:01:41Z rudolfkoenig
No Id found for if.pm
No Id found for INET.pm
No Id found for Inflate.pm
No Id found for integer.pm
No Id found for IO.pm
No Id found for IP.pm
No Id found for Kid.pm
No Id found for Liblist.pm
No Id found for libnet.cfg
No Id found for Load.pm
No Id found for Local.pm
No Id found for locale.pm
No Id found for MakeMaker.pm
No Id found for MD5.pm
No Id found for Metadata.pm
No Id found for MM.pm
No Id found for MM_Any.pm
No Id found for MM_Unix.pm
No Id found for MY.pm
No Id found for Open3.pm
No Id found for overload.pm
No Id found for overloading.pm
No Id found for ParseWords.pm
No Id found for Path.pm
No Id found for POSIX.pm
No Id found for PublicSuffix.pm
No Id found for randomize.al
No Id found for RawDeflate.pm
No Id found for RawInflate.pm
No Id found for regex.pm
No Id found for register.pm
No Id found for RTypes.pm
No Id found for Seekable.pm
No Id found for Select.pm
No Id found for SelectSaver.pm
No Id found for Simple.pm
No Id found for Socket.pm
No Id found for Spec.pm
No Id found for SQLite.pm
No Id found for SSL.pm
SSLeay.pm           422 2014-07-14 10:09:13Z mikem-guest
No Id found for strict.pm
No Id found for Symbol.pm
TcpServerUtils.pm 10346 2016-01-03 12:34:27Z rudolfkoenig
No Id found for Trig.pm
No Id found for Unix.pm
No Id found for UNIX.pm
No Id found for utf8.pm
No Id found for Util.pm
No Id found for vars.pm
No Id found for version.pm
No Id found for warnings.pm
No Id found for XSLoader.pm
No Id found for Zlib.pm
No Id found for _idna.pm
No Id found for _punycode.pm

[rudolfkoenig]

@Markus: kannst du das bitte einchecken, und dich als Maintainer eintragen?
Ich moechte CommandVersion aus fhem.pl erst danach entfernen.

[Markus Bloch]

@betateilchen: Gib mir bitte mal den Dumper-Output zu %INC:

@rudi: Um 98_version.pm einchecken zu können, müssten folgende Änderungen in fhem.pl gemacht werden. Mit deinem OK würde ich das übernehmen:

Code Auswählen Erweitern

Index: fhem.pl
===================================================================
--- fhem.pl     (revision 10449)
+++ fhem.pl     (working copy)

@@ -228,12 +226,15 @@
use vars qw(@authorize);        # List of authorization devices
use vars qw(@authenticate);     # List of authentication devices
use vars qw($auth_refresh);
+use vars qw($cvsid);            # necessary to access $cvsid in 98_version.pm

+$cvsid = '$Id$';
+
my $AttrList = "verbose:0,1,2,3,4,5 room group comment:textField-long alias ".
                 "eventMap userReadings:textField-long";
my $currcfgfile="";             # current config/include file
my $currlogfile;                # logfile, without wildcards
-my $cvsid = '$Id$';
+
my $duplidx=0;                  # helper for the above pool
my $evalSpecials;               # Used by EvalSpecials->AnalyzeCommand
my $intAtCnt=0;


[rudolfkoenig]

Ich werde fhem.pl selbst aendern, aber erst, nachdem du 98_version.pm eingecheckt hast.
Sonst: Ich hoffe, es ist nicht dein ernst, Version in allen Perl-Dateien zu suchen. Bitte nur die, die mit FHEM was zu tun haben.

[betateilchen]

Sonst: Ich hoffe, es ist nicht dein ernst, Version in allen Perl-Dateien zu suchen.

Das passiert aktuell tatsächlich...