FHEM Forum

FHEM - Entwicklung => FHEM Development => Thema gestartet von: betateilchen am 23 April 2014, 21:22:49

Titel: An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 23 April 2014, 21:22:49
Es gibt eine ganze Reihe von Modulen, die keine $Id haben - bitte kontrolliert Eure Module daraufhin, damit die Versionsinformationen auch vom Befehl "version" ausgegeben werden kann.

Danke!
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: justme1968 am 23 April 2014, 21:30:09
übrigens: damit das geht ist

- irgendwo im modul (am besten am anfang) ein kommetar der art# $Id:$ anzulegen

- die property svn:keywords für das modul file in subversion entsprechen zu setzen:svn propset svn:keywords "Id" <meinModul.pm>

- das file einzuchecken

- die versions information wird automatisch ins file kopiert. man muss das nicht von hand machen.

eigentlich kann man subversion so konfigurieren das das automatisch für alle files gemacht wird. ich habe aber keine ahnung ob man das auch bei der sourceforge installation projektweise machen kann.
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 23 April 2014, 21:37:03
Folgende Dateien in FHEM/ (und darunter) haben keine $Id:

Das executable-Flag muss übrigens nie gesetzt werden.


FHEM/00_FBAHA.pm|(svn:executable 1 *)
FHEM/00_NetzerI2C.pm|()
FHEM/00_OWX_ASYNC.pm|()
FHEM/00_RPII2C.pm|()
FHEM/00_THZ.pm|()
FHEM/10_FBDECT.pm|(svn:executable 1 *)
FHEM/10_FRM.pm|(svn:executable 1 *)
FHEM/10_RESIDENTS.pm|()
FHEM/10_UNIRoll.pm|()
FHEM/16_STACKABLE_CC.pm|(svn:executable 1 *)
FHEM/19_Revolt.pm|()
FHEM/20_FRM_AD.pm|(svn:executable 1 *)
FHEM/20_FRM_I2C.pm|(svn:executable 1 *)
FHEM/20_FRM_IN.pm|(svn:executable 1 *)
FHEM/20_FRM_LCD.pm|(svn:executable 1 *)
FHEM/20_FRM_OUT.pm|(svn:executable 1 *)
FHEM/20_FRM_PWM.pm|(svn:executable 1 *)
FHEM/20_FRM_RGB.pm|()
FHEM/20_FRM_ROTENC.pm|(svn:executable 1 *)
FHEM/20_FRM_SERVO.pm|(svn:executable 1 *)
FHEM/20_FRM_STEPPER.pm|(svn:executable 1 *)
FHEM/20_GUEST.pm|()
FHEM/20_ROOMMATE.pm|()
FHEM/23_LUXTRONIK2.pm|()
FHEM/51_I2C_BMP180.pm|()
FHEM/51_Netzer.pm|()
FHEM/51_RPI_GPIO.pm|()
FHEM/52_I2C_DS1307.pm|()
FHEM/52_I2C_LCD.pm|()
FHEM/52_I2C_PCA9532.pm|()
FHEM/52_I2C_PCF8574.pm|()
FHEM/52_I2C_SHT21.pm|()
FHEM/70_ENIGMA2.pm|()
FHEM/70_JSONMETER.pm|()
FHEM/70_Jabber.pm|()
FHEM/70_ONKYO_AVR.pm|()
FHEM/70_PHTV.pm|()
FHEM/70_Pushover.pm|()
FHEM/70_XBMC.pm|()
FHEM/73_MPD.pm|()
FHEM/88_LINDY_HDMI_SWITCH.pm|()
FHEM/91_eventTypes.pm|(svn:executable 1 *)
FHEM/95_Dashboard.pm|()
FHEM/95_remotecontrol.pm|()
FHEM/98_GEOFANCY.pm|()
FHEM/98_HTTPMOD.pm|(svn:executable 1 *)
FHEM/98_JsonList2.pm|()
FHEM/98_Text2Speech.pm|()
FHEM/98_WeekdayTimer.pm|()
FHEM/98_cmdalias.pm|()
FHEM/98_pilight.pm|()
FHEM/GPUtils.pm|()
FHEM/MaxCommon.pm|()
FHEM/ONKYOdb.pm|()
FHEM/OWX_DS2480.pm|()
FHEM/OWX_Executor.pm|()
FHEM/OWX_FRM.pm|()
FHEM/OWX_SER.pm|()
FHEM/WinService.pm|(svn:executable 1 *)
FHEM/lib|()
FHEM/lib/Device|()
FHEM/lib/Device/Firmata|()
FHEM/lib/Device/Firmata.pm|()
FHEM/lib/Device/Firmata/Base.pm|()
FHEM/lib/Device/Firmata/Constants.pm|()
FHEM/lib/Device/Firmata/Error.pm|()
FHEM/lib/Device/Firmata/IO|()
FHEM/lib/Device/Firmata/IO/NetIO.pm|()
FHEM/lib/Device/Firmata/IO/SerialIO.pm|()
FHEM/lib/Device/Firmata/Language.pm|()
FHEM/lib/Device/Firmata/Platform|()
FHEM/lib/Device/Firmata/Platform.pm|()
FHEM/lib/Device/Firmata/Platform/Arduino.pm|()
FHEM/lib/Device/Firmata/Protocol.pm|()
FHEM/lib/LiquidCrystal.pm|()
FHEM/lib/OWNet.pm|()
FHEM/lib/ProtoThreads.pm|()
FHEM/lib/README.perl-firmata|()
FHEM/lib/SWAP|()
FHEM/lib/SWAP/devices.xml|()
FHEM/lib/SWAP/justme|()
FHEM/lib/SWAP/justme/epd.xml|()
FHEM/lib/SWAP/justme/panfirmata.xml|()
FHEM/lib/SWAP/justme/rgbdriver.xml|()
FHEM/lib/SWAP/justme/statusmon.xml|()
FHEM/lib/SWAP/justme/streamtest.xml|()
FHEM/lib/SWAP/panStamp|()
FHEM/lib/SWAP/panStamp/bininps.xml|()
FHEM/lib/SWAP/panStamp/binouts.xml|()
FHEM/lib/SWAP/panStamp/binouts2.xml|()
FHEM/lib/SWAP/panStamp/chronos.xml|()
FHEM/lib/SWAP/panStamp/lcddriver.xml|()
FHEM/lib/SWAP/panStamp/meter.xml|()
FHEM/lib/SWAP/panStamp/pulsecounter.xml|()
FHEM/lib/SWAP/panStamp/pulsegen.xml|()
FHEM/lib/SWAP/panStamp/respira.xml|()
FHEM/lib/SWAP/panStamp/rgbdriver.xml|()
FHEM/lib/SWAP/panStamp/soilmoisture.xml|()
FHEM/lib/SWAP/panStamp/temp.xml|()
FHEM/lib/SWAP/panStamp/temphum.xml|()
FHEM/lib/SWAP/panStamp/temppress.xml|()
FHEM/lib/SWAP/template.xml|()

Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 24 April 2014, 22:51:07
Es werden immer mehr Module OHNE $Id anstatt weniger...


