[PATCH] - Formatierter Output für version-Befehl

[Markus Bloch]

Hallo Rudi,

ich habe mal den version-Befehl etwas verbessert und eine minimale Formatierung des Outputs durchgeführt, damit man es besser lesen kann (besonders bei großen Installationen). Die Funktionalität bleibt gleich (optionales Suchwort), nur der Output ist etwas besser:

Code Auswählen Erweitern

File                        Rev  Last Change

fhem.pl                     9307 2015-09-25 18:44:20Z rudolfkoenig
90_at.pm                    8326 2015-03-29 13:30:57Z rudolfkoenig
98_autocreate.pm            9216 2015-09-09 13:10:01Z rudolfkoenig
00_CUL.pm                   9002 2015-07-29 05:46:10Z rudolfkoenig
10_CUL_HM.pm                9325 2015-09-27 18:00:00Z martinp876
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                 9305 2015-09-25 18:34:16Z rudolfkoenig
10_FBDECT.pm                9304 2015-09-25 18:33:18Z rudolfkoenig
72_FB_CALLLIST.pm           9267 2015-09-17 15:38:04Z markusbloch
72_FB_CALLMONITOR.pm        8791 2015-06-21 11:23:23Z markusbloch
01_FHEMWEB.pm               9321 2015-09-27 12:44:35Z rudolfkoenig
92_FileLog.pm               9107 2015-08-22 13:16:05Z rudolfkoenig
98_GEOFANCY.pm              9058 2015-08-10 18:32:31Z loredo
00_HMLAN.pm                 9325 2015-09-27 18:00:00Z martinp876
95_holiday.pm               8723 2015-06-10 09:09:01Z rudolfkoenig
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              9111 2015-08-22 16:46:24Z markusbloch
33_readingsGroup.pm         8980 2015-07-26 08:03:43Z justme1968
98_structure.pm             8330 2015-03-29 16:14:59Z rudolfkoenig
99_SUNRISE_EL.pm            6765 2014-10-14 18:24:29Z rudolfkoenig
98_SVG.pm                   9319 2015-09-27 10:10:19Z rudolfkoenig
34_SWAP.pm                  8401 2015-04-08 16:23:25Z justme1968
35_SWAP_0000002200000003.pm 6546 2014-09-12 17:48:59Z justme1968
98_telnet.pm                8952 2015-07-13 12:30:26Z rudolfkoenig
59_Twilight.pm              8743 2015-06-14 12:14:57Z dietmar63
99_Utils.pm                 7914 2015-02-08 11:14:10Z rudolfkoenig
91_watchdog.pm              9185 2015-08-31 18:46:10Z 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            9093 2015-08-18 21:50:20Z markusbloch
71_YAMAHA_BD.pm             8114 2015-02-28 23:04:01Z markusbloch

ein "version fhem" gibt aus:

Code Auswählen Erweitern

File    Rev  Last Change

fhem.pl 9307 2015-09-25 18:44:20Z rudolfkoenig

ein "version SWAP" gibt aus:

Code Auswählen Erweitern

File                        Rev  Last Change

34_SWAP.pm                  8401 2015-04-08 16:23:25Z justme1968
35_SWAP_0000002200000003.pm 6546 2014-09-12 17:48:59Z justme1968

Finde ich persönlich deutlich übersichtlicher.

Desweiteren ist die Sortierung nun Case-Insensitiv, so das auch Module mit kleinem Anfangsbuchstaben richtig einsortiert sind.

Viele Grüße

Markus


edit: Diff angepasst, so dass SVN die Regexp nicht als $Id$ Keyword erkennt.

[rudolfkoenig]

Nach dem Patchen bekomme ich:

