nicodache a écrit :
voici le patch réalisé par diff -uNr source cible > fichier.patch avec :
- source : cvs officiel daté du 8 mars 2005 à 17h40
- cible : mes sources updatées du cvs officiel du 8 mars 2005 à 17h40
avec la commande cvs udpate -dP, et résolution des conflits manuellement
Bonjour,
tout d'abord c'est sympa de contribuer à GLPI, on ne l'as pas encore dit
sur la mailing liste, nous sommes content que la SSLL Opensides.be nous
file un petit coup de main sur GLPI en nous proposant de nouvelles idées
ainsi qu'en contribuant au code.
D'un point de vue plus technique, ce patch ne répond pas à un ensemble
de points auquels nous tenons :
La forme :
- C'est jamais agréable de recevoir un patch dans un mail sans aucune
explications, pensez au personnes qui reçoivent aussi cette mailing
liste, et qui voient passer un patch sans meme savoir de quelle
fonctionnalité il retourne.
- Autre chose aussi, les patch en -uNr, c'est vraiment compliqué pour
nous à integrer/debugguer étant donné la mobilité actuelle du CVS et le
nombre de fichiers impactés. Un tar comprenant les fichiers modifiés
complets sera bien plus simple de notre coté.
Nous pouvons vous donner un exemple du ce qu'on appelerai le "patch
ideal", il suffit de regarder (sur les archives de cette mailing liste)
la contribution de Karine Tavernier du 11.10.2004 , qui à été vraiment
agréable à tester, mettre en place et qui est actuellement la base de
tout ce que nous avons fait sur le module financier.
Le fond :
- Nous pensions que nous nous étions mis d'accord pour considerer qu'un
service (par exemple server web sur le port 80) n'était rien de plus
qu'un logiciel (software) qui ecoute sur un port particulier à l'aide
d'un protocole particulier (du moins ça nous semble être la réalité),
pourquoi avoir séparé les logiciels et les services ?
Sans vraiment reposer le probleme il serait interressant et constructif
que tous ceux qui se sentent plus ou moins concernés par ce genre de
features nous donnent leur avis à ce sujet la, il y a environ 80
personnes inscrites sur cette mailing liste, exprimez vous !!!
Enfin et c'est loin d'être un point accéssoire, le code :
Il y a beaucoup de bugs, des variables ne sont pas déclarées avant
utilisations,.... Il est indispensable de passer le error_reporting =
E_ALL dans le php.ini, et d'une manière globale de tourner sur une
configuration la plus "standard" possible, sinon c'est la porte ouverte
à toutes les erreurs.
Bref, pour les raisons sus-citées nous n'appliquons pas ce patch, et
comme nous n'avons pas le temps pour faire du debugging pour cette
fonctionnalité, d'autres étant beaucoup plus prioritaires, nous vous
laissons la lattitude de corriger ce qui ne va pas et nous retransmettre
vos travaux, si néanmoins vous le souhaitez.
Cordialement,
--
L'equipe de developpement de GLPI