FHEM/00_FBAHA.pm|(svn:executable 1 *)
FHEM/00_NetzerI2C.pm|()
FHEM/00_OWX_ASYNC.pm|()
FHEM/00_RPII2C.pm|()
FHEM/00_THZ.pm|()
FHEM/02_FRAMEBUFFER.pm|()
FHEM/10_FBDECT.pm|(svn:executable 1 *)
FHEM/10_FRM.pm|(svn:executable 1 *)
FHEM/10_Itach_IR.pm|()
FHEM/10_RESIDENTS.pm|()
FHEM/10_UNIRoll.pm|()
FHEM/16_STACKABLE_CC.pm|(svn:executable 1 *)
FHEM/19_Revolt.pm|()
FHEM/20_FRM_AD.pm|(svn:executable 1 *)
FHEM/20_FRM_I2C.pm|(svn:executable 1 *)
FHEM/20_FRM_IN.pm|(svn:executable 1 *)
FHEM/20_FRM_LCD.pm|(svn:executable 1 *)
FHEM/20_FRM_OUT.pm|(svn:executable 1 *)
FHEM/20_FRM_PWM.pm|(svn:executable 1 *)
FHEM/20_FRM_RGB.pm|()
FHEM/20_FRM_ROTENC.pm|(svn:executable 1 *)
FHEM/20_FRM_SERVO.pm|(svn:executable 1 *)
FHEM/20_FRM_STEPPER.pm|(svn:executable 1 *)
FHEM/20_GUEST.pm|()
FHEM/20_ROOMMATE.pm|()
FHEM/23_LUXTRONIK2.pm|()
FHEM/51_I2C_BMP180.pm|()
FHEM/51_I2C_TSL2561.pm|()
FHEM/51_Netzer.pm|()
FHEM/51_RPI_GPIO.pm|()
FHEM/52_I2C_DS1307.pm|()
FHEM/52_I2C_LCD.pm|()
FHEM/52_I2C_PCA9532.pm|()
FHEM/52_I2C_PCF8574.pm|()
FHEM/52_I2C_SHT21.pm|()
FHEM/70_ENIGMA2.pm|()
FHEM/70_JSONMETER.pm|()
FHEM/70_Jabber.pm|()
FHEM/70_ONKYO_AVR.pm|()
FHEM/70_PHTV.pm|()
FHEM/70_Pushover.pm|()
FHEM/70_XBMC.pm|()
FHEM/73_MPD.pm|()
FHEM/88_Itach_IRDevice.pm|()
FHEM/88_LINDY_HDMI_SWITCH.pm|()
FHEM/91_eventTypes.pm|(svn:executable 1 *)
FHEM/95_Dashboard.pm|()
FHEM/95_remotecontrol.pm|()
FHEM/98_GEOFANCY.pm|()
FHEM/98_HTTPMOD.pm|(svn:executable 1 *)
FHEM/98_JsonList2.pm|()
FHEM/98_Text2Speech.pm|()
FHEM/98_WeekdayTimer.pm|()
FHEM/98_cmdalias.pm|()
FHEM/98_pilight.pm|()
FHEM/FhemUtils/release.pm|()
FHEM/GPUtils.pm|()
FHEM/MaxCommon.pm|()
FHEM/ONKYOdb.pm|()
FHEM/OWX_DS2480.pm|()
FHEM/OWX_Executor.pm|()
FHEM/OWX_FRM.pm|()
FHEM/OWX_SER.pm|()
FHEM/WinService.pm|(svn:executable 1 *)
FHEM/lib/Device/Firmata.pm|()
FHEM/lib/Device/Firmata/Base.pm|()
FHEM/lib/Device/Firmata/Constants.pm|()
FHEM/lib/Device/Firmata/Error.pm|()
FHEM/lib/Device/Firmata/IO/NetIO.pm|()
FHEM/lib/Device/Firmata/IO/SerialIO.pm|()
FHEM/lib/Device/Firmata/Language.pm|()
FHEM/lib/Device/Firmata/Platform.pm|()
FHEM/lib/Device/Firmata/Platform/Arduino.pm|()
FHEM/lib/Device/Firmata/Protocol.pm|()
FHEM/lib/LiquidCrystal.pm|()
FHEM/lib/OWNet.pm|()
FHEM/lib/ProtoThreads.pm|()
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: justme1968 am 24 April 2014, 23:40:04
irgend etwas stimmt an deiner prüfung nicht.

