Le 20/05/11 10:59, Thierry Geraud a écrit :
--- milena/ChangeLog | 5 +++++ milena/img/lena.pgm | 3 ++- 2 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/milena/ChangeLog b/milena/ChangeLog index b3cd42e..251f5e1 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,3 +1,8 @@ +2011-05-20 Thierry GERAUD <thierry.geraud@lrde.epita.fr> + + * img/lena.pgm: Replace by official version; + see http://www.ece.rice.edu/~wakin/images/
OK, mais il aurait mieux valu pousser ça dans la branche « next » que dans « master » (ne serait-ce que parce que ça fait diverger « master » de « next », pas cool). On ne va pas annuler le patch parce que la branche est publique, mais il faut éviter ça à l'avenir. Par ailleurs : - certains tests reposent peut-être sur milena/img/lena.pgm comme entrée, et il faut vérifier qu'il passent encore ; - il existe d'autres versions de cette image (binaire, en couleurs, niveaux de gris sur 16 bits, plus petite, etc.) dans milena/img/ ; on a peut-être envie de les « synchroniser » (i.e., régénérer les images dérivées à partir du nouvel original) - c'est anecdotique, mais l'entrée de ChangeLog et le message Git de ce patch sont dissimilaires (le premier devrait être utilisé pour produire le dernier). Une propale pour simplifier ça : donner un nouveau nom à la nouvelle image (genre milena/img/lena-orig.pgm) et remettre l'ancienne image. Dans la foulée, il serait bon de tenir milena/img/README à jour.