Wiki-Artikel zu SVN Nutzungsregeln

[Markus Bloch]

Hallo zusammen,

ich habe aufgrund einiger neu hinzugekommenen Regeln bei der SVN Nutzung einen Wiki-Artikel zur SVN Nutzung verfasst:

http://www.fhemwiki.de/wiki/SVN_Nutzungsregeln

Ich würde euch bitten, da mal drüber zu schauen, ob das alles verständlich/vollständig ist.

Kritik/Kommentare sind herzlich willkommen.

Vielen Dank

Gruß
Markus

[HCS]

Ich finde es verständlich und übersichtlich. Ob es vollständig ist, vermag ich nicht zu 100% beurteilen.
Gab es da nicht noch Regeln, die den Comment beim svn checkin prüfen?

[Markus Bloch]

Stimmt, ganz vergessen. Siehst du, gut das du es gesagt hast

Hab ich ergänzt.

Gruß
Markus

[rudolfkoenig]

Finde ich auch sehr schoen. Ich wuerde noch erwaehnen, dass man vor dem Einchecken generell ein "svn diff" machen, und die Ausgabe durchlesen sollte: das verhindert, dass man Unbeabsichtigtes eincheckt, und man kann nochmal kurz ueber die Aenderungen nachdenken

[Markus Bloch]

Habe ich noch als grüne Info-Box spendiert.

Gruß
Markus

[Risiko]

Für mich auch völlig verständlich. Super Arbeit.
Hätte ich auch gern am Anfang gehabt 

[Dr. Boris Neubert]

Danke, Markus.

Ich meine, dass es noch weitere (nicht unumstrittene) pre-commit hooks gibt außer den Check auf einen syntaxkonformen Kommentar und das Vorhandensein der Id. Leider habe ich nach 5 Minuten Suche bei SF immer noch nicht die Stelle gefunden, wo man das einstellen/einsehen kann, sonst hätte ich die Regelungen hier gepostet.

Kann mir bitte jemand sagen, wo die pre-commit hooks definiert werden? Danke.

Viele Grüße
Boris

[dev0]

Ich fände einen kuzen step-by-step svn command line guide zum (erstmaligen) einchecken noch hilfreich. Falls das schon an anderer Stelle beschrieben ist, dann habe ich es nicht gefunden und ein Link dorthin würde helfen.

/Uli

[zap]

Ich hatte mal <div> ziemlich am Anfang der Commmandref stehen. Wurde auch korrekt am Ende geschlossen. Das Commit war erst möglich, nachdem ich die divs entfernt hatte.

Grundsätzlich zum Einchecken eines neuen Moduls: Ich hatte mal irgendwo gelesen, dass man erst um Erlaubnis fragen muss, bevor man ein neues Modul in den FHEM Zweig eincheckt (nicht im Contrib). Davon steht im Wiki Artikel nichts. Gilt das nicht mehr?

VG Dirk

[rudolfkoenig]

Das Commit war erst möglich, nachdem ich die divs entfernt hatte.

Das ist nicht unbedingt noetig, es sei denn, die divs haben Attribute (z.Bsp. Style). Style gehoert meiner Ansicht nach ins CSS und nicht in die Tags.

erst um Erlaubnis fragen muss, bevor man ein neues Modul in den FHEM Zweig eincheckt

Das ist nicht noetig, es sei denn, man will Bibliotheken anschleppen, ader neue Verzeichnisse anlegen.
Beide sind keine reine Formalien, und es wird darueber ernst nachgedacht, ob es notwendig ist.

[rudolfkoenig]

Leider habe ich nach 5 Minuten Suche bei SF immer noch nicht die Stelle gefunden, wo man das einstellen/einsehen kann, sonst hätte ich die Regelungen hier gepostet.

Soweit mir bekannt, kann man die Regeln nur ueber ein shell-account setzen.
Eine Kopie der Datei ist in contrib/pre-commit

[Dr. Boris Neubert]

Danke, Rudi, gucke ich mir mal an.

Christian hat noch vorgeschlagen, in die SVN-Doku aufzunehmen, dass bei gravierenden Änderungen ein Beitrag im Board Ankündigungen erstellt wird.

Weil nach einem Tutorial gefragt wurde: ich lese das hier http://svnbook.red-bean.com/index.de.html

[viegener]

Klasse Arbeit, das baut auch weiter Einstiegshürden ab!

Einträge im Board Ankündigungen bei grossen Änderungen finde ich sehr erwähnenswert, auch wenn natürlich nicht alle die Ankündigungen lesen 

Sollte noch etwas zu contrib aufgenommen werden (also z.B. Unterschied zwischen contrib und "offiziellen" Modulen oder Wo gehört anderer Content hin, also Firmwares/Bilder/...). Aber vielleicht wäre das einen eigenen Artikel wert?

[Dr. Boris Neubert]

Hallo,

hier die Liste der Prüfungen, die beim Checkin vorgenommen werden.

Grüße
Boris


A FHEM SVN comment must have the following format
  module: text-describing-the-change
or
  module: text-describing-the-change (Forum #<forum.fhem.de threadnumber>)


CHANGED file must not contain tabulators

CHANGED file must not have over 80 characters per line

Files FHEM/*.pm must have SVN Id

No DOS line encoding for files FHEM/*.pm

Files FHEM/*.pm must have empty line after =begin html

Files FHEM/*.pm with =begin html must contain content and <a name="...">...</a> link

No unbalanced tags in documentation in FHEM/*.pm

[Markus Bloch]

Hallo,

hier die Liste der Prüfungen, die beim Checkin vorgenommen werden.

Grüße
Boris


A FHEM SVN comment must have the following format
  module: text-describing-the-change
or
  module: text-describing-the-change (Forum #<forum.fhem.de threadnumber>)


CHANGED file must not contain tabulators

CHANGED file must not have over 80 characters per line

Files FHEM/*.pm must have SVN Id

No DOS line encoding for files FHEM/*.pm

Files FHEM/*.pm must have empty line after =begin html

Files FHEM/*.pm with =begin html must contain content and <a name="...">...</a> link

No unbalanced tags in documentation in FHEM/*.pm

Ist alles im Artikel erfasst und erklärt. Sollte noch weiterer Bedarf, dann gerne Bescheid sagen.

Gruß
Markus