| chat.freenode.net #tryton-fr log beginning Thu Apr 19 00:00:01 CEST 2012 | ||
| 2012-04-19 15:06 <cedk> smoldersan: ça avance le patch? | ||
| 2012-04-19 15:08 <sisalp> cedk: le fpdf pour les documents fiscaux était proposé au gsoc ? | ||
| 2012-04-19 15:10 <nicoe> sisalp: non | ||
| 2012-04-19 15:11 <sisalp> nicoe: merci, alors c'est une erreur | ||
| 2012-04-19 15:11 <sisalp> juste une confirmation : il y a bien bilan, compte de resultat et rapport de tva dans la compta standard de tryton ? | ||
| 2012-04-19 15:19 <smoldersan> cedk: Pour l'instant je refais une install clean de tryton/trytond avec mecurial. | ||
| 2012-04-19 15:19 <smoldersan> J'ai déjà jetté un oeil au fichié incriminé, mais je t'avoue que je ne comprend pas bien ce qui cloche, ou comment le résoudre... Si tu as une piste supplémentaire, ça m'intéresse. :) | ||
| 2012-04-19 15:28 <cedk> smoldersan: c'est assez simple, DatabaseIntegrityError est importé et puis remis à None | ||
| 2012-04-19 16:06 <smoldersan> cedk: Je ne comprend pas... Il ne suffit pas de simplement supprimer 'DatabaseIntegrityError = None' ? | ||
| 2012-04-19 16:13 <cedk> smoldersan: si mais pour bien faire faut tester | ||
| 2012-04-19 16:15 <smoldersan> cedk: ok! | ||
| 2012-04-19 16:35 <nicolastr> cedk: Il y a un champ spécial sur https://bugs.tryton.org/ pour inclure le patch uploadé sur http://codereview.tryton.org/ ? Est-ce suffisant de mettre le lien dans le commentaire? | ||
| 2012-04-19 16:37 <cedk> nicolastr: non, il suffit de mettre: review12345 | ||
| 2012-04-19 16:41 <cedk> smoldersan: the state testing is when there is a patch to test | ||
| 2012-04-19 16:41 <cedk> smoldersan: l'état testing c'est quand il y a un patch à tester | ||
| 2012-04-19 16:44 <smoldersan> cedk: ok... | ||
| 2012-04-19 16:53 <nicolastr> cedk: J'ai posté un patch qui corrige le bug mais j'ai mis en évidence un autre bug avec les numéros SIRET et SIREN | ||
| 2012-04-19 16:58 <cedk> nicolastr: s'il y a un autre bug, il faut créer une autre issue | ||
| 2012-04-19 17:01 <nicolastr> cedk: ok, même si ça concerne quand même la vérification de la taille? https://bugs.tryton.org/issue2342 | ||
| 2012-04-19 17:04 <cedk> nicolastr: alors je comprend pas le problem que tu décrit | ||
| 2012-04-19 17:05 <cedk> nicolastr: pour moi, ça n'a rien avoir avec la longueur | ||
| 2012-04-19 17:06 <nicolastr> cedk: Oui c'est vrai, je crée une autre issue alors? | ||
| 2012-04-19 17:07 <nicolastr> cedk: Je suppose que oui vu ton commentaire sur celle-ci | ||
| 2012-04-19 17:07 <cedk> nicolastr: oui | ||
| 2012-04-19 17:16 <nicolastr> cedk: tu aurais un autre bug facile pour moi? J'essayerai de le faire plus tard dans la soirée. | ||
| 2012-04-19 17:55 <smoldersan> cedk: patch envoyé | ||
| 2012-04-19 18:06 <cedk> nicolastr: termine d'abord celui-ci, il faut faire le commit et l'exported | ||
| 2012-04-19 18:06 <cedk> smoldersan: je check | ||
| 2012-04-19 18:09 <smoldersan> cedk: je n'ai pas encore fait le commit et l'export non plus | ||
| 2012-04-19 18:24 <nicoe> smoldersan: corrige les petites fautes qu'on a trouvé | ||
| 2012-04-19 18:24 <nicoe> nicolastr: ton patch est OK, il ne reste plus qu'à suivre la procédure pour qu'il soit inclus | ||
| 2012-04-19 18:25 <smoldersan> nicoe: ok, je regarde | ||
| 2012-04-19 18:32 <nicoe> smoldersan: normallement tu dois recevoir un email ;) | ||
| 2012-04-19 18:34 <smoldersan> nicoe: oui je confirme. :) | ||
| 2012-04-19 18:35 <smoldersan> nicoe: pour la modification du patch, quel type de message je dois mettre dans ce cas? | ||
| 2012-04-19 18:35 <smoldersan> nicoe: quelque chose comme "patch modified accordingly to nicoe comments" convient? | ||
| 2012-04-19 18:35 <nicoe> smoldersan: oui | ||
| 2012-04-19 18:36 <smoldersan> ok, merci | ||
| 2012-04-19 18:36 <nicoe> mais si c'est plus complexe un "remove OperationalXXXX too" | ||
| 2012-04-19 18:36 <nicoe> une description quoi ;) | ||
| 2012-04-19 18:37 <smoldersan> ok | ||
| 2012-04-19 18:48 <smoldersan> nicoe: je peux faire un hg commit et export de ces modifs? | ||
| 2012-04-19 18:51 <nicoe> smoldersan: oui avec une bonne description et tout et tout | ||
| 2012-04-19 18:52 <smoldersan> nicoe: bien sur! :) | ||
| 2012-04-19 19:07 <nicoe> smoldersan: dis je n'ai pas encore vu passer ton patch c'est normal ? | ||
| 2012-04-19 19:08 <smoldersan> nicoe: j'ai fait le commit | ||
| 2012-04-19 19:08 <smoldersan> et je regarde comment faire pour l'export | ||
| 2012-04-19 19:09 <nicoe> http://code.google.com/p/tryton/wiki/HowtoContribute | ||
| 2012-04-19 19:09 <nicoe> ACTION va s'en aller ... | ||
| 2012-04-19 19:09 <smoldersan> je donne quel nom à mon changeset? | ||
| 2012-04-19 19:13 <pilou> au nom de fichier contenant le patch ? | ||
| 2012-04-19 19:14 <smoldersan> oui | ||
| 2012-04-19 19:20 <cedk> nicolastr, smoldersan: n'oubliez pas d'uploader vos patch | ||
| 2012-04-19 19:20 <cedk> sinon, je ne peux pas les référencer pour le GSoC | ||
| 2012-04-19 19:21 <smoldersan> cedk: fait à l'instant | ||
| 2012-04-19 19:21 <nicolastr> cedk: Il faut juste faire le commit directement dans mercurial? | ||
| 2012-04-19 19:22 <nicolastr> cedk: comme ceci? http://code.google.com/p/tryton/wiki/HowtoContribute#Mercurial_Usage | ||
| 2012-04-19 19:23 <cedk> nicolastr: oui | ||
| 2012-04-19 19:28 <cedk> smoldersan: I don't understand the commit message | ||
| 2012-04-19 19:28 <cedk> smoldersan: je comprend pas le message de commit | ||
| 2012-04-19 19:28 <cedk> ACTION n'arrive pas à switcher de langue | ||
| 2012-04-19 19:28 <cedk> smoldersan: le message est faux | ||
| 2012-04-19 19:29 <cedk> le problème n'est pas que mysql redéfinisse DatabaseIntegrityError | ||
| 2012-04-19 19:29 <cedk> mais qu'il le fasse 2 fois dont la second à None | ||
| 2012-04-19 19:30 <smoldersan> ok | ||
| 2012-04-19 19:30 <cedk> smoldersan: de plus ça ne sert à rien de mettre des chemin de fichier | ||
| 2012-04-19 19:30 <cedk> smoldersan: le message doit just expliquer pourquoi le changement, pas le contenu de celui-ci | ||
| 2012-04-19 19:31 <cedk> smoldersan: tu peux faire un: hg rollback et refaire le commit | ||
| 2012-04-19 19:31 <smoldersan> cedk: Ok, je fais ça | ||
| 2012-04-19 19:36 <nicolastr> cedk: Un commentaire court suffit ou il faut aussi un long? Il faut préfixer par le nom du module? Ca serait redondant de le faire vu que le patch sera appliqué dans le dépôt du module | ||
| 2012-04-19 19:37 <cedk> nicolastr: une description longue dépend si le message court explique suffisement ou pas | ||
| 2012-04-19 19:38 <cedk> nicolastr: il ne faut pas préfixer par le module | ||
| 2012-04-19 19:44 <nicolastr> cedk: ok. Tu peux appliquer les patchs directement depuis le bugtracker ou bien tu dois les appliquer à la main? | ||
| 2012-04-19 19:45 <nicolastr> cedk: J'ai ajouté le patch | ||
| 2012-04-19 19:48 <cedk> nicolastr: j'import à la main | ||
| 2012-04-19 19:49 <smoldersan> cedk: J'ai modifié le message et réuploadé le patch, j'espère que c'est correct maintenant! :) | ||
| 2012-04-19 19:50 <cedk> smoldersan: tu n'a pas simplement éditer mais bien fait un rollback et puis commit? | ||
| 2012-04-19 19:50 <smoldersan> cedk: oui, rollback, commit, export puis re-upload | ||
| 2012-04-19 19:56 <nicolastr> cedk: C'est bon maintenant pour nos patchs? :-) | ||
| 2012-04-19 20:15 <cedk> nicolastr: ok | ||
| 2012-04-19 20:22 -!- smoldersan(~smoldersa@as76-240.tontut.fi) has left #tryton-fr | ||
| 2012-04-19 20:53 -!- nicolastr(~nicolas@82-212-129-87.teledisnet.be) has left #tryton-fr |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!