Le Jeudi 10 Mars 2005 01:44, baaZ a écrit : > 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.
Merci.. > 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. ok on mettra un descriptif avec sortie de notre cvs pour faire changelog ;-) > - 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é. ok on peut faire cela.. > 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. pas trouve dans les archives... > 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 ? Parce que dans la pratique de tout les jours les logiciels sont gères indépendamment des services. Les personnes responsables sont différentes et dans différentes équipes. Cela correspond à la réalité tel que nous la vivons tout les jours ;-) > 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 !!! Effectivement, plusieurs points de vues seraient bienvenus > 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 pouvez vous nous fournir des exemples de ce que vous considérez comme des bugs ?? > 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. Notre config de base pour les développements est debian sarge + apache 1.3 +php4. En ce qui concerne l'error reporting on va l'activer sur les postes de développement. > 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. Nous allons faire les changement demandés et nous resoumettrons le code dans le courant de la semaine prochaine. Bonne journée L'équipe OpenSides -- Benoit Mortier Linux Engineer www.opensides.be