mal stichprobenweise geprüft haben z.b. FBDECT, JsonList2 und WeekdayTimer alle eine $Id.

von denen die keine $Id haben gibt es einige mir leerer $Id. d.h. da wurde 'nur' vergessen die svn property für das file zu setzen.
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 25 April 2014, 09:39:23
*lach*

Zitat von: justme1968 am 24 April 2014, 23:40:04
irgend etwas stimmt an deiner prüfung nicht.

Eine solche Behauptung solltest Du nicht aufstellen, solange Du die Auswahlkriterien für meine Mängelliste nicht kennst und deshalb höchstwahrscheinlich falsche Basisdaten vergleichst.

Zitat von: justme1968 am 24 April 2014, 23:40:04
mal stichprobenweise geprüft haben z.b. FBDECT, JsonList2 und WeekdayTimer alle eine $Id.

Stimmt. Und das ist auch nicht verwunderlich, denn wenn wir ehrlich sind, müssen wir wohl alle zugeben, dass wir ein neues Modul per copy&paste eines vorhandenen Moduls zur Welt bringen und dann den Quelltext verändern. Die Wahrscheinlichkeit, eine Vorlage zu erwischen, in der $Id: schon drinsteht, ist relativ hoch.

Zitat von: justme1968 am 24 April 2014, 23:40:04
von denen die keine $Id haben gibt es einige mir leerer $Id.

