Community mailing list archives

community@mail.odoo.com

Re: broken Travis builds in OCA repositories

by
Camptocamp SA, Nicolas Bessi
- 03/25/2015 05:25:55
Hello,

I think we have here a nice topic to be discussed during OCA meetup in Odoo days :). 
I guess we should try to address this issue is directly in the core of Odoo in a correct manner. 
We should also include Odoo dev team in this topic because it's a problem they are going to encounter with 
the new app store. 

Fabien and Anthony if you hear me... your welcome to join ;)

Regards

Nicolas


2015-03-25 10:17 GMT+01:00 Stefan <stefan@therp.nl>:
On 25-03-15 08:32, Alexandre Fayolle wrote:
> Hello everyone,
>
> Following the introduction of
> https://github.com/OCA/maintainer-quality-tools/issues/105 in maintainer
> quality tools, some branch and PR builds have been failing.
>
> This is caused by external dependencies being imported by some addons
> without a try...except block in a file which is imported at instance
> startup, which is a bug in that module. Likely a couple of offending
> addons are causing a large number of unrelated failures.
>
> I will make the required PRs and fixes today if noone beats me to it.
> Apologies for the inconvenience.

Hi Alexandre,

thank you for picking this up. When we encountered this in the past, our
solution was to import the external dependency locally in the methods
themselves. This preserves the clear runtime error mentioning the
missing dependency. Would you consider this solution instead of catching
ImportError at the top of the modules?

Cheers,
Stefan


-- 
Therp - Maatwerk in open ontwikkeling

Stefan Rijnhart - Ontwerp en implementatie

mail: stefan@therp.nl
tel: +31 (0) 614478606
web: http://therp.nl

_______________________________________________
Mailing-List: https://www.odoo.com/groups/community-59
Post to: mailto:community@mail.odoo.com
Unsubscribe: https://www.odoo.com/groups?unsubscribe




--
Nicolas Bessi
Senior ERP consultant
Business Solution technical manager

Camptocamp SA
PSE A
CH-1015 Lausanne