Eigene Änderungen in den TYPO3-Kern bringen

Inspiring people toshareÄnderungen in den TYPO3-Kern bringen

TYPO3-Camp München - 11./12. September 2010

knipseline / pixelio.de

Inspiring people toshare

Eigene Änderungen in denTYPO3-Kern bringen

Steffen Gebert <[email protected]>

TYPO3-Camp München - 11./12. September 2010

Änderungen in den TYPO3-Kern bringen

mailto:[email protected]

mailto:[email protected]


Einführung

Vorstellung

Steffen Gebert

Student, Freelancer

TYPO3 Core Team Member


Fehler passieren... auch bei TYPO3


Einführung


Bug-Report verfassendetaillierte Problembeschreibung

wo tritt der Fehler auf?

was wird erwartet?

was passiert stattdessen?

Screenshot?

relevante Konfigurationseinstellungen?

eben wie man sich selbst Fehlerbeschreibungen wünscht.. ;-)


Einführung


Bug Report - und was dann?gemeldete Fehler sind gut

beseitigte Fehler noch besser :)

leider mangelnde Resourcen, daher häufig nicht bearbeitet

niemand *muss* Bugs fixen

Ziel

Fehler selbst lösen

In nächstes Release bringen


Einführung


InstrumenteBug Trackerhttp://bugs.typo3.org

Core-Team Mailingliste (typo3.teams.core)news://lists.typo3.orghttp://lists.typo3.org/cgi-bin/mailman/listinfo/typo3-team-core

Subversion Repositoryhttps://svn.typo3.org/TYPO3v4/Core/

http://bugs.typo3.org


http://lists.typo3.org/cgi-bin/mailman/listinfo/typo3-team-core

http://lists.typo3.org/cgi-bin/mailman/listinfo/typo3-team-core

https://svn.typo3.org/TYPO3v4/Core/



Einführung


Workflow

Bug Tracker SVN RepositoryCore Team Liste

PatchReport Reviews Commit

Release


Einen Patch erstellen


Patch erstellenüber SVN-Repository gegen typo3_src-Verzeichnis

nicht diff typo3_src-alt/ typo3_src-gepatcht/

Einspielen mit patch -p0 -i 12345.diff




KommandozeileRepository auscheckensvn checkout https://svn.typo3.org/TYPO3v4/Core/trunk .

Änderung durchführen

Änderungen betrachtensvn diff

Patch erstellensvn diff > bugnummer.diff

https://svn.typo3.org/TYPO3v4/Core/trunk

https://svn.typo3.org/TYPO3v4/Core/trunk




TortoiseSVNRepository auschecken




TortoiseSVNPatch extrahieren




TYPO3 Coding GuidelinesExtension doc_core_cgl

Patches müssen den CGLs entsprechen

ein paar Auszüge

Einrückungen per Tabs, keine Spaces

keine Whitespaces am Zeilenende

Tipp: Whitespaces in IDE anzeigen lassen




TYPO3 Coding GuidelinesLeerzeichen

Tabs




TYPO3 Coding GuidelinesUppercase Booleans

String-Vergleiche mit ===

nur geänderte Zeilen an CGL anpassen


Senden an die Core-Liste


Bug Tracker aktualisierenPatch anhängen

Tag “pending in core list” hinzufügen




Die Core Team Mailingliste/Newsgroupseit ca. 2 Jahren offen für jeden

jeder darf Reviewen und Patches senden

Mailinglisten-Regeln

Requests for Comments (RFCs)

angelehnt an Internetstandards der IETF

vorgegebenes Nachrichtenformat auf der Core-Liste




Requests for Comments

Text

Betreff: RFC #BUGNUMMER: BUGTITEL

“This is a SVN patch request.”

Type: Bugfix / Feature / Code cleanup

Bug Tracker Link

Branches

Patch im Anhang




RFC: BranchesZweige für jeweilige Major-Version(Verzeichnis branches/)

Aktueller Entwicklungszweig: trunk/