Stimmt auch.

Zitat von: justme1968 am 24 April 2014, 23:40:04
d.h. da wurde 'nur' vergessen die svn property für das file zu setzen.

Meine Prüfung erfolgt nicht auf Ebene der Quelltexte, sondern direkt in der SVN Datenbank. Dort selektiere ich "alle Dateien, die in /FHEM oder darunter liegen UND mit .pm enden UND das keyword Id nicht haben"

Ob es einen Eintrag $Id im Quelltext gibt oder nicht, wird dabei nicht berücksichtigt. Meiner Meinung nach sollte jedes in /FHEM eingecheckte Modul eine korrekte Id haben, sonst gehört es da nicht hin.
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: justme1968 am 25 April 2014, 09:48:18
ZitatEine solche Behauptung solltest Du nicht aufstellen, solange Du die Auswahlkriterien für meine Mängelliste nicht kennst und deshalb höchstwahrscheinlich falsche Basisdaten vergleichst.

klar kann ich. so funktionieren behauptungen. sonst müsste ich mich ja den tatsachen beugen:)

aber im ernst. du hast recht und ich nicht aufgepasst. es war scheinbar doch zu spät.
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: maxritti am 25 April 2014, 14:59:16
Hm

wie kommt man denn diesem "leeren" Eintrag in Zeile 4 auf die Schliche?
Irgendein Modul will da wohl etwas nicht verraten.

# $Id: fhem.pl 5632 2014-04-24 19:16:01Z rudolfkoenig $
# $Id: configDB.pm 5601 2014-04-22 18:56:44Z betateilchen $
# $Id: 10_CUL_HM.pm 5621 2014-04-24 06:10:02Z martinp876 $
# $Id:$
# $Id: 93_DbLog.pm 5612 2014-04-23 18:12:07Z tobiasfaust $
# $Id: 01_FHEMWEB.pm 5627 2014-04-24 10:48:11Z rudolfkoenig $
# $Id: 92_FileLog.pm 5452 2014-04-06 06:24:47Z rudolfkoenig $
# $Id: 00_HMLAN.pm 5449 2014-04-05 14:36:30Z martinp876 $
# $Id: 98_HMinfo.pm 5561 2014-04-18 18:32:01Z martinp876 $
# $Id: 98_RandomTimer.pm 5566 2014-04-19 13:10:27Z dietmar63 $
# $Id: 99_SUNRISE_EL.pm 4537 2014-01-03 08:28:59Z rudolfkoenig $
# $Id: 98_SVG.pm 5607 2014-04-23 10:48:59Z rudolfkoenig $
# $Id: 59_Twilight.pm 5519 2014-04-13 19:17:22Z dietmar63 $
# $Id: 99_Utils.pm 5488 2014-04-08 11:32:17Z rudolfkoenig $
# $Id: 90_at.pm 5319 2014-03-25 10:11:47Z rudolfkoenig $
# $Id: 98_autocreate.pm 5268 2014-03-20 20:46:00Z rudolfkoenig $
# $Id: 98_dummy.pm 4934 2014-02-15 08:23:12Z rudolfkoenig $
# $Id: 91_eventTypes.pm 2982 2013-03-24 17:47:28Z rudolfkoenig $
# $Id: 91_notify.pm 5470 2014-04-07 08:32:35Z rudolfkoenig $
# $Id: 33_readingsGroup.pm 5414 2014-04-02 11:55:24Z justme1968 $
# $Id: 98_structure.pm 5050 2014-02-26 08:29:44Z rudolfkoenig $
# $Id: 98_telnet.pm 4844 2014-02-08 07:54:03Z rudolfkoenig $
# $Id: 98_weblink.pm 5608 2014-04-23 10:57:16Z rudolfkoenig $
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: maxritti am 25 April 2014, 15:01:00
Ah, scheint das Dashboard Modul zu sein.