Scalar found where operator expected at fhem.pl line 2655, near "return sprintf("%-".$max."s %s","File","Rev  Last Change\n\n").join("\n", grep(($param ? $_ =~ /$param"
  (Might be a runaway multi-line // string starting on line 2653)
    (Do you need to predeclare return?)
syntax error at fhem.pl line 2655, near "return sprintf("%-".$max."s %s","File","Rev  Last Change\n\n").join("\n", grep(($param ? $_ =~ /$param"
syntax error at fhem.pl line 2663, near "sub
HandleTimeout()
"
BEGIN not safe after errors--compilation aborted at fhem.pl line 2683.

Und ich haette gerne den Patch mit 80-Zeichen/Zeile.

[Markus Bloch]

Hab ordentlich geklammert, damit es keine Verwechslung der Operator-Prioritäten gibt. Ebenfalls auf 80-Zeichen/Zeile geändert.

Bitte nicht falsch verstehen, aber wieso beharrst du so auf die 80-Zeichen/Zeile?

Viele Grüße

Markus

[rudolfkoenig]

Habs eingecheckt.

80 Zeichen/Zeile ist ein Marotte von mir: ich arbeite mit 2-3 Fenster nebeneinander, und ich will alles auf einmal sehen. Wenn es automatisch umgebrochen wird, dann ist es deutlich schlechter lesbar.

[betateilchen]

Bei der durchgeführten Änderung gibt es noch Verbesserungspotenzial...

Code Auswählen Erweitern


File                 Rev  Last Change

fhem.pl              9356 2015-10-03 13:41:25Z rudolfkoenig
configDB.pm          7949 2015-02-12 22:04:09Z betateilchen
90_at.pm             8326 2015-03-29 13:30:57Z rudolfkoenig
99_myUtils.pm        68 2014-04-25 12:57:10Z udo


Vielleicht sollte man angesichts der bevorstehenden 10.000 Marke bei den svn commits die Ausgabelänge der Versionsnummer gleich auf 5 Stellen festlegen und die Versionsnummer dann bitte rechtsbündig innerhalb dieser 5 Stellen ausgeben.

[Markus Bloch]

Stimmt, daran habe ich garnicht gedacht.

Anbei ein Patch um die genannten Vorschläge von betateilchen umzusetzen.

Output (Auszug):

Code Auswählen Erweitern

File                        Rev   Last Change

fhem.pl                      9356 2015-10-03 13:41:25Z rudolfkoenig
90_at.pm                     8326 2015-03-29 13:30:57Z rudolfkoenig
98_autocreate.pm             9216 2015-09-09 13:10:01Z rudolfkoenig

00_HMLAN.pm                  9325 2015-09-27 18:00:00Z martinp876
95_holiday.pm                8723 2015-06-10 09:09:01Z rudolfkoenig
82_LGTV2012.pm                  2 2014-03-17 11:05:19Z juliantatsch
91_notify.pm                 8953 2015-07-13 15:13:06Z rudolfkoenig
34_panStamp.pm               8803 2015-06-22 19:21:07Z justme1968

71_YAMAHA_AVR.pm            93655 2015-10-04 11:26:22Z markusbloch
71_YAMAHA_BD.pm              8114 2015-02-28 23:04:01Z markusbloch

Viele Grüße

Markus

[rudolfkoenig]

eingecheckt.

[betateilchen]

Danke!

80 Zeichen/Zeile ist ein Marotte von mir:

80 Zeichen/Zeile finde ich auch super.

[betateilchen]

(aus gegebenem Anlass)

Könnte man die Fähigkeit, bis 80 zählen zu können, nicht als Bedingung dafür machen, dass jemand Dateien einchecken darf? *grummel*
Oder einen pre-commit-hook schreiben 

Und die 80-Zeichen Grenze würde ich mir auch für CHANGED wünschen.

[rudolfkoenig]

Kommt noch
Im CHANGED steht es sogar oben drin. Und ich formatiere es immer wieder neu.

Dieser Jugendlichen, die nicht mal wissen, was ein VT100 Terminal ist.
Und ich meine nicht die Emulation

[betateilchen]

Dieser Jugendlichen, die nicht mal wissen, was ein VT100 Terminal ist.

genau... die sind die Schlimmsten 

Ich komme übrigens aus einer Entwicklungswelt, in der eine Zeilenlänge von maximal 72 Zeichen das Maß aller Dinge war.

[justme1968]

dann sollten wir auch gleich noch die spalten 1-7 verbieten damit wir den FHEM code auch schön fortran kompatibel auf lochkarten bekommen.

sogar das vt100 könnte schon 132 zeichen pro zeile. 

[betateilchen]

dann sollten wir auch gleich noch die spalten 1-7 verbieten ... lochkarten ...

was denkst Du, wo die bereits erwähnten 72 Zeichen herkommen...

[justme1968]

ich weiss es