IRC logs of #tryton-fr for Thursday, 2012-04-19

chat.freenode.net #tryton-fr log beginning Thu Apr 19 00:00:01 CEST 2012
cedksmoldersan: ça avance le patch?15:06
sisalpcedk: le fpdf pour les documents fiscaux était proposé au gsoc ?15:08
nicoesisalp: non15:10
sisalpnicoe: merci, alors c'est une erreur15:11
sisalpjuste une confirmation : il y a bien bilan, compte de resultat et rapport de tva dans la compta standard de tryton ?15:11
smoldersancedk: Pour l'instant je refais une install clean de tryton/trytond avec mecurial.15:19
smoldersanJ'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. :)15:19
cedksmoldersan: c'est assez simple, DatabaseIntegrityError est importé et puis remis à None15:28
smoldersancedk: Je ne comprend pas... Il ne suffit pas de simplement supprimer 'DatabaseIntegrityError = None' ?16:06
cedksmoldersan: si mais pour bien faire faut tester16:13
smoldersancedk: ok!16:15
nicolastrcedk: 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?16:35
cedknicolastr: non, il suffit de mettre: review1234516:37
cedksmoldersan: the state testing is when there is a patch to test16:41
cedksmoldersan: l'état testing c'est quand il y a un patch à tester16:41
smoldersancedk: ok...16:44
nicolastrcedk: J'ai posté un patch qui corrige le bug mais j'ai mis en évidence un autre bug avec les numéros SIRET et SIREN16:53
cedknicolastr: s'il y a un autre bug, il faut créer une autre issue16:58
nicolastrcedk: ok, même si ça concerne quand même la vérification de la taille? https://bugs.tryton.org/issue234217:01
cedknicolastr: alors je comprend pas le problem que tu décrit17:04
cedknicolastr: pour moi, ça n'a rien avoir avec la longueur17:05
nicolastrcedk: Oui c'est vrai, je crée une autre issue alors?17:06
nicolastrcedk: Je suppose que oui vu ton commentaire sur celle-ci17:07
cedknicolastr: oui17:07
nicolastrcedk: tu aurais un autre bug facile pour moi? J'essayerai de le faire plus tard dans la soirée.17:16
smoldersancedk: patch envoyé17:55
cedknicolastr: termine d'abord celui-ci, il faut faire le commit et l'exported18:06
cedksmoldersan: je check18:06
smoldersancedk: je n'ai pas encore fait le commit et l'export non plus18:09
nicoesmoldersan: corrige les petites fautes qu'on a trouvé18:24
nicoenicolastr: ton patch est OK, il ne reste plus qu'à suivre la procédure pour qu'il soit inclus18:24
smoldersannicoe: ok, je regarde18:25
nicoesmoldersan: normallement tu dois recevoir un email ;)18:32
smoldersannicoe: oui je confirme. :)18:34
smoldersannicoe: pour la modification du patch, quel type de message je dois mettre dans ce cas?18:35
smoldersannicoe: quelque chose comme "patch modified accordingly to nicoe comments" convient?18:35
nicoesmoldersan: oui18:35
smoldersanok, merci18:36
nicoemais si c'est plus complexe un "remove OperationalXXXX too"18:36
nicoeune description quoi ;)18:36
smoldersanok18:37
smoldersannicoe: je peux faire un hg commit et export de ces modifs?18:48
nicoesmoldersan: oui avec une bonne description et tout et tout18:51
smoldersannicoe: bien sur! :)18:52
nicoesmoldersan: dis je n'ai pas encore vu passer ton patch c'est normal ?19:07
smoldersannicoe: j'ai fait le commit19:08
smoldersanet je regarde comment faire pour l'export19:08
nicoehttp://code.google.com/p/tryton/wiki/HowtoContribute19:09
nicoeACTION va s'en aller ...19:09
smoldersanje donne quel nom à mon changeset?19:09
pilouau nom de fichier contenant le patch ?19:13
smoldersanoui19:14
cedknicolastr, smoldersan: n'oubliez pas d'uploader vos patch19:20
cedksinon, je ne peux pas les référencer pour le GSoC19:20
smoldersancedk: fait à l'instant19:21
nicolastrcedk: Il faut juste faire le commit directement dans mercurial?19:21
nicolastrcedk: comme ceci? http://code.google.com/p/tryton/wiki/HowtoContribute#Mercurial_Usage19:22
cedknicolastr: oui19:23
cedksmoldersan: I don't understand the commit message19:28
cedksmoldersan: je comprend pas le message de commit19:28
cedkACTION n'arrive pas à switcher de langue19:28
cedksmoldersan: le message est faux19:28
cedkle problème n'est pas que mysql redéfinisse DatabaseIntegrityError19:29
cedkmais qu'il le fasse 2 fois dont la second à None19:29
smoldersanok19:30
cedksmoldersan: de plus ça ne sert à rien de mettre des chemin de fichier19:30
cedksmoldersan: le message doit just expliquer pourquoi le changement, pas le contenu de celui-ci19:30
cedksmoldersan: tu peux faire un: hg rollback et refaire le commit19:31
smoldersancedk: Ok, je fais ça19:31
nicolastrcedk: 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 module19:36
cedknicolastr: une description longue dépend si le message court explique suffisement ou pas19:37
cedknicolastr: il ne faut pas préfixer par le module19:38
nicolastrcedk: ok. Tu peux appliquer les patchs directement depuis le bugtracker ou bien tu dois les appliquer à la main?19:44
nicolastrcedk: J'ai ajouté le patch19:45
cedknicolastr: j'import à la main19:48
smoldersancedk: J'ai modifié le message et réuploadé le patch, j'espère que c'est correct maintenant! :)19:49
cedksmoldersan: tu n'a pas simplement éditer mais bien fait un rollback et puis commit?19:50
smoldersancedk: oui, rollback, commit, export puis re-upload19:50
nicolastrcedk: C'est bon maintenant pour nos patchs? :-)19:56
cedknicolastr: ok20:15
-!- smoldersan(~smoldersa@as76-240.tontut.fi) has left #tryton-fr20:22
-!- nicolastr(~nicolas@82-212-129-87.teledisnet.be) has left #tryton-fr20:53

Generated by irclog2html.py 2.11.0 by Marius Gedminas - find it at mg.pov.lt!