# $Id:$
########################################################################################
#
# 95_Dashboard.pm
#
########################################################################################
# Released : 20.12.2013 @svenson08
# Version :...
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 25 April 2014, 15:24:45
Da fehlt ein Leerzeichen nach $Id:

Solchen Sachen auf die Schliche zu kommen, ist völlig simpel: Ins Log schauen.
Der Befehl "version" loggt im Level 4 alle angefragten Dateien.
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: ntruchsess am 28 April 2014, 14:03:32
Ein nicht unwesentlicher Teil der unten genannten Dateien sind keine autonom ladbaren module im FHEM-Sinn. Teilweise ist es Dateien, die in primär in einem anderen Repository gepflegt werden und zu FHEM nur als Dependency eines FHEM-moduls (da z.B nicht über CPAN installierbar) hinzugefügt worden sind. Teilweise sind es xml-dateien, die einem XML-Schema genügen müssen. Da ist das hinzufügen einer SVN-Id (die sich ja auf das FHEM-repository bezieht) ziemlich kontraproduktiv.

Wäre es nicht sinnvoller den 'version'-Befehl so umzubauen, dass der die Ausgabe von 'svn info' (enthält alles, was im $Id steckt und noch mehr...) entsprechend aufbereitet?

Zitat von: betateilchen am 23 April 2014, 21:37:03
Folgende Dateien in FHEM/ (und darunter) haben keine $Id:

Das executable-Flag muss übrigens nie gesetzt werden.


