Fabien Freling <fabien.freling(a)lrde.epita.fr> writes:
2009/9/7 Roland Levillain
<roland(a)lrde.epita.fr>fr>:
Alexandre Duret-Lutz <adl(a)lrde.epita.fr>
writes:
>>
"FF" == Fabien Freling <fabien.freling(a)lrde.epita.fr> writes:
FF> URL:
https://svn.lrde.epita.fr/svn/oln/trunk/milena/sandbox
FF> ChangeLog:
FF> 2009-09-04 Fabien Freling <fabien.freling(a)lrde.epita.fr>
FF> Implement HQ2x upscaling algorithm.
FF> * fabien/mln/upsampling/hq2x.hh: Implement HQ2x algorithm.
FF> * fabien/mln/upsampling/hq3x.hh: Remove dead code.
FF> * fabien/tests/upsampling/Makefile: Add `hq2x' target.
FF> * fabien/tests/upsampling/hq2x.cc: New test file for HQ2x.
Je suis assez choqué. Ce code est une adaptation d'un code
de Maxim Stepin sous LGPL 2.1 : on reconnaît clairement la
logique du code original, même si les accès à l'image ont changé.
Cependant
1) le nom de l'auteur n'apparaît nul part
2) son copyright a été modifié en (c) LRDE
3) la license a été changée en GPL 2 + exception.
Le dépôt SVN de milena est public. Tu ne peux pas prendre le
code de quelqu'un et le redistribuer sous une autre license
comme ça !
J'appuie. Il faut faire très attention à ça.
Lorsque l'on reprend du code, il faut laisser le copyright header à
l'identique : ce n'est pas parce que tu modifies un fichier que tu en
deviens légalement l'auteur.
Et ça fait partie des avantages du libre : on peut panacher des bouts de
code à condition de respecter le droit d'auteur et les termes de la
licence.
J'étais justement en train de reprendre les headers.
Ce premier commit etait surtout histoire de sauvegarder le travail.
Je comprends. Mais comme le dépôt est public, il faut qu'on fasse
attention avec ça.
Je vais refaire encore un peu d'évangélisation pour Git ;) mais il
s'agit là de l'un des cas d'utilisations que ce dernier sait bien gérer.
Tu peux en effet créer un certain nombre de commits locaux, puis,
lorsque tu es content du résultat, fusionner tout ou partie de ces
commits, avant de les envoyer au serveur. Ainsi, tu peux masquer des
étapes intermédiaires qui ne sont que le produit du processus de
travail.
Et indépendamment du problème de copyright/license,
quand on
implémente un algo précis, c'est une bonne idée d'en donner les
références (article, page web, ...) dans l'entête.
Absolument ! Il faut penser aux autres (nous + les utilisateurs
externes du projet).
C'est prévu dans le prochain commit avec les références nécessaires.
J'aurais du le faire au début, mea culpa.
OK !
Roland