openerp-expert-framework team mailing list archive

[Christophe Combelles <ccomb@xxxxxxx>]

Le 22/02/2012 11:23, Stéphane Wirtel a écrit :
> Hi Christophe,
>
> I don't like this workaround but the real bug is in the packaging of babel on
> Win32 with py2exe.
>
> if you have another solution, tell me.

ah ok :-/

In that case shouldn't this code be included in the "if os.name == 'nt':" block?
Now the workaround for windows is a bug for linux :(

Maybe it would also be worth adding a few comments explaining what the problem is


>
> Stéphane
>
> On 02/22/2012 11:03 AM, Christophe Combelles wrote:
>> Hi,
>>
>> While trying to install 6.1 I get a babel import error, because there is an
>> import babel in the middle of the setup.py. This is not a good packaging
>> practice because the setup.py file is expected to *define* dependencies, and not
>> *use* dependencies.
>> The best thing is to execute the minimal amount of code in setup.py. With
>> distutil2 now there is no more executed code, juste a static setup.cfg file with
>> a small domain specific language to handle complex requirements (windows, linux,
>> 32/64 bits, etc...).
>>
>> Is this babel thing absolutely necessary?
>>
>> Christophe
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~openerp-expert-framework
>> Post to : openerp-expert-framework@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~openerp-expert-framework
>> More help : https://help.launchpad.net/ListHelp
>
>