En relisant certaines entrées de ChangeLog, je m'aperçois qu'il y a du relâchement ! Certaines entrées sont bâclées, voire non remplies ! Ce n'est pas admissible. Olena n'est pas un projet kleenex, on a besoin que vous remplissiez les ChangeLog correctement et précisément pour 1. suivre l'activité du projet en général ; 2. suivre votre travail en particulier ; 3. retrouver l'information plus tard (typiquement, dans plusieurs semaines/mois/années), ce qui est capital (et beaucoup plus courant que vous ne le pensez). Merci de lire attentivement ce qui suit, et de mettre en pratique ! * La première chose à faire est de revoir les fondamentaux. Relisez donc : - https://www.lrde.epita.fr/dload/guide/guide.html#htoc66 - http://www.gnu.org/prep/standards/html_node/Change-Logs.html * Ensuite, regardez comment les autres entrées de ChangeLog sont écrites. Vous pouvez prendre exemple sur Akim, Alexandre, Théo et moi-même, qui avons l'habitude de cet exercice. * Les outils sont là pour vous aider. En particulier, Emacs peut vous aider à pré-remplir un ChangeLog plus facilement avec le mode ad hoc : M-x change-log-mode Pour éviter les lignes-qui-dépasse-les-80-colonnes, utilisez M-x auto-fill-mode Quant aux problèmes d'espace (à traiter directement sur le fichier ChangeLog dans un patch à part, le cas échéant), utilisez M-x whitespace-cleanup * Corrigez vos erreurs de ChangeLog au plus vite (i.e., pas trop lomgtemps après le patch contenant ces erreurs), *dans un patch séparé* et *ne faites pas mention de ces changements dans le ChangeLog lui-même* ! Pour cela, il vaut mieux éviter d'utiliser Vcs ou svn-wrapper : utilisez « \svn » au lieu de « svn » pour déconnecter temporairement votre alias « svn » : \svn commit -m "Clean up ChangeLog" ChangeLog Puis n'oubliez pas de poster les changements ; par ex. : svn diff -r PREV >diffs.patch et postez diffs.patch dans olena-patches@lrde.epita.fr dans un message qqch qui ressemble à ce que génerent Vcs et svn-wrapper. * Enfin, voici quelques erreurs classiques, qu'il ne *faut pas* repoduire. Merci d'avance !
Index: ChangeLog =================================================================== --- ChangeLog (revision 1821) +++ ChangeLog (revision 1822) @@ -6,20 +6,24 @@ * sandbox/jardonnet/test/bin: New Bin directory. * sandbox/jardonnet/test/Makefile: Update for testes.
- * sandbox/jardonnet/test/icp_lazy.cc: New dedicated to lazy_map. - * sandbox/jardonnet/registration/icp_lazy.hh: New dedicated to lazy map. + * sandbox/jardonnet/test/icp_lazy.cc: New dedicated to lazy_map. + * sandbox/jardonnet/registration/icp_lazy.hh: New dedicated to + lazy map.
Attention à l'indentation: dans ChangeLog, les lignes correspondant aux fichiers démarrent par une tabulation (C-i), puis par une étoile (*) et une espace, et enfin le nom du fichier.
* sandbox/jardonnet/registration/quat7.hh: Update. * sandbox/jardonnet/registration/chamfer.hh: Update. - * sandbox/jardonnet/registration/icp.hh: Update (dedicated to distance_map). + * sandbox/jardonnet/registration/icp.hh: Update (dedicated to + distance_map).
- * sandbox/jardonnet/registration/misc.hh: Move outside quat/ (removed). - * sandbox/jardonnet/registration/rotation.hh: Move outside quat/ (removed). + * sandbox/jardonnet/registration/misc.hh: Move outside quat/ + (removed). + * sandbox/jardonnet/registration/rotation.hh: Move outside quat/ + (removed).
Prière de ne pas faire du cinémascope (plus de 80 colonnes), tant dans le code que dans les ChangeLogs. [...]
2008-04-02 Caroline Vigouroux <vigour_c@epita.fr>
- change RGB encoding from int_u to float01_. + Change RGB encoding from int_u to float01_.
Attention aux majuscules, et à la ponctuation en général. [...]
* sandbox/vigouroux/color/rgb_to_hsi.hh: .
Remplir *toutes* les lignes! Laisser des ligne vides est inadmissible.
@@ -1277,7 +1293,7 @@
Chamfer DT visual tests (beautiful images inside !). * sandbox/folio/chamfer_dt.cc: New visual tests inside - (Beware! There's a picasso painting here !). + (Beware! There's a picasso painting here !).
Attention aux espaces.
@@ -10993,11 +11013,11 @@ 2007-09-06 Matthieu Garrigues <garrigues.matthieu@lrde.epita.fr>
add println_with_border - * mln/debug/println.hh: - add 3 function : - println_with_border(const box2d& b, const Fast_Image<I>& input_) - impl::println_with_border(const S&, const Fast_Image<I>& input_) - impl::println_with_border(const box2d& b, const Fast_Image<I>& input_) + * mln/debug/println.hh + (println_with_border(const box2d&, const Fast_Image<I>&)) + (impl::println_with_border(const S&, const Fast_Image<I>&)) + (impl::println_with_border(const box2d&, const Fast_Image<I>&)): + New functions. * tests/println_with_border.cc: New.
Utiliser les idiomes des ChangeLogs. Lorsque vous voulez parler d'un individu `bar' (une fonction, par ex.) dans un fichier `foo.cc' par exemple, ne faites pas ça : * foo.cc: Add a new function `bar'. Mais faites ceci : * foo.cc (bar): New function.