FHEM/00_FBAHA.pm|(svn:executable 1 *)
FHEM/00_NetzerI2C.pm|()
FHEM/00_OWX_ASYNC.pm|()
FHEM/00_RPII2C.pm|()
FHEM/00_THZ.pm|()
FHEM/10_FBDECT.pm|(svn:executable 1 *)
FHEM/10_FRM.pm|(svn:executable 1 *)
FHEM/10_RESIDENTS.pm|()
FHEM/10_UNIRoll.pm|()
FHEM/16_STACKABLE_CC.pm|(svn:executable 1 *)
FHEM/19_Revolt.pm|()
FHEM/20_FRM_AD.pm|(svn:executable 1 *)
FHEM/20_FRM_I2C.pm|(svn:executable 1 *)
FHEM/20_FRM_IN.pm|(svn:executable 1 *)
FHEM/20_FRM_LCD.pm|(svn:executable 1 *)
FHEM/20_FRM_OUT.pm|(svn:executable 1 *)
FHEM/20_FRM_PWM.pm|(svn:executable 1 *)
FHEM/20_FRM_RGB.pm|()
FHEM/20_FRM_ROTENC.pm|(svn:executable 1 *)
FHEM/20_FRM_SERVO.pm|(svn:executable 1 *)
FHEM/20_FRM_STEPPER.pm|(svn:executable 1 *)
FHEM/20_GUEST.pm|()
FHEM/20_ROOMMATE.pm|()
FHEM/23_LUXTRONIK2.pm|()
FHEM/51_I2C_BMP180.pm|()
FHEM/51_Netzer.pm|()
FHEM/51_RPI_GPIO.pm|()
FHEM/52_I2C_DS1307.pm|()
FHEM/52_I2C_LCD.pm|()
FHEM/52_I2C_PCA9532.pm|()
FHEM/52_I2C_PCF8574.pm|()
FHEM/52_I2C_SHT21.pm|()
FHEM/70_ENIGMA2.pm|()
FHEM/70_JSONMETER.pm|()
FHEM/70_Jabber.pm|()
FHEM/70_ONKYO_AVR.pm|()
FHEM/70_PHTV.pm|()
FHEM/70_Pushover.pm|()
FHEM/70_XBMC.pm|()
FHEM/73_MPD.pm|()
FHEM/88_LINDY_HDMI_SWITCH.pm|()
FHEM/91_eventTypes.pm|(svn:executable 1 *)
FHEM/95_Dashboard.pm|()
FHEM/95_remotecontrol.pm|()
FHEM/98_GEOFANCY.pm|()
FHEM/98_HTTPMOD.pm|(svn:executable 1 *)
FHEM/98_JsonList2.pm|()
FHEM/98_Text2Speech.pm|()
FHEM/98_WeekdayTimer.pm|()
FHEM/98_cmdalias.pm|()
FHEM/98_pilight.pm|()
FHEM/GPUtils.pm|()
FHEM/MaxCommon.pm|()
FHEM/ONKYOdb.pm|()
FHEM/OWX_DS2480.pm|()
FHEM/OWX_Executor.pm|()
FHEM/OWX_FRM.pm|()
FHEM/OWX_SER.pm|()
FHEM/WinService.pm|(svn:executable 1 *)
FHEM/lib|()
FHEM/lib/Device|()
FHEM/lib/Device/Firmata|()
FHEM/lib/Device/Firmata.pm|()
FHEM/lib/Device/Firmata/Base.pm|()
FHEM/lib/Device/Firmata/Constants.pm|()
FHEM/lib/Device/Firmata/Error.pm|()
FHEM/lib/Device/Firmata/IO|()
FHEM/lib/Device/Firmata/IO/NetIO.pm|()
FHEM/lib/Device/Firmata/IO/SerialIO.pm|()
FHEM/lib/Device/Firmata/Language.pm|()
FHEM/lib/Device/Firmata/Platform|()
FHEM/lib/Device/Firmata/Platform.pm|()
FHEM/lib/Device/Firmata/Platform/Arduino.pm|()
FHEM/lib/Device/Firmata/Protocol.pm|()
FHEM/lib/LiquidCrystal.pm|()
FHEM/lib/OWNet.pm|()
FHEM/lib/ProtoThreads.pm|()
FHEM/lib/README.perl-firmata|()
FHEM/lib/SWAP|()
FHEM/lib/SWAP/devices.xml|()
FHEM/lib/SWAP/justme|()
FHEM/lib/SWAP/justme/epd.xml|()
FHEM/lib/SWAP/justme/panfirmata.xml|()
FHEM/lib/SWAP/justme/rgbdriver.xml|()
FHEM/lib/SWAP/justme/statusmon.xml|()
FHEM/lib/SWAP/justme/streamtest.xml|()
FHEM/lib/SWAP/panStamp|()
FHEM/lib/SWAP/panStamp/bininps.xml|()
FHEM/lib/SWAP/panStamp/binouts.xml|()
FHEM/lib/SWAP/panStamp/binouts2.xml|()
FHEM/lib/SWAP/panStamp/chronos.xml|()
FHEM/lib/SWAP/panStamp/lcddriver.xml|()
FHEM/lib/SWAP/panStamp/meter.xml|()
FHEM/lib/SWAP/panStamp/pulsecounter.xml|()
FHEM/lib/SWAP/panStamp/pulsegen.xml|()
FHEM/lib/SWAP/panStamp/respira.xml|()
FHEM/lib/SWAP/panStamp/rgbdriver.xml|()
FHEM/lib/SWAP/panStamp/soilmoisture.xml|()
FHEM/lib/SWAP/panStamp/temp.xml|()
FHEM/lib/SWAP/panStamp/temphum.xml|()
FHEM/lib/SWAP/panStamp/temppress.xml|()
FHEM/lib/SWAP/template.xml|()


Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 28 April 2014, 14:11:36
Zitat von: ntruchsess am 28 April 2014, 14:03:32
Ein nicht unwesentlicher Teil der unten genannten Dateien sind keine autonom ladbaren module im FHEM-Sinn.

