98_Version.pm - Semver Zusätzlich zu REF ID unterstützen

[Sidey]

Hi Betateilchen,


Wäre es möglich, für Module die nicht im SVN liegen auch die Angabe einer semver zu untersützen?

Ich denke, das wäre schon möglich wenn die Regex an diese Stelle https://svn.fhem.de/trac/browser/trunk/fhem/FHEM/98_version.pm#L67 ein bisschen angepasst wird:

Code Auswählen Erweitern

\$Id\: (\S+?) ((\d+?)|v?\d+.\d{0,}.\d{0,}) (\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}Z \S+?) \$

Viele Grüße
Sidey

[betateilchen]

Zwei Bitten:

• verschiebe den Thread bitte in den Developer-Bereich, ich möchte das nicht alleine entscheiden.
• füge ein sinnvolles Beispiel der gewünschten Ausgabe hinzu, damit man sieht, wie sich das auswirkt.

Da in einer semver auch ziemlich viel "Käse" stehen kann, wird die Angabe vermutlich die Ausgabe der Versionstabelle unkontrolliert sprengen. Die verwendete Ausgabe mit sprintf() wird wahrscheinlich nicht mehr passen.

Spontan habe ich gerade keine wirkliche Lust, das zu ändern, zumal die Ausgabe von "version" auch jetzt schon an der einen oder anderen Stelle bei Anwendern für eigene Auswertungen genutzt wird.

[Sidey]

Hallo Betateilchen,

An der Regex müsste ich noch etwas arbeiten und an besagtem sprintf auch, da hast Du recht.
Da das mit der Länge der Versionsangabe natürlich gewisse Herausforderungen mit sich bringt, könnte ich mir folgenden Kompromiss gut vorstellen:

Folgende ID Zeile:

Code Auswählen Erweitern

# $Id: 99_DockerImageInfo.pm v1 2025-03-25 11:20:51Z sidey79 $

Könnte dann zu folgendem Ergebnis führen:

Code Auswählen Erweitern

File                  Rev   Last Change

fhem.pl               29402 2024-12-05 16:03:59Z rudolfkoenig
39_alexa.pm           23820 2021-02-24 19:29:15Z justme1968
98_autocreate.pm      29378 2024-11-30 18:25:51Z rudolfkoenig
99_DockerImageInfo.pm     0 2025-03-25 11:20:51Z sidey79


Wenn das auf Zustimmung trifft, würde ich den Patch dafür auch erstellen.

Grüße Sidey

[Sidey]

@betateilchen


wie wollen wir hier denn weiter machen?

Grüße Sidey

[betateilchen]

wie wollen wir hier denn weiter machen?

Kurzfassung: derzeit gar nicht.
Langfassung:

für Module die nicht im SVN liegen auch die Angabe einer semver

Wenn eine Moduldatei es offenbar nicht "wert" ist, in das offizielle repository (ggf. in ./contrib) eingecheckt zu werden und damit eine reguläre Versionsnummer zu erhalten, verstehe ich nicht, warum man es dann in der für offizielle Module vorgesehenen Liste mit Versionsangaben aufnehmen sollte.

Darüber hinaus stelle ich fest:

• außer Dir hat offenbar niemand ein Interesse an der vorgeschlagenen Änderung (bzw. nichtmal eine Meinung dazu)
• semver sind derzeit in den Entwicklungsstandards von FHEM nicht vorgesehen
• zusätzliche, vom Standard abweichende Versionsangaben sind geeignet, bei Anwendern Verwirrung zu stiften
• die Ausgabe von "version" dient regelmäßig des Debuggings und der Hilfe bei Problemmeldungen hier im Forum. Man sollte sich als Helfender darauf verlassen können, welche einheitlichen Informationen die Ausgabe enthält

Das von Dir gelieferte Beispiel

Code Auswählen Erweitern

# $Id: 99_DockerImageInfo.pm v1 2025-03-25 11:20:51Z sidey79 $Könnte dann zu folgendem Ergebnis führen:

Code Auswählen Erweitern

99_DockerImageInfo.pm     0 2025-03-25 11:20:51Z sidey79


habe ich nicht verstanden. Welchen Sinn soll es haben, als Versionsangabe eine 0 darzustellen, wenn im $Id String doch v1 steht? Sowas würde ich auf keinen Fall umsetzen.

Falls es Dir nur darum geht, alle geladenen Moduldateien (egal ob mit oder ohne $Id) in der Liste zu haben, solltest Du die Anforderung bitte nochmal neu formulieren.

Wie gesagt: Im Moment sehe ich keinen Bedarf, an dieser Stelle irgendetwas zu verändern.

[Sidey]

Wenn eine Moduldatei es offenbar nicht "wert" ist, in das offizielle repository (ggf. in ./contrib) eingecheckt zu werden und damit eine reguläre Versionsnummer zu erhalten, verstehe ich nicht, warum man es dann in der für offizielle Module vorgesehenen Liste mit Versionsangaben aufnehmen sollte.

Das war mir nicht bewusst, dass z.B. github.com/fhem keine offiziellen repositorys sind.

habe ich nicht verstanden. Welchen Sinn soll es haben, als Versionsangabe eine 0 darzustellen, wenn im $Id String doch v1 steht? Sowas würde ich auf keinen Fall umsetzen.

Das war ein Kompromissversuch, weil Du dich zur Formatierung geäußert hattest.
Aktuell wird die ID Zeile ausgegeben, wenn in der Versionsnummer keine valide SVN Revisionsnummer steht.

Falls es Dir nur darum geht, alle geladenen Moduldateien (egal ob mit oder ohne $Id) in der Liste zu haben, solltest Du die Anforderung bitte nochmal neu formulieren.

Das war tatsächlich mein Bedarf neben der REV id auch andere Versionsangaben zu erlauben:
Also so etwas:

Code Auswählen Erweitern

99_DockerImageInfo.pm    v1 2025-03-25 11:20:51Z sidey79
Oder auch sowas:

Code Auswählen Erweitern

99_DockerImageInfo.pm v1.0.0 2025-03-25 11:20:51Z sidey79
Gerne auch die Daten aus dem META block, wenn keine SVN Revisions ID gefunden wurde.

Grüße Sidey