Ggf. getrennte Patches für unterschiedliche Branches mitschicken

Neue Features *nur* für trunkNur Bugfixes für veröffentlichte Versionen




RFC: Beschreibung und LösungProblemProblembeschreibung: Alle nötigen Details, aber nicht zu viel Text!

SolutionErklärung der Änderung

Optional: NoteHinweise, ausführliche Erklärung, How to reproduce, Step by step

sehrempfehlenswert!




VotingsAndere Programmierer müssen Änderung absegnen

Zustimmung durch Vergabe von “+1”

by reading by testing

Core Team Mitglied +1 +1

beliebiger Reviewer +1 +1

Fehler beseitigt UND keine

neuen Probleme

Durchsicht des Quellcodes




Reviews & Votingshäufig Aufforderung zur Überarbeitung

überarbeiteten Patch formlos in Antwort senden

Aus diesem Grund: Patch nicht nur im Bug Tracker anhängen, sondern selbst an die Liste senden, um bei Rückfragen reagieren zu können




Keine Reaktion..?nicht jeder Patch wird sofort begutachtet

Beschreibung verständlich?

“How to reproduce” vorhanden?

REMINDER nach 7 Tagen Schweigen erlaubt

Kontaktnetzwerk aufbauen, potenzielle Reviewer ansprechen




Committed to..Nach erfolgreichem Review - Bearbeitung durch Kernentwickler

Aktualisierung ChangeLog

Einpflegen in Subversion Repository

Schließen des Bug Reports

Hinweis im Mailinglistenthread




Bug DaysProbleme bei der Lösungsfindung?

Unterstützung durch erfahrene Entwickler

Versuch, neue Entwickler an die Core-Arbeit heranzuführen

Immer letzter Freitag im Monat

http://wiki.typo3.org/wiki/BugDay

irc://irc.freenode.net/#typo3-bugday




Schattenseiten...


Core Liste ist unübersichtlichgroße Anzahl Mails

viel Diskussion

nicht erkennbar, ob schon comitted

Feedback von mir erwartet?

Patches hin- und herschicken


Git & Gerrit


Git und GerritGit

Ablösung von Subversion

verteilte Versionskontrolle

Gerrit

Review-Server

Webinterface

von Google für Android-Entwicklung entwickelt, genutzt u.a. bei Eclipse-Entwicklung


Git & Gerrit


Git und GerritFLOW3/Phoenix-Team nutzt Git/Gerrit seit aktuellem Sprint-Release

zur T3CON Veröffentlichung des Sprint-Releases

Core Team Meeting, vermutlich mit Bericht über Erfahrungen

Wunsch: Einführung auch für TYPO3 v4 (nach 4.5-Release?)


Git & Gerrit


Gerrit

https://review.typo3.org




Git & Gerrit


Gerrit

reading

testing

Patch-versionen

Branches

Git-checkout Befehl


Git & Gerrit


GerritCommit auf Klick

vernünftige Diff-Ansicht

zeilengenaues Kommentieren


WeblinksBug Trackerhttp://bugs.typo3.org

Core-Team Mailingliste (typo3.teams.core)news://lists.typo3.org

Core Mailinglist Ruleshttp://typo3.org/teams/core/core-mailinglist-rules/

TYPO3 Coding Guidelineshttp://typo3.org/documentation/document-library/core-documentation/doc_core_cgl/current/

Artikel auf typo3blogger.dehttp://typo3blogger.de/ab-in-den-core/

SVN Repositoryhttps://svn.typo3.org/TYPO3v4/Core/



http://typo3.org/teams/core/core-mailinglist-rules/

http://typo3.org/teams/core/core-mailinglist-rules/

http://typo3.org/documentation/document-library/core-documentation/doc_core_cgl/current/




http://typo3blogger.de/ab-in-den-core/

http://typo3blogger.de/ab-in-den-core/




?????????????

inspiring people to share.

Technology

Eigene Änderungen in den TYPO3-Kern bringen