Da hast Du völlig Recht, deshalb waren in meiner zweiten hier geposteten Liste auch nur noch Moduldateien mit der Endung .pm enthalten.

Zitat von: ntruchsess am 28 April 2014, 14:03:32
Wäre es nicht sinnvoller den 'version'-Befehl so umzubauen, dass der die Ausgabe von 'svn info' (enthält alles, was im $Id steckt und noch mehr...) entsprechend aufbereitet?

Diese Daten stehen aber in den geladenen Modulen (und nur diese werden von "version" sinnvollerweise überhaupt berücksichtigt) nicht zur Verfügung. Ausserdem wird die Mehrzahl der Anwender von fhem überhaupt kein SVN ausgecheckt haben.
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 07 Mai 2014, 21:38:16
Zur Erinnerung - hier die aktuelle Liste der Module ohne $Id:


FHEM/00_FBAHA.pm|(svn:executable 1 *)
FHEM/00_OWX_ASYNC.pm|()
FHEM/10_FBDECT.pm|(svn:executable 1 *)
FHEM/10_FRM.pm|(svn:executable 1 *)
FHEM/10_Itach_IR.pm|()
FHEM/10_RESIDENTS.pm|()
FHEM/16_STACKABLE_CC.pm|(svn:executable 1 *)
FHEM/19_Revolt.pm|()
FHEM/20_FRM_AD.pm|(svn:executable 1 *)
FHEM/20_FRM_I2C.pm|(svn:executable 1 *)
FHEM/20_FRM_IN.pm|(svn:executable 1 *)
FHEM/20_FRM_LCD.pm|(svn:executable 1 *)
FHEM/20_FRM_OUT.pm|(svn:executable 1 *)
FHEM/20_FRM_PWM.pm|(svn:executable 1 *)
FHEM/20_FRM_RGB.pm|()
FHEM/20_FRM_ROTENC.pm|(svn:executable 1 *)
FHEM/20_FRM_SERVO.pm|(svn:executable 1 *)
FHEM/20_FRM_STEPPER.pm|(svn:executable 1 *)
FHEM/20_GUEST.pm|()
FHEM/20_ROOMMATE.pm|()
FHEM/23_LUXTRONIK2.pm|()
FHEM/52_I2C_DS1307.pm|()
FHEM/52_I2C_LCD.pm|()
FHEM/70_ENIGMA2.pm|()
FHEM/70_JSONMETER.pm|()
FHEM/70_Jabber.pm|()
FHEM/70_ONKYO_AVR.pm|()
FHEM/70_PHTV.pm|()
FHEM/70_Pushover.pm|()
FHEM/70_XBMC.pm|()
FHEM/73_MPD.pm|()
FHEM/88_Itach_IRDevice.pm|()
FHEM/88_LINDY_HDMI_SWITCH.pm|()
FHEM/95_remotecontrol.pm|()
FHEM/98_GEOFANCY.pm|()
FHEM/98_JsonList2.pm|()
FHEM/98_cmdalias.pm|()
FHEM/98_pilight.pm|()
FHEM/98_statistics.pm|()
FHEM/FhemUtils/release.pm|()
FHEM/GPUtils.pm|()
FHEM/MaxCommon.pm|()
FHEM/ONKYOdb.pm|()
FHEM/OWX_DS2480.pm|()
FHEM/OWX_Executor.pm|()
FHEM/OWX_FRM.pm|()
FHEM/OWX_SER.pm|()
FHEM/WinService.pm|(svn:executable 1 *)
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 19 Mai 2014, 19:54:55
Es wird langsam besser :)


