Changes between Version 2 and Version 3 of fr/patchs

Timestamp:

Mar 5, 2008, 5:54:07 PM (13 years ago)

Author:

laurentj

Comment:

--

fr/patchs

@@ -12 +12 @@
 Vous avez vu [wiki:fr/sources toutes les étapes 1 à 5] précédemment. Maintenant voyons pour le patch.
 
-Dans le répertoire des sources issues du dépôt subversion, vous avez donc normalement effectué vos modifications. Placez vous alors dans le répertoire trunk, et tapez ensuite la commande
+== ce que doit contenir votre patch ==
+
+Votre patch contient bien sûr les modifications dans le code, mais pas seulement. Il doit aussi contenir '''les modifications des en-têtes''' (nom, copyright etc..). '''Bien lire attentivement''' [wiki:fr/conventions les conventions de codage].
+
+== création du patch ==
+
+Dans le répertoire des sources issues du dépôt subversion, vous avez donc normalement effectué vos modifications. Placez vous alors dans le répertoire trunk (ou celui d'une branche si c'est une modification sur une branche), et tapez ensuite la commande
 
   svn diff > monpatch.diff
 
-Vous obtenez un fichier monpatch.diff, que vous enregistrerez dans un ticket sur le présent site, soit dans un ticket existant concernant la modification que vous avez effectué, soit dans un nouveau ticket. Suivant la qualité et la pertinence du patch, il sera intégré ou non dans Jelix.
+Vous obtenez un fichier monpatch.diff, que vous proposerez ensuite dans un ticket.
 
-=== ce que doit contenir votre patch ===
+== proposer le patch ==
 
-Votre patch contient bien sûr les modifications dans le code, mais pas seulement. Il doit aussi contenir '''les modifications des en-têtes''' (nom, copyright etc..). '''Bien lire attentivement''' [wiki:fr/conventions les conventions de codage].
+Enregistrez votre fichier patch, soit dans un ticket existant concernant la modification que vous avez effectué, soit dans un nouveau ticket. Vous enregistrez le patch en cliquant sur le bouton "attach file" dans le ticket.
 
+Il se peut que ce soit un patch incomplet (vous n'avez pas le temps de le finir, vous voulez passer la main, ou alors ce n'est qu'une première étape etc...), dans ce cas là, vous vous arrêtez là. (Toutefois un commentaire ne sera pas de trop pour indiquer le pourquoi, où vous en êtes etc..)
 
+Sinon, si votre patch vous semble prêt, selectionnez "review?" dans le champs review du ticket. Cela veut dire que vous demandez une review.
+
+Un "reviewer" regardera alors votre patch, et vous dira en retour si il y a des améliorations à faire (il mettra "review-") ou si c'est bon ("review+").
+
+Si il y a "review-", il faudra faire les améliorations demandées et proposer un nouveau patch que vous attacherez sur le même ticket. Et vous demanderez alors une nouvelle review ("review?"). Et ainsi de suite.
+
+Une fois que vous avez "review+", si vous avez les droits pour commiter dans le dépôt svn, vous incluez alors vos modifications dans le dépôt. Sinon un "commiter" le fera à votre place.