[weboob] win32 console lib

Laurent Dufrechou laurent.dufrechou at gmail.com
Tue Apr 5 14:47:39 CEST 2011


OK.

Pour l'encodage je t'envoi les 2 erreurs que j'ai.
Tu me diras ce que tu en pense.

Le 5 avril 2011 10:49, Christophe Benz <christophe.benz at gmail.com> a écrit :

> Bonjour Laurent,
>
> C'est mieux maintenant, mais je suis contre le fait de quitter (sys.exit)
> si le paquet n'est pas installé. Ce n'est pas une raison suffisante, c'est
> juste pour la présentation. Qu'en pensent les autres développeurs ?
>
> Dans le cas où on choisisse quand même d'obliger l'utilisateur à installer
> colorama, il faut juste laisser planter ImportError, et ajouter une
> dépendance au setup.py
>
> Concernant ta question d'encodage, peux-tu montrer l'erreur qui se produit
> si jamais tu ne "hot-patch" pas les sources ?
> A priori je dirais qu'il faut encoder en UTF-8 en amont de l'appel à
> colorama, ne serait-ce pas un cas où on envoie de l'unicode à afficher ?
>
> Merci,
> à+
>
>
> On 04/04/2011 23:14, Laurent Dufrechou wrote:
>
>> Bonsoir,
>> Je viens d'envoyer la dernière version du patch.
>> J'ai supprimé colorama et ajouté le try...except sous windows.
>>
>> Si des connaisseurs qui gèrent l'unicode savent comment éviter le patch "à
>> la volée" je suis preneur.
>> Je n'ai hélas pas trouvé mieux :/
>> Sinon ca fonctionne bien.
>>
>> Laurent
>>
>> -----Message d'origine-----
>> De : Romain Bignon [mailto:romain at peerfuse.org]
>> Envoyé : lundi 4 avril 2011 14:18
>> À : Laurent Dufrechou
>> Cc : 'Christophe Benz'; weboob at lists.symlink.me
>> Objet : Re: [weboob] win32 console lib
>>
>> On 04/Apr - 13:56, Laurent Dufrechou wrote:
>>
>>> J'ai fais ca car la taille de la dépendance est minime et sous windows
>>> c'est un peu galère d'installer les dépendances.
>>> (sous linux, colorama est inutile dans le sens ou colorama.init()<->
>>> return())
>>> Mais si vous préférez que l'utilisateur l'installe ca me va aussi.
>>> A ca moment la je peux modifier le patch ne le rendre actif que sous
>>> windows et demander a l'utilisateur d'installer colorama.
>>>
>>
>> Si ça n'a pas [encore] d'intérêt sous Linux, je pense en effet que tu peux
>> faire de colorama une dépendance optionnelle.
>>
>> Par contre, je préfèrerais que le init() se fasse dans
>> ConsoleApplication.run plutôt que ConsoleApplication.__init__, je trouve
>> ça
>> a davantage de sens.
>>
>> Par la suite on pourrait proposer un package Windows de Weboob qui
>> inclurait
>> toutes les dépendances, comme c'est coutume sur les sous-OS.
>>
>>  Par contre j'ai vu que quand on lance boobank il liste les dépendances
>>> non résolues, comment puis je faire la même chose?
>>>
>>
>> Je pense qu'il s'agit des messages d'erreurs de chargement des backends.
>>
>> Vu que colorama est cherché par l'application, le même mécanisme ne peut
>> pas
>> être utilisé. Tu peux éventuellement rajouter ça dans le
>> ConsoleApplication.run, un code du genre:
>>
>> if os.platform == 'win32':
>>     try:
>>         import colorama
>>     except ImportError:
>>         print>>sys.stderr, 'Please install colorama blabla'
>>         sys.exit(1)
>>     else:
>>         colorama.init()
>>
>> Est-ce que ça te va ?
>>
>> Romain
>>
>> _______________________________________________
>> weboob mailing list
>> weboob at lists.symlink.me
>> http://lists.symlink.me/mailman/listinfo/weboob
>>
>
> _______________________________________________
> weboob mailing list
> weboob at lists.symlink.me
> http://lists.symlink.me/mailman/listinfo/weboob
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.symlink.me/pipermail/weboob/attachments/20110405/bdde58db/attachment.html>


More information about the weboob mailing list