FHEM/00_OWX_ASYNC.pm|()
FHEM/10_FRM.pm|(svn:executable 1 *)
FHEM/10_Itach_IR.pm|()
FHEM/10_RESIDENTS.pm|()
FHEM/19_Revolt.pm|()
FHEM/20_FRM_AD.pm|(svn:executable 1 *)
FHEM/20_FRM_I2C.pm|(svn:executable 1 *)
FHEM/20_FRM_IN.pm|(svn:executable 1 *)
FHEM/20_FRM_LCD.pm|(svn:executable 1 *)
FHEM/20_FRM_OUT.pm|(svn:executable 1 *)
FHEM/20_FRM_PWM.pm|(svn:executable 1 *)
FHEM/20_FRM_RGB.pm|()
FHEM/20_FRM_ROTENC.pm|(svn:executable 1 *)
FHEM/20_FRM_SERVO.pm|(svn:executable 1 *)
FHEM/20_FRM_STEPPER.pm|(svn:executable 1 *)
FHEM/20_GUEST.pm|()
FHEM/20_ROOMMATE.pm|()
FHEM/52_I2C_DS1307.pm|()
FHEM/52_I2C_LCD.pm|()
FHEM/70_ENIGMA2.pm|()
FHEM/70_Jabber.pm|()
FHEM/70_ONKYO_AVR.pm|()
FHEM/70_PHTV.pm|()
FHEM/70_Pushover.pm|()
FHEM/70_XBMC.pm|()
FHEM/73_MPD.pm|()
FHEM/88_Itach_IRDevice.pm|()
FHEM/88_LINDY_HDMI_SWITCH.pm|()
FHEM/95_remotecontrol.pm|()
FHEM/98_GEOFANCY.pm|()
FHEM/98_pilight.pm|()
FHEM/GPUtils.pm|()
FHEM/MaxCommon.pm|()
FHEM/ONKYOdb.pm|()
FHEM/OWX_DS2480.pm|()
FHEM/OWX_Executor.pm|()
FHEM/OWX_FRM.pm|()
FHEM/OWX_SER.pm|()
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: ntruchsess am 22 Mai 2014, 00:03:04
jetzt wollte ich betateilchen eine Freude machen und hab in meine Module je eine Zeile '# $Id: $' eingefügt, dann noch bei allen das property svn:keywords auf Id gesetzt und committet. Jetzt ist überall der ': ' weg (nur noch # $Id$ übrig) :-(
Titel: Antw:An alle Entwicklerkollegen: Bitte $Id setzen!
Beitrag von: betateilchen am 22 Mai 2014, 09:00:38
Zitat von: ntruchsess am 22 Mai 2014, 00:03:04
jetzt wollte ich betateilchen eine Freude machen

es geht überhaupt nicht darum, mir eine Freude zu machen...

Zitat von: ntruchsess am 22 Mai 2014, 00:03:04
und hab in meine Module je eine Zeile '# $Id: $' eingefügt,

prima. Und irgendwie hast Du es ja doch noch geschafft und die Liste ist auf einen Schlag sehr viel kürzer geworden :)


FHEM/10_Itach_IR.pm|()
FHEM/10_RESIDENTS.pm|()
FHEM/19_Revolt.pm|()
FHEM/20_GUEST.pm|()
FHEM/20_ROOMMATE.pm|()
FHEM/70_ENIGMA2.pm|()
FHEM/70_Jabber.pm|()
FHEM/70_ONKYO_AVR.pm|()
FHEM/70_PHTV.pm|()
FHEM/70_Pushover.pm|()
FHEM/70_XBMC.pm|()
FHEM/73_MPD.pm|()
FHEM/88_Itach_IRDevice.pm|()
FHEM/88_LINDY_HDMI_SWITCH.pm|()
FHEM/95_remotecontrol.pm|()
FHEM/98_GEOFANCY.pm|()
FHEM/98_pilight.pm|()
FHEM/MaxCommon.pm|()
FHEM/ONKYOdb.pm|()